JDK-4783423 : DefaultTreeSelectionModel.insureRowContinuity has ambiguous assertion
  • Type: Bug
  • Component: client-libs
  • Sub-Component: javax.swing
  • Affected Version: 1.4.2,5.0
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic,solaris_8
  • CPU: generic,sparc
  • Submitted: 2002-11-22
  • Updated: 2017-05-16
  • Resolved: 2004-09-27
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availabitlity Release.

To download the current JDK release, click here.
Other
5.0 b25Fixed
Description
Name: dsR10051			Date: 11/22/2002


Javadoc for method of class javax.swing.tree.DefaultTreeSelectionModel
protected void insureRowContinuity()
says:
"
 * @(#)DefaultTreeSelectionModel.java   1.45 02/03/20
    /**
     * Makes sure the currently selected TreePaths are valid
     * for the current selection mode.
     * If the selection mode is <code>CONTIGUOUS_TREE_SELECTION</code>
     * and a RowMapper exists, this will make sure all the rows are
     * contiguous. If the selection isn't contiguous, the selection is
     * reset to contain the first set of contiguous paths.
     * <p>
     * If the selection mode is <code>SINGLE_TREE_SELECTION</code> and
     * more than one TreePath is selected, the selection is reset to
     * contain the first path currently selected.
     */
    protected void insureRowContinuity()
"
The assertion "If the selection isn't contiguous, the selection is reset 
to contain the first set of contiguous paths." for CONTIGUOUS_TREE_SELECTION 
selection mode is ambiguous because "first set of contiguous paths" 
has several meanings. (See bug #4485322 for more detatils).
Evaluation for #4485322 explains algorithm of the selection's retrieval 
if the selection isn't contiguous:
"
If the selection isn't contiguous, the selection should be
reseted to contain the first set of contiguous paths sorted
in order of rows mapped by RowMapper.
###@###.###                                                                
"
This explanation should be included to javadoc.

======================================================================

Comments
CONVERTED DATA BugTraq+ Release Management Values COMMIT TO FIX: tiger tiger-beta FIXED IN: tiger tiger-beta INTEGRATED IN: tiger-b25 tiger-beta VERIFIED IN: 1.5.0_01
2004-09-28

PUBLIC COMMENTS .
2004-09-28

EVALUATION Not critical for mantis. Reassigning to tiger. ###@###.### 2003-01-07 Reviewed by Scott. Approved by CCC. Fixed for tiger. ###@###.### 2003-10-02
2003-01-07