This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 96255 - Hide o.n.m.editor.CustomizableSideBar and CustomizableSideBarClass2LayerFolder
Summary: Hide o.n.m.editor.CustomizableSideBar and CustomizableSideBarClass2LayerFolder
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: apireviews
URL:
Keywords: API, API_REVIEW_FAST
Depends on:
Blocks:
 
Reported: 2007-02-22 03:01 UTC by Vitezslav Stejskal
Modified: 2007-11-05 13:44 UTC (History)
0 users

See Also:
Issue Type: TASK
Exception Reporter:


Attachments
The diff showing the change (3.16 KB, text/plain)
2007-02-22 03:07 UTC, Vitezslav Stejskal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vitezslav Stejskal 2007-02-22 03:01:52 UTC
This is the same story as with PopupActions (issue #95765). The
o.n.m.editor.CustomizableSideBar and CustomizableSideBarClass2LayerFolder should
have never been made publically accessible. They are part of editor's internal
implementation that provides support for registering side bar components in
MimeLookup. The 'SideBar' folder and MimeLookup is the public contract, which is
used by all modules. There is no reason for anybody to call or use
CustomizableSideBar and CustomizableSideBarClass2LayerFolder classes directly,
hence there is no reason for having them in a public package.
Comment 1 Vitezslav Stejskal 2007-02-22 03:07:41 UTC
Created attachment 38799 [details]
The diff showing the change
Comment 2 Vitezslav Stejskal 2007-02-22 03:10:39 UTC
I would like to move these two classes away from the public package. None of the
modules in standard Netbeans distribution uses them and as I believe no one else
uses them either. The attached diff shows the change. If there are no objections
I'll write a note to apichanges.xml and commit the change.

Thanks for the review.
Comment 3 Vitezslav Stejskal 2007-02-26 08:56:23 UTC
Done.

Checking in src/org/netbeans/modules/editor/NbEditorUI.java;
/cvs/editor/src/org/netbeans/modules/editor/NbEditorUI.java,v  <--  NbEditorUI.java
new revision: 1.62; previous revision: 1.61
done
Removing src/org/netbeans/modules/editor/CustomizableSideBar.java;
/cvs/editor/src/org/netbeans/modules/editor/CustomizableSideBar.java,v  <-- 
CustomizableSideBar.java
new revision: delete; previous revision: 1.14
done
Removing src/org/netbeans/modules/editor/CustomizableSideBarClass2LayerFolder.java;
/cvs/editor/src/org/netbeans/modules/editor/CustomizableSideBarClass2LayerFolder.java,v
 <--  CustomizableSideBarClass2LayerFolder.java
new revision: delete; previous revision: 1.2
done
Checking in apichanges.xml;
/cvs/editor/apichanges.xml,v  <--  apichanges.xml
new revision: 1.8; previous revision: 1.7
done
Checking in nbproject/project.properties;
/cvs/editor/nbproject/project.properties,v  <--  project.properties
new revision: 1.31; previous revision: 1.30
done
RCS file:
/cvs/editor/src/org/netbeans/modules/editor/impl/CustomizableSideBarClass2LayerFolder.java,v
done
Checking in
src/org/netbeans/modules/editor/impl/CustomizableSideBarClass2LayerFolder.java;
/cvs/editor/src/org/netbeans/modules/editor/impl/CustomizableSideBarClass2LayerFolder.java,v
 <--  CustomizableSideBarClass2LayerFolder.java
initial revision: 1.1
done
RCS file:
/cvs/editor/src/org/netbeans/modules/editor/impl/CustomizableSideBar.java,v
done
Checking in src/org/netbeans/modules/editor/impl/CustomizableSideBar.java;
/cvs/editor/src/org/netbeans/modules/editor/impl/CustomizableSideBar.java,v  <--
 CustomizableSideBar.java
initial revision: 1.1
done
Checking in
src/META-INF/services/org.netbeans.spi.editor.mimelookup.Class2LayerFolder;
/cvs/editor/src/META-INF/services/org.netbeans.spi.editor.mimelookup.Class2LayerFolder,v
 <--  org.netbeans.spi.editor.mimelookup.Class2LayerFolder
new revision: 1.7; previous revision: 1.6
done