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 61546 - Project Delete API changes/adjustments, extending the API for project copy/rename/move
Summary: Project Delete API changes/adjustments, extending the API for project copy/re...
Status: RESOLVED FIXED
Alias: None
Product: projects
Classification: Unclassified
Component: Generic Infrastructure (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Jan Lahoda
URL:
Keywords: API, API_REVIEW_FAST
Depends on:
Blocks:
 
Reported: 2005-07-29 15:59 UTC by Jan Lahoda
Modified: 2005-09-05 10:10 UTC (History)
0 users

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments
API diff. (50.36 KB, patch)
2005-07-29 16:00 UTC, Jan Lahoda
Details | Diff
Full diff. (187.71 KB, patch)
2005-07-29 16:00 UTC, Jan Lahoda
Details | Diff
Tests for DefaultProjectsOperationImplementation. (3.94 KB, application/octet-stream)
2005-07-29 16:02 UTC, Jan Lahoda
Details
Log from commit. (14.48 KB, text/plain)
2005-08-22 21:59 UTC, Jan Lahoda
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Lahoda 2005-07-29 15:59:49 UTC
As stated in issue #61137, the new API for project delete is should be improved.
Moreover I also propose to extend this API with support for project
copy/rename/move.

Summary of proposed changes:
-move ProjectOperations from org.netbeans.api.project to
 org.netbeans.spi.project.support. Reasons are that:
 1. only a limited number of clients should use the class (for NB
    probably only two: the default project operations implementation and
    the J2EE project)
 2. this class is not strictly necessary: the client can go through
    the lookup itself
-do top level interfaces from the DeleteOperationImplementation (and
 added two new CopyOperationImplementation, MoveOperationImplementation).
-move the default delete(/copy/rename/move) operation into the
 Project UI API.
-change performClean to notifyDeleting
-add COMMAND_COPY, COMMAND_RENAME and COMMAND_MOVE and corresponding methods in
ActionsFactory.
-add ReferenceHelper.fixReferences to help resolve broken references after
copy/rename/move.

I am attaching:
-copy-move-api.diff showing only API changes
-copy-move-full.diff showing also implementation (the impl. is not completelly
finished)
-test.jar with tests for the default project delete/copy/rename/move implementation
Comment 1 Jan Lahoda 2005-07-29 16:00:28 UTC
Created attachment 23363 [details]
API diff.
Comment 2 Jan Lahoda 2005-07-29 16:00:59 UTC
Created attachment 23364 [details]
Full diff.
Comment 3 Jan Lahoda 2005-07-29 16:02:03 UTC
Created attachment 23365 [details]
Tests for DefaultProjectsOperationImplementation.
Comment 4 Jan Lahoda 2005-08-22 21:58:07 UTC
I have implemented the proposed changes, see attached log from commit.
Comment 5 Jan Lahoda 2005-08-22 21:59:22 UTC
Created attachment 24134 [details]
Log from commit.