Problemns with JTree

Question

Hi! I'm having problems with the Tree cell renderer. I must render a tree that have leafs that is also JPanels compounded by a checkbox, an icon and a the text label. It's supposed to alter the state of the checkbox when clicked or when press the spacebar over the cell, but it isn't working...
If I set the tree as editable (down in the code, in main method), the checkbox is selected only after the second click, but the cell is not being highlighted and cannot collapse the category nodes. Otherwise, when set the Tree as NOT editable, the highlight of cell and the rest works, but obviously I cannot edit the checkboxes...
Can someone help me, please?
package test;
import com.interact.sas.ResourceLocator;
import com.interact.sas.dms.*;
import com.interact.sas.ui.UserIconLabel;
import java.awt.*;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.util.EventObject;
import java.util.Vector;
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.tree.*;
public class DocumentControllerTreeTest
    static class MyTreeCellEditor extends AbstractCellEditor implements TreeCellEditor, TreeCellRenderer
        private static Icon iconCategory = ResourceLocator.getIcon( "sas/dms/tb_categories.png" );
        private static Border noFocusBorder = new EmptyBorder( 1, 1, 1, 1 );
        private JPanel cell = new JPanel();
        private JCheckBox checkRequired = new JCheckBox();
        private UserIconLabel userIconLabel = new UserIconLabel();
        private DocumentControllerTree.Item currentItem = null;
        public MyTreeCellEditor()
            userIconLabel.setOpaque( false );
            checkRequired.setOpaque( false );
            cell.setLayout( new BorderLayout() );
            cell.add( checkRequired, BorderLayout.WEST );
            cell.add( userIconLabel, BorderLayout.CENTER );
            cell.addKeyListener( new KeyAdapter()
                public void keyPressed( KeyEvent e )
                    if ( e.getKeyCode() == KeyEvent.VK_SPACE )
                        checkRequired.setSelected( ! checkRequired.isSelected() );
                        e.consume();
            cell.addMouseListener( new java.awt.event.MouseAdapter()
                public void mouseClicked(java.awt.event.MouseEvent evt)
                    checkRequired.setSelected( ! checkRequired.isSelected() );
         * getCellEditorValue
         * @return Object
        public Object getCellEditorValue()
            currentItem.getController().setRequired( checkRequired.isSelected() );
            return new Boolean( currentItem.getController().isRequired() );
         * getTreeCellEditorComponent
         * @param JTree tree,
         * @param Object value,
         * @param boolean isSelected,
         * @param boolean expanded,
         * @param boolean leaf,
         * @param int row
         * @return Component
        public Component getTreeCellEditorComponent( JTree tree,
                Object value,
                boolean isSelected,
                boolean expanded,
                boolean leaf,
                int row)
            return getTreeCellRendererComponent( tree, value, isSelected, expanded, leaf, row, true );
         * getTreeCellRendererComponent
         * @param JTree tree,
         * @param Object value,
         * @param boolean selected,
         * @param boolean expanded,
         * @param boolean leaf,
         * @param int row,
         * @param boolean hasFocus
         * @return Component
        public Component getTreeCellRendererComponent( JTree tree,
                Object value,
                boolean isSelected,
                boolean isExpanded,
                boolean isLeaf,
                int row,
                boolean hasFocus )
            DefaultMutableTreeNode node = (DefaultMutableTreeNode) value;
            Object object = node.getUserObject();
            if ( object instanceof DocumentControllerTree.Item )
                currentItem = (DocumentControllerTree.Item) object;
                checkRequired.setSelected( currentItem.getController().isRequired() );
                //checkRequired.setSelected( !checkRequired.isSelected() );
                int action = currentItem.getController().getAction();
                if ( action == DocumentController.ACTION_APROVE ||
                        action == DocumentController.ACTION_VERIFY )
                    checkRequired.setVisible( true );
                else
                    checkRequired.setVisible( false );
                userIconLabel.setFont( tree.getFont() );
                userIconLabel.setUser( currentItem.getUser() );
                userIconLabel.setText( currentItem.getUser().getName() );
            else
                // nunca poderia chegar aqui ...
                checkRequired.setVisible( false );
                userIconLabel.setFont( tree.getFont() );
                userIconLabel.setIcon( ! isLeaf ? iconCategory : null );
                userIconLabel.setText( object.toString() );
            if ( isSelected )
                cell.setBackground( UIManager.getColor( "Tree.selectionBackground") );
                cell.setForeground( UIManager.getColor( "Tree.selectionForeground") );
            else
                cell.setBackground( tree.getBackground() );
                cell.setForeground( tree.getForeground() );
            Border border = noFocusBorder;
            if (hasFocus)
                if (isSelected)
                    border = UIManager.getBorder("List.focusSelectedCellHighlightBorder");
                if (border == null)
                    border = UIManager.getBorder("List.focusCellHighlightBorder");
            cell.setBorder(border);
            return cell;
    public static void main( String[] args )
        try
            DocumentControllerTree tree = new DocumentControllerTree();
            DocumentManager dm = DocumentManager.getInstance();
            DocumentCategory dc = dm.getCategory( 32 );
            Vector<DocumentController> vdc = dm.getControllers( dc );
            tree.setCellRenderer( new MyTreeCellEditor() );
            tree.setCellEditor( new MyTreeCellEditor() );
            tree.setCategory( dc );
            tree.setControllers( vdc );
            // This must be true
            //tree.setEditable( true );
            JScrollPane scroller = new JScrollPane( tree );
            scroller.setPreferredSize( new Dimension( 240, 320 ) );
            JOptionPane.showMessageDialog( null, scroller );
        catch ( Exception e )
            e.printStackTrace();
        System.exit( 0 );
}

Answer 1

OK, here's briefly what I'd do:
1
In your tree constructor, set a custom TreeCellRenderer. (I assume your tree extends JTree.)
2
Create a custom TreeCellRenderer by extending JPanel and implementing TreeCellRenderer. In the getTreeCellRendererComponent method, remember you don't have to use a component that you have around. You use the tree node's text: tree.convertValueToText(value, sel, expanded, leaf, row, hasFocus)); This returns a string. The string could be "true" or "false". You then create a JCheckBox based on this value. Add the JCheckBox to �this�, i.e. to the custom TreeCellRenderer that extends a JPanel. Return �this�. Remember that once �this� is returned, the value is forgotten and the JCheckBox is unclickable. (What you click actually is the TreeCellRenderer, that then decides you want to focus a node and thereafter passes focus and subsequent clicks to your panel and/or the TreeCellEditor).
3
Back to the tree constructor. Set your TreeCellEditor to a custom class of yours. Copy the code from DefaultCellEditor. There is a constructor in DefaultCellEditor that accepts a JCheckBox as sole argument. Try that one. In an inner class in DefaultCellEditor, EditorDelegate, you'll find the stopCellEditing method. Override it and make sure to convert boolean values back to "true" or "false" in the tree node, so that the renderer will have the right value to work with.
From there on, I guess things should work by themselves. (Warning: completely unchecked!!!) If not, at least the code is sufficiently separated for us to know where it fails (renderer, editor, tree...)
PS. Second thought: DefaultMutableTreeNode accepts an Object as object, not only a string. Maybe you could use a boolean value or a Boolean object directly in the tree node instead of strings...
Just my 2 cents