You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Sven Meier (JIRA)" <ji...@apache.org> on 2011/09/16 11:29:08 UTC
[jira] [Resolved] (WICKET-3962) the wrong option in select
components is rendered (Combination of reusing panels and
defaultFormProcessing==false)
[ https://issues.apache.org/jira/browse/WICKET-3962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sven Meier resolved WICKET-3962.
--------------------------------
Resolution: Fixed
Fix Version/s: 1.4.19
aligned Select with CheckGroup and RadioGroup, i.e. use uuid for values
> the wrong option in select components is rendered (Combination of reusing panels and defaultFormProcessing==false)
> ------------------------------------------------------------------------------------------------------------------
>
> Key: WICKET-3962
> URL: https://issues.apache.org/jira/browse/WICKET-3962
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 1.4.18, 1.5-RC5.1
> Environment: mac os / windows / java 1.5 / java 1.6 / tomcat
> Reporter: Olaf Siefart
> Assignee: Sven Meier
> Labels: defaultFormProcessing, select
> Fix For: 1.4.19
>
> Attachments: PageScopedValue.patch, Select.java, select-option-uuid.patch, wicket-3962.zip
>
> Original Estimate: 10m
> Remaining Estimate: 10m
>
> We are reusing a panel (let's say a PersonPanel, two instances). There is a forward / back ajax-navigation between the panels, which is implemented with component.replace. So the component path for the select of both panels are the same. The back-function is implemented with setDefaultFormProcessing(false). If now showing PersonPanel1 --> Forward --> PersonPanel2 --> Back --> PersonPanel1 --> Forward --> PersonPanel2, the selects of the panel 2 are showing the values of panel1.
> The Reason:
> In the 'Select.isSelected(...)' the getInputAsArray() is used (instead of the getRawInput()). So request parameters are overruling the raw input, if the component path is the same.
> Wicket-Code:
> if (hasRawInput()) {
> String[] paths = getInputAsArray();
> I think, instead of calling getInputAsArray() the getRawInput()(With a split for the MultiSelect-support) should be used.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira