You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Grant Smith (JIRA)" <de...@myfaces.apache.org> on 2006/11/10 00:29:40 UTC
[jira] Reopened: (TOMAHAWK-521) Values from checkboxes not received
on submit, when branches are collapsed.
[ http://issues.apache.org/jira/browse/TOMAHAWK-521?page=all ]
Grant Smith reopened TOMAHAWK-521:
----------------------------------
Forgot to state the fix version...
> Values from checkboxes not received on submit, when branches are collapsed.
> ---------------------------------------------------------------------------
>
> Key: TOMAHAWK-521
> URL: http://issues.apache.org/jira/browse/TOMAHAWK-521
> Project: MyFaces Tomahawk
> Issue Type: Bug
> Components: Tree2
> Affects Versions: 1.1.3
> Environment: NetBean IDE 5.0
> JSF 1.1 RI
> Sun Java System Application Server 9
> Java EE SDK 5 Windows
> Reporter: Bjørn Stenfeldt
> Assigned To: Grant Smith
> Priority: Critical
> Fix For: 1.1.5-SNAPSHOT
>
> Attachments: TOMAHAWK-521.patch
>
>
> I think this issue is best explained with an example, so here goes...
> I have a tree root. It is not expanded. It has a checkbox, which is not checked.
> [+][ ] root
> Then I expand the root, which reveals 2 children. The children too have checkboxes that are not checked.
> [-][ ] root
> |
> - [ ] child1
> |
> - [ ] child2
> I now check the root and the 2 children. If I were to submit now, everything would work as expected. Values from all 3 checkboxes would be received. But I won't...
> [-][x] root
> |
> - [x] child1
> |
> - [x] child2
> ... Instead I will collaps the tree again, leaving only the root visible.
> [+][x] root
> Finally, I submit the tree. In my backing bean I will then only receive the checkbox value of the root. The checkbox value of child1 and child2 is not submitted. Values from the 2 children do exist in the request scope, but for some reason the backing bean is not set with them.
> Here follows the code I'm using...
> JSF:
> <html:form id="createHost">
> <tomahawk:tree2 id="clientTree" value="#{ModuleBean.moduleHierarchy}"
> var="node" showRootNode="false" clientSideToggle="true">
> <core:facet name="root">
> <html:panelGroup>
> <tomahawk:htmlTag value="div">
> <html:outputText value="#{node.description}"/>
> <html:outputText value=" (#{node.childCount})"
> styleClass="childCount"
> rendered="#{!empty node.children}"/>
> </tomahawk:htmlTag>
> </html:panelGroup>
> </core:facet>
> <core:facet name="module">
> <html:panelGroup>
> <tomahawk:htmlTag value="div">
> <tomahawk:selectManyCheckbox id="moduleId" layout="spread"
> converter="#{HostBean.moduleIdTreeConverter}"
> value="#{HostBean.moduleId}">
> <tomahawk:treeCheckbox for="moduleId"
> itemValue="#{node.identifier}"
> itemLabel="#{node.description}"/>
> </tomahawk:selectManyCheckbox>
> <html:outputText value=" (#{node.childCount})"
> styleClass="childCount"
> rendered="#{!empty node.children}"/>
> </tomahawk:htmlTag>
> </html:panelGroup>
> </core:facet>
> </tomahawk:tree2>
> <html:commandButton value="OK" action="#{HostBean.updateHostAction}"
> style="width: 25%;" styleClass="button"/>
> </html:form>
> HostBean:
> private void addModuleId(String moduleId) {
> if (moduleId != null && !moduleId.equals(""))
> if (!this.moduleId.contains(moduleId))
> this.moduleId.add(moduleId);
> }
> public List getModuleId() {
> return moduleId;
> }
> public void setModuleId(List moduleId) {
> for (int i = 0; i < moduleId.size(); i++) {
> String value = (String)moduleId.get(i);
> addModuleId(value);
> }
> }
> public Converter getModuleIdTreeConverter() {
> return new Converter() {
> public Object getAsObject(FacesContext context,
> UIComponent component, String newValue)
> throws ConverterException {
> addModuleId(newValue);
> return newValue;
> }
> public String getAsString(FacesContext context,
> UIComponent component, Object value)
> throws ConverterException {
> return (String)value;
> }
> };
> }
> public void loadHost(ActionEvent event) {
> /* Initializes data, when the user first clicks the edit button on a
> host.
> Loads both host data and modules. Modules are added using
> the addModuleId(String) method. */
> }
> public String updateHostAction() {
> CachingServiceLocator csl = CachingServiceLocator.getInstance();
> HostLocal hostLocal = csl.lookupHostBean();
> Integer[] moduleId = new Integer[this.moduleId.size()];
> for (int i = 0; i < moduleId.length; i++) {
> moduleId[i] = Integer.valueOf((String)this.moduleId.get(i));
> }
> try {
> hostLocal.updateHost(id, languageId, currencyId,
> moduleId, name, mail, address);
> }
> catch (UpdateException ue) {}
> return "success";
> }
> ModuleBean:
> private void addChildModules(TreeNode node, TOList modules) {
> for (int i = 0; i < modules.getLimitedSize(); i++) {
> ModuleTO module = (ModuleTO)modules.get(i);
> TreeNode child = new TreeNodeBase("module",
> module.getName(), String.valueOf(module.getId()), false);
> node.getChildren().add(child);
> addChildModules(child, module.getChildModules());
> }
> }
> public TreeNode getModuleHierarchy() {
> CachingServiceLocator csl = CachingServiceLocator.getInstance();
> ModuleLocal ml = csl.lookupModuleBean();
> TreeNode base = new TreeNodeBase("root", "ROOT", false);
> addChildModules(base, ml.loadModuleHierarchy());
> return base;
> }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira