Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353001 - [Compatibility] 'forcePluginActivation' attribute not honoured for property testers
Summary: [Compatibility] 'forcePluginActivation' attribute not honoured for property t...
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.1   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 4.2 M1   Edit
Assignee: Paul Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-25 08:32 EDT by Paul Webster CLA
Modified: 2011-07-27 15:30 EDT (History)
2 users (show)

See Also:

Attachments
store the boolean in the context (30.90 KB, patch)
2011-07-26 16:46 EDT, Paul Webster CLA 		no flags Details | Diff
store the boolean in the context v02 (3.37 KB, patch)
2011-07-26 16:52 EDT, Paul Webster CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Webster CLA 2011-07-25 08:32:48 EDT 
It seems like we don't honour the allowPluginActivation/forcePluginActivation any more.

<visibleWhen
      checkEnabled="false">
   <iterate
         operator="or">
      <or>
         <test
               forcePluginActivation="true"
               property="z.ex.rcp.mail.propertyTester1.test">
         </test>
         <test
               forcePluginActivation="true"
               property="z.ex.rcp.mail.propertyTester2.test">
         </test>
      </or></iterate>
</visibleWhen>


PW
Comment 1 Paul Webster CLA 2011-07-26 16:46:55 EDT 
Created attachment 200395 [details]
store the boolean in the context

Remy, what about this?

PW
Comment 2 Paul Webster CLA 2011-07-26 16:47:38 EDT 
Grumble, my patch includes way to many changes.

PW
Comment 3 Paul Webster CLA 2011-07-26 16:52:24 EDT 
Created attachment 200396 [details]
store the boolean in the context v02

The patch is worse, but at least it is accurate.

PW
Comment 4 Remy Suen CLA 2011-07-26 17:22:51 EDT 
(In reply to comment #3)
> Created attachment 200396 [details]
> store the boolean in the context v02
> 
> The patch is worse, but at least it is accurate.

It still has a lot of other changes. Or are you aware of that?
Comment 5 Paul Webster CLA 2011-07-26 17:37:35 EDT 
(In reply to comment #4)
> 
> It still has a lot of other changes. Or are you aware of that?

No, those are the changes.

PW
Comment 6 Remy Suen CLA 2011-07-27 09:06:55 EDT 
(In reply to comment #5)
> > It still has a lot of other changes. Or are you aware of that?
> 
> No, those are the changes.

Forget that. I was clicking on the wrong attachment.

(In reply to comment #3)
> Created attachment 200396 [details]
> store the boolean in the context v02

Confirmed on my end that this patch allows property testers to force the activation of its parent bundle.