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.
Summary: | Property sheet loses the context node in some conditions | ||
---|---|---|---|
Product: | platform | Reporter: | Nikita Krjukov <supernikita> |
Component: | Explorer | Assignee: | Stanislav Aubrecht <saubrecht> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | frisino, mikk, mmirilovic, praveensavur, supernikita, ttran |
Priority: | P1 | ||
Version: | 5.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 80994 | ||
Attachments: | proposed patch |
Description
Nikita Krjukov
2006-07-25 15:14:14 UTC
this is hardly a P1 bug... Hi Stanislav I think that there is a disconnect in communication here. Unless there is a disconnect, I object to the quick dismissal and downgrade without further explanation or consideration. I hope there is a disconnect and that this matter can be resolved easily. I am resetting to P1 because according to our QE we have a strong P1 bug that according to Nikita is directly caused by this bug. Please read bug 80994 and tell us what priority you think that should have? Bear in mind, that bug results in frequent user visible NPE's on simple property sheet access. If this bug here is directly responsible for bug 80994 and there is no workaround to improve bug 80994, then this bug should have the same priority as 80994. Do you agree with that? Ok, here is what I hope is happening. I hope that Nikita is wrong and that his NPEs are actually not directly dependent on this bug. Or that he is not doing the right thing to work around this bug. Perhaps this is why Stanislav is so quick to downgrade and dismiss this bug. I hope, Stanislav, that you do not believe that this bug described here could cause the behavior described in bug 80994? Is that what you think? Or perhaps you are aware of an easy way to avoid the behavior described in bug 80994 that would not require a fix for this bug? We would really appreciate your thoughtful response here, since bug 80994 is quite visible to our users and QE. sorry, i didn't notice this issue depends on another p1 bug (btw, i haven't seen such a polite 'fix it right now' comment in ages :)) Created attachment 32257 [details]
proposed patch
nikita, pls try the attached patch and let me know if it works for you. also, do you want this fixed in release5.5? Stas, thanks for very quick turnaround. I am glad politeness is effective. Nikita or Praveen Savur will test your patch. Yes, we would definitely like the fix to be in 5.5, assuming it is not a high risk for regressions. We will give you more info soon. Thanks again. saubrecht, I verified that the patch fixes the issue and works for us. fixed in trunk Checking in explorer/src/org/openide/explorer/propertysheet/PropertySheet.java; /cvs/openide/explorer/src/org/openide/explorer/propertysheet/PropertySheet.java,v <-- PropertySheet.java new revision: 1.12; previous revision: 1.11 done Checking in explorer/src/org/openide/explorer/propertysheet/SheetCellEditor.java; /cvs/openide/explorer/src/org/openide/explorer/propertysheet/SheetCellEditor.java,v <-- SheetCellEditor.java new revision: 1.5; previous revision: 1.4 done Thanks Stas. fixed in release55 branch Checking in SheetCellEditor.java; /cvs/openide/explorer/src/org/openide/explorer/propertysheet/SheetCellEditor.java,v <-- SheetCellEditor.java new revision: 1.1.14.1.2.2; previous revision: 1.1.14.1.2.1 done Checking in PropertySheet.java; /cvs/openide/explorer/src/org/openide/explorer/propertysheet/PropertySheet.java,v <-- PropertySheet.java new revision: 1.1.14.2.2.2; previous revision: 1.1.14.2.2.1 done |