Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345883 - Privileged checkbox not working for new aspect wizard
Summary: Privileged checkbox not working for new aspect wizard
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: UI (show other bugs)
Version: 2.1.2   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 minor (vote)
Target Milestone: 2.1.3   Edit
Assignee: AJDT-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-16 04:17 EDT by Andrew Eisenberg CLA
Modified: 2011-06-21 17:57 EDT (History)
1 user (show)

See Also:

Attachments
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 Andrew Eisenberg CLA 2011-05-16 04:17:06 EDT 
Checking the 'privileged' keyword is not having any effect in the new aspect wizard.
Comment 1 Andrew Eisenberg CLA 2011-05-30 17:27:47 EDT 
I'm disappointed.  I was hoping that the problem was a simple regression that was added when we moved over to JDT weaving, but it looks like the problem is that the wizard is very, very broken.  This code hasn't really been touched since 2006 (way before I started working on this) and it's possible that the wizard has been broken since then.
Comment 2 Andrew Eisenberg CLA 2011-05-30 17:29:17 EDT 
I have been able to get the wizard working again by reworking the code a bit.  However, the checkbox for creating an inner aspect is now broken.  This is really not a feature that is used much.  If I can't get this working easily, then I will remove the feature.
Comment 3 Andrew Eisenberg CLA 2011-05-30 19:43:21 EDT 
I have decided that disabling the inner aspect creation is the right thing to do.  I will be waiting to commit this work since the Webmasters at Eclipse.org have admonished us to hold off on unnecessary builds for a little while until they can fix the Eclipse.org hardware that recently had a failure.
Comment 4 Andrew Eisenberg CLA 2011-06-21 17:57:57 EDT 
Now fixed and committed.