Tapestry
  1. Tapestry
  2. TAPESTRY-2408

FormInjector doesn't work properly if the content contains multiple form elements

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: 5.0.12
    • Fix Version/s: 5.0.14
    • Component/s: Core Components
    • Labels:
      None
    • Environment:
      tapestry-core-5.0.12-20080424.110110-44.jar

      Description

      I've been trying to use FormInjector to create a dynamic form and not getting very far. To demonstrate this problem I've taken the FormInjectorDemo integration test and simply taken it out of the border element. It works fine until I add a t:label component in the block. It seems to me that as soon as there is more than one kind of component in the injected block, the inserted HTML is just "<li class="t-beaneditor-row"/>" without any form elements. Two textfields work, and a textfield and an 'any' component work, but not label or radio with a textfield. I suspect it's all form components.

      I'll attach the modified demo. Remove the label component to see it working. (Just remove the "t:" before the label) Add back in to see it broken again.

      1. FormInjectorDemo.java
        1 kB
        Andy Blower
      2. FormInjectorDemo.tml
        0.9 kB
        Andy Blower

        Issue Links

          Activity

          Andy Blower created issue -
          Andy Blower made changes -
          Field Original Value New Value
          Attachment FormInjectorDemo.java [ 12381494 ]
          Andy Blower made changes -
          Attachment FormInjectorDemo.tml [ 12381495 ]
          Andy Blower made changes -
          Description I've been trying to use FormInjector to create a dynamic form and not getting very far. To demonstrate this problem I've taken the FormInjectorDemo integration test and simply taken it out of the border element. It works fine until I add a t:label component in the block. It seems to me that as soon as there is more than one kind of component in the injected block, the inserted HTML is just "<li class="t-beaneditor-row"/>" without any form elements. Two textfields work, and a textfield and an 'any' component work, but not label or radio with a textfield. I suspect it's all form components.

          I'll attach the modified demo. Remove the label component to see it working. Add back in to see it broken again.
          I've been trying to use FormInjector to create a dynamic form and not getting very far. To demonstrate this problem I've taken the FormInjectorDemo integration test and simply taken it out of the border element. It works fine until I add a t:label component in the block. It seems to me that as soon as there is more than one kind of component in the injected block, the inserted HTML is just "<li class="t-beaneditor-row"/>" without any form elements. Two textfields work, and a textfield and an 'any' component work, but not label or radio with a textfield. I suspect it's all form components.

          I'll attach the modified demo. Remove the label component to see it working. (Just remove the "t:" before the label) Add back in to see it broken again.
          Andy Blower made changes -
          Link This issue relates to TAPESTRY-2322 [ TAPESTRY-2322 ]
          Howard M. Lewis Ship made changes -
          Assignee Howard M. Lewis Ship [ hlship ]
          Howard M. Lewis Ship made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Howard M. Lewis Ship made changes -
          Resolution Invalid [ 6 ]
          Status In Progress [ 3 ] Closed [ 6 ]
          Fix Version/s 5.0.14 [ 12313214 ]
          Mark Thomas made changes -
          Workflow jira [ 12430433 ] Default workflow, editable Closed status [ 12569322 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12569322 ] jira [ 12590581 ]

            People

            • Assignee:
              Howard M. Lewis Ship
              Reporter:
              Andy Blower
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development