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
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 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.