Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 353001

Summary: [Compatibility] 'forcePluginActivation' attribute not honoured for property testers
Product: [Eclipse Project] Platform Reporter: Paul Webster <pwebster>
Component: UIAssignee: Paul Webster <pwebster>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: ob1.eclipse, remy.suen
Version: 4.1   
Target Milestone: 4.2 M1   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
store the boolean in the context
none
store the boolean in the context v02 none

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.