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 175275

Summary: [68cat] java.beans.PropertyVetoException: Not allowed
Product: utilities Reporter: giorgio42 <giorgio42>
Component: PropertiesAssignee: Victor Vasilyev <vvg>
Status: RESOLVED FIXED    
Severity: blocker CC: aldobrucale, blaha, georges, mmirilovic, mslama, ulfzibis
Priority: P3    
Version: 6.x   
Hardware: All   
OS: All   
URL: http://statistics.netbeans.org/exceptions/detail.do?id=160700
Issue Type: DEFECT Exception Reporter: 160700
Attachments: stacktrace
stacktrace
stacktrace
stacktrace

Description giorgio42 2009-10-23 11:17:54 UTC
This issue was originally marked as duplicate of issue 172564, that is already resolved. This issue is still valid, so this seems to be another issue, but it might be related.

Build: NetBeans IDE Dev (Build nbms-and-javadoc-4132-on-091021)
VM: Java HotSpot(TM) Client VM, 14.0-b16, Java(TM) SE Runtime Environment, 1.6.0_14-b08
OS: Windows XP, 5.1, x86

User Comments:
yardus: Trying to apply change in the diff window

giorgio42: Tried to copy a change from source to dest while diffiing.

GUEST: diffing files with CVS diff editor

argv: Was attempting to revert a single local change in a Subversion diff on a properties file. No other local changes are present in said file.

GUEST: I viewed the subersion changes of a file and tried to add a deleted line from the trunk with the arrow in the middle to the local file. However it did not work, but this exception was thrown.

gorrus: clicked on the arrow in diff

ronnystandtke: Clicking on an arrow in the middle of a diff view.

dcherk: in CVS diff window, moving a change from 'Base' to 'Locally Modified'



Stacktrace: 
java.beans.PropertyVetoException: Not allowed
        at org.openide.text.CloneableEditorSupport$Listener.vetoableChange(CloneableEditorSupport.java:2849)
        at org.netbeans.editor.BaseDocument.notifyModify(BaseDocument.java:1828)
        at org.netbeans.editor.BaseDocument.notifyModifyCheckStart(BaseDocument.java:977)
        at org.netbeans.editor.BaseDocument.insertString(BaseDocument.java:721)
        at org.netbeans.modules.diff.builtin.visualizer.editable.EditableDiffView.rollback(EditableDiffView.java:594)
        at org.netbeans.modules.diff.builtin.visualizer.editable.LineNumbersActionsBar.performAction(LineNumbersActionsBar.java:198)
Comment 1 giorgio42 2009-10-23 11:18:00 UTC
Created attachment 89971 [details]
stacktrace
Comment 2 Exceptions Reporter 2009-10-23 11:18:04 UTC
This issue already has 12 duplicates 
see http://statistics.netbeans.org/exceptions/detail.do?id=160700
Comment 3 mslama 2009-11-20 08:58:01 UTC
Again in IDE log:
INFO [org.openide.text.CloneableEditorSupport]: callNotifyModified notifyModified returns false this:org.netbeans.modules.properties.PropertiesEditorSupport

Please evaluate.
Comment 4 giorgio42 2009-11-24 09:22:00 UTC
Created attachment 91621 [details]
stacktrace
Comment 5 Victor Vasilyev 2009-11-24 15:05:17 UTC
It is regression after a fix of the bug 89029.
http://hg.netbeans.org/main/rev/6f190c73dac1

The true value should be returned in the method notifyModified() of the class org.netbeans.modules.properties.PropertiesEditorSupport after a successful saving of a properties file.

Moreover, the flag "modified" is false after the saving, but it should have the true value and all titles of the tabs for the property file under modification should be updated. Therefore, super.notifyModified() should be called again after saving.
Comment 6 Victor Vasilyev 2009-11-24 15:06:10 UTC
Fixed in the main trunk
http://hg.netbeans.org/main/rev/72542a8aaa77
Comment 7 mslama 2009-11-25 01:52:55 UTC
*** Bug 174414 has been marked as a duplicate of this bug. ***
Comment 8 ulfzibis 2009-12-14 12:23:01 UTC
Created attachment 92558 [details]
stacktrace
Comment 9 Petr Blaha 2010-01-15 02:58:22 UTC
Created attachment 93307 [details]
stacktrace
Comment 10 giorgio42 2010-02-09 01:35:41 UTC
Any idea why this fix has not bee included in the first set of patches for NB 6.8 ? It makes the "diff to" functionality a lot less valuable (in other words, is a royal pain in the ass...).

As the bug has not been fixed in 6.8 patch1 I am reopening it (never claim to have fixed a bug unless it is included in an official release or patch).
Comment 11 Alexei Mokeev 2010-02-09 01:41:17 UTC
Sorry for the inconvenience. Please check patch nomination process:
http://wiki.netbeans.org/NetBeansPatchesProcess

This bug is fixed in the 6.9 development with target milestone set to 6.9
Comment 12 Georg Schmid 2010-02-09 01:52:23 UTC
Yes, but you have not yet delivered this bug to the customer. Now whenenver I submit this exception to the exception reporter it tells me that the bug has already been fixed, but (from the customer perspective) this is simply wrong and really confusing (I am familiar with this by now, but a first-time user running into the same bug 5 times in a row and each time NetBeans tells him the bug is fixed, what is she or he going to think ?).
Comment 13 Alexei Mokeev 2010-02-09 02:10:33 UTC
73 exc reports overall. Let's consider it for the next patch(if any).