This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 408899 - [Help] Extenders of help.base.activitySupport see a warning regarding "localScopeCheckbox" although that element is unused
Summary: [Help] Extenders of help.base.activitySupport see a warning regarding "localS...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.4 M1   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 411467 (view as bug list)
Depends on: 308128
Blocks: 411467
  Show dependency tree
 
Reported: 2013-05-23 16:50 EDT by Martin Oberhuber CLA
Modified: 2013-06-26 04:11 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2013-05-23 16:50:40 EDT
+++ This bug was initially created as a clone of Bug #308128 +++

This change from bug 308128, which just removed the "local scope" checkbox in the Preference page where search scopes can be configured, left several open questions and obsolete code in place:

1. org.eclipse.help.ui/internal/Messages.{java|properties}
   - obsolete "LocalHelpPage_capabilityFiltering_name"

2. org.eclipse.help.internal.base.IHelpActivitySupport
   - getLocalScopeCheckboxLabel() should be marked obsolete, since unused now

3. Extension point org.eclipse.help.base.activitySupport
   - localScopeCheckbox should be marked obsolete

Point #3 is the most severe IMO, since the current *.exsd of the extension point marks the element as "required" and thus any extende who doesn't provide the element (that's actually optional and obsolete) is presented with a warning in its plugin.xml .

To reproduce, simply load the "org.eclipse.sdk" bundle from the "eclipse.platform.git" repository, and enable the PDE Nature on it. A warning is shown for the activitySupport extension.

I'm not sure what's a better approach - mark all those places obsolete / deprecated, or fix the original problem which bug 308128 apparently fixed in an incorrect way.
Comment 2 Dani Megert CLA 2013-06-24 03:41:11 EDT
This caused a test failure. See bug 411467.
Comment 3 Dani Megert CLA 2013-06-24 03:48:52 EDT
(In reply to comment #2)
> This caused a test failure. See bug 411467.

Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=a5faed5b525f9ff7f8bfc1a96087b5ca37645045
Comment 4 Dani Megert CLA 2013-06-24 03:49:08 EDT
*** Bug 411467 has been marked as a duplicate of this bug. ***