0006685: Possible bug in PluggableListMorphByItemPlus>>getCurrentSelectionIndex

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0006685

[Squeak] ToolBuilder

minor

always

09-19-07 09:49

01-09-08 14:26

Reporter

Damien Cassou

View Status

public

Assigned To

mikevdg

Priority

normal

Resolution

open

Status

assigned

Product Version

Summary

0006685: Possible bug in PluggableListMorphByItemPlus>>getCurrentSelectionIndex

Description

The method PluggableListMorphByItemPlus>>getCurrentSelectionIndex is
implemented as:

getCurrentSelectionIndex
"Answer the index of the current selection."
| item |
getIndexSelector == nil ifTrue: [^ 0].
item := model perform: getIndexSelector.
^ list findFirst: [ :x | x = item]

However, list is a list of strings. Wouldn't it make more sense to
enumerate the list of models which can be accessed through itemList
instead of list?

Additional Information

Attached Files

Relationships

There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
09-19-07 09:49	Damien Cassou	New Issue
09-19-07 09:49	Damien Cassou	Status	new => assigned
09-19-07 09:49	Damien Cassou	Assigned To	=> mikevdg
01-09-08 14:26	Damien Cassou	Project	Squeak Packages => Squeak

Mantis 1.0.8[^]

31 total queries executed.
28 unique queries executed.