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 8988 - property change event from modifiers editor is fired before confirmation
Summary: property change event from modifiers editor is fired before confirmation
Status: CLOSED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 3.x
Hardware: Sun Solaris
: P3 normal (vote)
Assignee: Tomas Hurka
URL:
Keywords:
: 13070 (view as bug list)
Depends on:
Blocks:
 
Reported: 2000-12-22 23:41 UTC by Rochelle Raccah
Modified: 2007-09-26 09:14 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
proposed patch (9.15 KB, patch)
2001-07-20 20:33 UTC, Tomas Hurka
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rochelle Raccah 2000-12-22 23:41:23 UTC
I listen on modifier property change events and have a confirmation dialog for
certain cases.  When the modifier of a class is changed using the property
editor (... from the property sheet) and abstract is checked on, my confirmation
dialog comes up immediately - before the ok button on the property editor is
chosen.  Svata points out that this is fine for the customizer, but I am seeing
this behavior from the property editor.
Comment 1 Tomas Hurka 2001-01-24 10:06:59 UTC
Property editors for modifiers, method parameters and implemented interfaces
were changed, so that they did not fire their property change events and wait
for OK button. Proposed patch will be attached in the next comment.
Comment 2 Tomas Hurka 2001-01-24 10:07:59 UTC
Created attachment 495 [details]
proposed patch
Comment 3 Jan Becicka 2001-02-16 14:32:47 UTC
Fixed
Comment 4 Jan Becicka 2001-02-21 15:08:26 UTC
[010220_1] Verified
Comment 5 Jan Becicka 2001-06-18 16:50:19 UTC
[dev-212] Regression
Comment 6 Rochelle Raccah 2001-06-18 20:37:19 UTC
Is the regression in 3.2.1/Pilsen builds or 3.3/Trunk or both?
Comment 7 Jan Becicka 2001-06-19 08:36:51 UTC
3.3 only
Comment 8 David Strupl 2001-06-20 14:33:46 UTC
Sorry. I don't understand why it was reopened. Making invalid for now. I have
fixed the property panel (PropertyPanel 1.27) to fire the changes. If you
encounter the problem with current version of PropertyPanel please reopen and
add more info (steps to reproduce etc.). Thanks for your help.
Comment 9 Jan Becicka 2001-06-21 12:46:20 UTC
[dev-200106210100]
Well, steps to reproduce:
1. Select field in Explorer and in Properties Window click [...] to change 
field's modifiers. -> Property Editor is invoked.
2. Check e.g. static check box -> "static" modifier appears before the field, 
although OK button was not pushed.
3. Push Cancel button -> "static" modifier remains in code.
Comment 10 David Strupl 2001-06-21 13:38:49 UTC
Tomas, you didn't apply your proposed patch. See:

http://www.netbeans.org/unbranded-source/browse/openide/src/org/openide/explorer/propertysheet/editors/ModifierEditor.java.diff?r1=1.7&r2=1.8

reassigning back.
Comment 11 Tomas Hurka 2001-06-21 14:47:31 UTC
Assigning back to David. :-))
Comment 12 Jan Becicka 2001-06-22 12:53:44 UTC
*** Issue 13070 has been marked as a duplicate of this issue. ***
Comment 13 Svata Dedic 2001-09-12 19:12:21 UTC
Seems to work. Many cruel things happened to the property sheet impl 
meanwhile.
Comment 14 Jan Becicka 2001-09-19 14:26:26 UTC
[200109190100]
Reopening, see my steps to reproduce (comment from 2001-06-21 04:46)
Comment 15 David Strupl 2001-09-21 14:01:11 UTC
Now you can make the property editor without firing (waiting for the
Ok button to be pressed). It can be achieved by supplying custom
property named "changeImmediate" (with boolean value). For how to pass
the custom parameter please check explorer documentation (or ask me
directly).
Comment 16 Tomas Hurka 2001-10-01 14:32:41 UTC
fixed in trunk
Comment 17 Jan Becicka 2001-10-03 15:15:41 UTC
[200110030100] Verified
Comment 18 Quality Engineering 2003-07-01 13:19:58 UTC
Resolved for 3.4.x or earlier, no new info since then -> closing.