Bugs item #1509831, was opened at 2006-06-21 11:32
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1509831&group_id=119783

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: extensions
Group: 1.2
Status: Open
Resolution: None
Priority: 5
Submitted By: tdrencak (tdrencak)
Assigned to: Nobody/Anonymous (nobody)
Summary: Invalid id generation in Recorder component 

Initial Comment:
In class
wicket.extensions.markup.html.form.palette.component.Recorder
constructor Recorder(String id, Palette palette)

In while loop there is some string generation which
makes some id-string representation... But when using
default ChoiceRenderer without idExpression, then index
is taken as item's id, but index is not changing in the
while loop (0 constant is there).


....
                while (selection.hasNext())
                {
                
modelStringBuffer.append(renderer.getIdValue(selection.next(),
0));
                        if (selection.hasNext())
                        {
                                modelStringBuffer.append(",");
                        }
                }
...

should be changed to:

int i=0;
                while (selection.hasNext())
                {
                
modelStringBuffer.append(renderer.getIdValue(selection.next(),
i++));
                        if (selection.hasNext())
                        {
                                modelStringBuffer.append(",");
                        }
                }

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1509831&group_id=119783


_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to