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 59091 - API Change: ProblemDetails & ReadOnlyFileHandlers
Summary: API Change: ProblemDetails & ReadOnlyFileHandlers
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Refactoring (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: apireviews
URL:
Keywords: API, API_REVIEW_FAST
Depends on:
Blocks:
 
Reported: 2005-05-20 08:28 UTC by Jan Becicka
Modified: 2007-04-03 18:02 UTC (History)
1 user (show)

See Also:
Issue Type: TASK
Exception Reporter:


Attachments
Proposed change (20.14 KB, patch)
2005-05-20 08:29 UTC, Jan Becicka
Details | Diff
Javadoc (205.91 KB, application/x-compressed)
2005-05-20 08:30 UTC, Jan Becicka
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Becicka 2005-05-20 08:28:22 UTC
UI Framewrok needs improvement for integration with CVS and for SafeDelete
refactoring.

ReadOnlyFilesHandler is needed for CVS integration, while rest of classes are
required for both CVS integration and Safe Delete as well.

methods added:
=============
ProblemDetails Problem.getDetails()

Classes added:
=============
ProblemDetails
ProblemDetailsFactory
ProblemDetailsImplementation
ReadOnlyFilesHandler

This api change is backward compatible.
Comment 1 Jan Becicka 2005-05-20 08:29:13 UTC
Created attachment 22205 [details]
Proposed change
Comment 2 Jan Becicka 2005-05-20 08:30:09 UTC
Created attachment 22206 [details]
Javadoc
Comment 3 Jan Becicka 2005-05-20 08:32:16 UTC
Please review the change.
Comment 4 Martin Matula 2005-05-23 15:22:17 UTC
The change looks OK. I have only two minor comments:
- the constructor in ProblemDetailsFactory should be private
- description of "files" parameter in ReadOnlyFilesHandler.createProblem() is
missing
Comment 5 Jan Becicka 2005-05-27 10:51:44 UTC
Checking in refactoring/apichanges.xml;
/cvs/refactoring/apichanges.xml,v  <--  apichanges.xml
new revision: 1.2; previous revision: 1.1
done
Checking in refactoring/arch/arch-refactoring.xml;
/cvs/refactoring/arch/arch-refactoring.xml,v  <--  arch-refactoring.xml
new revision: 1.16; previous revision: 1.15
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/AnonymousToInnerRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/AnonymousToInnerRefactoringPlugin.java,v
 <--  AnonymousToInnerRefactoringPlugin.java
new revision: 1.3; previous revision: 1.2
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/CopyClassRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/CopyClassRefactoringPlugin.java,v
 <--  CopyClassRefactoringPlugin.java
new revision: 1.2; previous revision: 1.1
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/ExtractMethodRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/ExtractMethodRefactoringPlugin.java,v
 <--  ExtractMethodRefactoringPlugin.java
new revision: 1.9; previous revision: 1.8
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/IntroduceVariableRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/IntroduceVariableRefactoringPlugin.java,v
 <--  IntroduceVariableRefactoringPlugin.java
new revision: 1.4; previous revision: 1.3
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PluginsFactory.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PluginsFactory.java,v
 <--  PluginsFactory.java
new revision: 1.10; previous revision: 1.9
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PullUpRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PullUpRefactoringPlugin.java,v
 <--  PullUpRefactoringPlugin.java
new revision: 1.7; previous revision: 1.6
done
Checking in
refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PushDownRefactoringPlugin.java;
/cvs/refactoring/experimental/src/org/netbeans/modules/refactoring/experimental/plugins/PushDownRefactoringPlugin.java,v
 <--  PushDownRefactoringPlugin.java
new revision: 1.4; previous revision: 1.3
done
Checking in refactoring/nbproject/project.properties;
/cvs/refactoring/nbproject/project.properties,v  <--  project.properties
new revision: 1.11; previous revision: 1.10
done
Checking in refactoring/src/org/netbeans/modules/refactoring/APIAccessor.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/APIAccessor.java,v  <-- 
APIAccessor.java
new revision: 1.2; previous revision: 1.1
done
Checking in refactoring/src/org/netbeans/modules/refactoring/ChangeModElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ChangeModElement.java,v 
<--  ChangeModElement.java
new revision: 1.13; previous revision: 1.12
done
Checking in refactoring/src/org/netbeans/modules/refactoring/CommentElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/CommentElement.java,v  <--
 CommentElement.java
new revision: 1.5; previous revision: 1.4
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/CreateMethodElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/CreateMethodElement.java,v
 <--  CreateMethodElement.java
new revision: 1.15; previous revision: 1.14
done
Checking in refactoring/src/org/netbeans/modules/refactoring/JavadocElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/JavadocElement.java,v  <--
 JavadocElement.java
new revision: 1.4; previous revision: 1.3
done
Checking in refactoring/src/org/netbeans/modules/refactoring/SPIAccessor.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/SPIAccessor.java,v  <-- 
SPIAccessor.java
new revision: 1.2; previous revision: 1.1
done
Checking in refactoring/src/org/netbeans/modules/refactoring/WhereUsedElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/WhereUsedElement.java,v 
<--  WhereUsedElement.java
new revision: 1.20; previous revision: 1.19
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/api/AbstractRefactoring.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/AbstractRefactoring.java,v
 <--  AbstractRefactoring.java
new revision: 1.24; previous revision: 1.23
done
Checking in refactoring/src/org/netbeans/modules/refactoring/api/AccessorImpl.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/AccessorImpl.java,v 
<--  AccessorImpl.java
new revision: 1.2; previous revision: 1.1
done
Checking in refactoring/src/org/netbeans/modules/refactoring/api/Problem.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/Problem.java,v  <-- 
Problem.java
new revision: 1.5; previous revision: 1.4
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/ProblemDetails.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/api/ProblemDetails.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/ProblemDetails.java,v
 <--  ProblemDetails.java
initial revision: 1.1
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/api/RefactoringElement.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/api/RefactoringElement.java,v
 <--  RefactoringElement.java
new revision: 1.10; previous revision: 1.9
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/plugins/ChangeParametersPlugin.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/plugins/ChangeParametersPlugin.java,v
 <--  ChangeParametersPlugin.java
new revision: 1.16; previous revision: 1.15
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/plugins/MoveClassRefactoringPlugin.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/plugins/MoveClassRefactoringPlugin.java,v
 <--  MoveClassRefactoringPlugin.java
new revision: 1.24; previous revision: 1.23
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/plugins/RenameRefactoringPlugin.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/plugins/RenameRefactoringPlugin.java,v
 <--  RenameRefactoringPlugin.java
new revision: 1.27; previous revision: 1.26
done
Removing
refactoring/src/org/netbeans/modules/refactoring/plugins/SimpleRefactoringElementImpl.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/plugins/SimpleRefactoringElementImpl.java,v
 <--  SimpleRefactoringElementImpl.java
new revision: delete; previous revision: 1.3
done
Checking in refactoring/src/org/netbeans/modules/refactoring/spi/AccessorImpl.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/AccessorImpl.java,v 
<--  AccessorImpl.java
new revision: 1.2; previous revision: 1.1
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/Bundle.properties,v
done
Checking in refactoring/src/org/netbeans/modules/refactoring/spi/Bundle.properties;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/Bundle.properties,v 
<--  Bundle.properties
initial revision: 1.1
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsFactory.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsFactory.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsFactory.java,v
 <--  ProblemDetailsFactory.java
initial revision: 1.1
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsImplementation.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsImplementation.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ProblemDetailsImplementation.java,v
 <--  ProblemDetailsImplementation.java
initial revision: 1.1
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ReadOnlyFilesHandler.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/ReadOnlyFilesHandler.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ReadOnlyFilesHandler.java,v
 <--  ReadOnlyFilesHandler.java
initial revision: 1.1
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/RefactoringElementImplementation.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/RefactoringElementImplementation.java,v
 <--  RefactoringElementImplementation.java
new revision: 1.2; previous revision: 1.1
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/RefactoringElementsBag.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/RefactoringElementsBag.java,v
 <--  RefactoringElementsBag.java
new revision: 1.3; previous revision: 1.2
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/SimpleRefactoringElementImpl.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/SimpleRefactoringElementImpl.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/SimpleRefactoringElementImpl.java,v
 <--  SimpleRefactoringElementImpl.java
initial revision: 1.1
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/ui/ParametersPanel.form;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ui/ParametersPanel.form,v
 <--  ParametersPanel.form
new revision: 1.3; previous revision: 1.2
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/spi/ui/ParametersPanel.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/spi/ui/ParametersPanel.java,v
 <--  ParametersPanel.java
new revision: 1.9; previous revision: 1.8
done
Checking in refactoring/src/org/netbeans/modules/refactoring/ui/Bundle.properties;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/Bundle.properties,v 
<--  Bundle.properties
new revision: 1.53; previous revision: 1.52
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/ui/CheckNodeListener.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/CheckNodeListener.java,v
 <--  CheckNodeListener.java
new revision: 1.13; previous revision: 1.12
done
Checking in refactoring/src/org/netbeans/modules/refactoring/ui/ErrorPanel.form;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ErrorPanel.form,v  <--
 ErrorPanel.form
new revision: 1.7; previous revision: 1.6
done
Checking in refactoring/src/org/netbeans/modules/refactoring/ui/ErrorPanel.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ErrorPanel.java,v  <--
 ErrorPanel.java
new revision: 1.8; previous revision: 1.7
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.form,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.form;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.form,v
 <--  ProblemComponent.form
initial revision: 1.1
done
RCS file:
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.java,v
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/ProblemComponent.java,v
 <--  ProblemComponent.java
initial revision: 1.1
done
Checking in
refactoring/src/org/netbeans/modules/refactoring/ui/RefactoringPanel.java;
/cvs/refactoring/src/org/netbeans/modules/refactoring/ui/RefactoringPanel.java,v
 <--  RefactoringPanel.java
new revision: 1.53; previous revision: 1.52
done
Checking in j2ee/refactoring/nbproject/project.xml;
/cvs/j2ee/refactoring/nbproject/project.xml,v  <--  project.xml
new revision: 1.11; previous revision: 1.10
done
Checking in
j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/changeparam/AbstractChangeParamsRefactoringElement.java;
/cvs/j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/changeparam/AbstractChangeParamsRefactoringElement.java,v
 <--  AbstractChangeParamsRefactoringElement.java
new revision: 1.4; previous revision: 1.3
done
Checking in
j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/moveclass/AbstractMoveClassRefactoringElement.java;
/cvs/j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/moveclass/AbstractMoveClassRefactoringElement.java,v
 <--  AbstractMoveClassRefactoringElement.java
new revision: 1.4; previous revision: 1.3
done
Checking in
j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/rename/AbstractRenameRefactoringElement.java;
/cvs/j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/rename/AbstractRenameRefactoringElement.java,v
 <--  AbstractRenameRefactoringElement.java
new revision: 1.4; previous revision: 1.3
done
Checking in
j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/whereused/AbstractWhereUsedRefactoringElement.java;
/cvs/j2ee/refactoring/src/org/netbeans/modules/j2ee/refactoring/whereused/AbstractWhereUsedRefactoringElement.java,v
 <--  AbstractWhereUsedRefactoringElement.java
new revision: 1.4; previous revision: 1.3
done