ColumnChooserDialog not working with hidden columns

[ricci]

Hello,

I noticed some strange behavior while using the TableColumnChooserDialog. I have a table where I want to remove some columns. So the table itself has for example 20 columns. I want to show 10 of them at application start. 5 can be added with the right click menu on the table header. The last 5 shouldn't be shown at all. The reason for this is, that we want one table for different workflows and want to be able to configure which columns the user can show/hide for each workflow.

The TableColumnChooserDialog is displayed correctly with correct columns selected/fixed. It also doesn't show the columns that the user isn't allowed to see. But when I select one of the columns that aren't selected yet, I get all the columns added that the user shouldn't be able to add.

This behavior only only comes up in the Dialog while die right-click menu on the table works fine.

Is there a way to work around this issue?

[JIDE Support]

Thanks for bug report. Will fix it in 2.10.0. For now, please use the following code for work around.

Code: Select all

        TableColumnChooserPopupMenuCustomizer menuCustomizer = new TableColumnChooserPopupMenuCustomizer() {
            @Override
            protected TableColumnChooserDialog createTableColumnChooserDialog(Window owner, String title, JTable table) {
                if (owner instanceof Frame) {
                    return new TableColumnChooserDialog((Frame) owner, title, table) {
                        @Override
                        public ButtonPanel createButtonPanel() {
                            ButtonPanel buttonPanel = super.createButtonPanel();
                            if (buttonPanel.getComponent(0) instanceof JButton) {
                                Object value = ((JButton) buttonPanel.getComponent(0)).getAction().getValue(Action.NAME);
                                ((JButton) buttonPanel.getComponent(0)).setAction(new AbstractAction("" + value) {
                                    @Override
                                    public void actionPerformed(ActionEvent e) {
                                        showOrHideColumns(getSelectedModelIndices(getSelectedColumns()));
                                        setDialogResult(RESULT_AFFIRMED);
                                        setVisible(false);

                                    }
                                    private int[] getSelectedModelIndices(int[] selectedViewIndices) {
                                        Set<Integer> modelIndexList = new HashSet<Integer>();
                                        if (getHiddenColumns() != null) {
                                            for (int column : getHiddenColumns()) {
                                                if (TableColumnChooser.isVisibleColumn(_table, column)) {
                                                    modelIndexList.add(column);
                                                }
                                            }
                                        }
                                        for (int viewIndex : selectedViewIndices) {
                                            modelIndexList.add(getActualModelIndex(viewIndex));
                                        }
                                        Integer[] indexArray = modelIndexList.toArray(new Integer[modelIndexList.size()]);
                                        Arrays.sort(indexArray);
                                        int[] modelIndices = new int[modelIndexList.size()];
                                        for (int i = 0; i < modelIndexList.size(); i++) {
                                            modelIndices[i] = indexArray[i];
                                        }
                                        return modelIndices;
                                    }

                                    private int getActualModelIndex(int viewListIndex) {
                                        List<Integer> hiddenColumns = getHiddenColumns();
                                        if (hiddenColumns == null || hiddenColumns.size() == 0) {
                                            return viewListIndex;
                                        }

                                        for (int modelIndex = 0, currentIndex = 0; modelIndex < _table.getModel().getColumnCount(); modelIndex++) {
                                            if (!hiddenColumns.contains(modelIndex)) {
                                                if (currentIndex == viewListIndex) {
                                                    return modelIndex;
                                                }
                                                currentIndex++;
                                            }
                                        }
                                        System.err.println("View index should not be larger than column count in the model.");
                                        return -1;
                                    }
                                });
                            }
                            return buttonPanel;
                        }
                    };
                }
                else if (owner instanceof Dialog) {
                    return new TableColumnChooserDialog((Dialog) owner, title, table) {
                        @Override
                        public ButtonPanel createButtonPanel() {
                            ButtonPanel buttonPanel = super.createButtonPanel();
                            return buttonPanel;
                        }
                    };
                }
                return null;
            }
        };


Thanks,

[ricci]

Thanks for the quick reply.

Your workaround is working as expected.

[JIDE Support]

Just so you know, this is fixed in 2.10.0 just released.

Thanks,