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: | TableCellEditor getCellEditor(Object node, String columnID) called on wrong node when the column is sorted | ||
---|---|---|---|
Product: | platform | Reporter: | epele83 <epele83> |
Component: | Outline&TreeTable | Assignee: | Martin Entlicher <mentlicher> |
Status: | VERIFIED FIXED | ||
Severity: | normal | ||
Priority: | P1 | ||
Version: | 7.4 | ||
Hardware: | PC | ||
OS: | Windows 7 | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
epele83
2015-10-27 14:32:19 UTC
Digging more on the issue I've found that the same issue exists also for getCellRenderer(..) method. I found also a possible patch to apply in: org.netbeans.modules.viewmodel.DelegatingCellRenderer org.netbeans.modules.viewmodel.DelegatingCellEditor Instead to use: Node n = DelegatingCellRenderer,getNodeAt(om, row) we can fix it using Node n = Visualizer.findNode(outline.getValueAt(row, 0)) I am not very sure about the patch but it works in all the tests I did. Thanks -Paolo Reproduced. Fixed by changeset: 293362:3840ea6ba356 http://hg.netbeans.org/core-main/rev/3840ea6ba356 I'm marking this as a candidate for 8.1 patch. It's too late to merge into 8.1 final release. We'll not distribute patches for 7.4, but it can be fixed in a similar fashion there. You can create a patch yourself by applying the change to 7.4 sources and build the affected module. Then follow http://wiki.netbeans.org/DevFaqModulePatching Ask me, or write a comment here, if you need a help with this. Thank you for the patch, we have already put it in our platform. Thank you again -Paolo Integrated into 'main-silver', will be available in build *201510300002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-silver/rev/3840ea6ba356 User: mentlicher@netbeans.org Log: #256206: Convert rows to the model index before retrieving the node. The fix is transplanted into release81 branch: http://hg.netbeans.org/releases/rev/a24feab6f488 http://hg.netbeans.org/releases/rev/a700277cd50c epele83 can you please verify thins in the 8.1 patch1? The staged update centers are distro: http://updates.netbeans.org/netbeans/updates/8.1/uc/final/distribution/catalog_stage.xml.gz plugins: http://updates.netbeans.org/netbeans/updates/8.1/uc/final/certified/catalog_stage.xml.gz If you have question about applying the patch let me know Thanks I applied the patch time ago on NB7.4 It worked, I will test also the patch on NB8.1. |