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 74029 - Create some documentation to refactioring module
Summary: Create some documentation to refactioring module
Status: RESOLVED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Refactoring (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: Milos Kleint
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-27 12:55 UTC by Tomas Danek
Modified: 2006-11-03 19:38 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
html document describing refactorings. (3.14 KB, text/html)
2006-04-13 09:20 UTC, Milos Kleint
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tomas Danek 2006-03-27 12:55:09 UTC
Please provide some kind of document (can be brief), which clarifies what
refactoring functionality is provided in apisupport. Some functionality is
provided, some not, but definitly hard to guess what is expected to work and
what not. Thx.
Comment 1 Milos Kleint 2006-04-13 09:20:57 UTC
Created attachment 29832 [details]
html document describing refactorings.
Comment 2 Milos Kleint 2006-04-13 09:24:52 UTC
here is the doc. it describes the situation as of 5.0 (also current trunk)

given that the layer refactorings are not quite there it would be probably
better to remove them. Which also means removing the layer where-user.

opinions?
Comment 3 Martin Krauskopf 2006-04-13 10:59:30 UTC
You mean that you would remove "Safe delete" and "Find Usages" capabilities at
all for layer.xml? Please don't do that I really like both and I think they are
both quite useful (even with that harmless(?) bug leaving a shadow there). And
also it will safe (or ease) a some mailing list traffic "How can I delete my
action?" ;)

I would vote for just putting the attached document somewhere. As a starting
point wiki would be enough IMHO.
Comment 4 Martin Krauskopf 2006-04-13 11:01:30 UTC
> Which also means removing the layer where-user.

Umm, I do not understand the above sentence...
Comment 5 Milos Kleint 2006-04-13 11:12:02 UTC
the sentence should read "Which also means removing the where-used query for
layer files."

Basically having a where used and some refactorings partly written lets the user
assume the rest works as well, which is not true and I have a few bugs regarding
that and I unfortunately don't plan to fix them. The layer usecases are way too
complex for a bulletproof undo for example. 
Additionaly it doens't matter for meta-inf/services and manifest entries to be
limited to one module only, but with layers and their orderings, this design
simplification that I imposed on the refactorings, starts to leak out..

it is my fault that it even got there. The layer refactorings are not ready for
prime-time imho.