Uploaded image for project: 'Shale'
  1. Shale
  2. SHALE-441

validator script should not be generated for child components inside not rendered or not editable or not enabled parent component.

    XMLWordPrintableJSON

Details

    • Bug
    • Status: In Progress
    • Major
    • Resolution: Unresolved
    • 1.0.4
    • None
    • Validator
    • None
    • jsf 1.1, commons-validator 1.3.1

    Description

      A mail at user mailing list exists about this issue: http://www.nabble.com/-validator--possible-bug-in-validator-script-generation-for-child-comp.-inside-not-rendered-parent-comp.-tf3681172.html

      ValidatorScript class is generating scripts only for (editable AND rendered AND [not disabled or readonly]) fields... this is ok.. issue is; if i put a validator for a component comforming above conditions but inside a parent component not comforming above condition, it's still generating scripts for that field... but when parent not rendered, my child component won't be rendered too. In this case, generated scripts throw javascript errors since they can not find the related field to validate...

      Suppose i have a text box with a validator inside... if i use this textbox directly, it should generate validation script for this field... But if i put this textbox inside a NOT RENDERED panel, it shouldn't generate validation scripts for my textbox... like below case;

      <h:panelGroup rendered="false"> // child components won't be rendered because parent's not rendered
      <h:inputText id="txt_name" ....>
      <s:commonsValidator type="required" arg="Name" server="true" client="true" />
      </h:inputText>
      </h:panelGroup>

      Attachments

        1. validator.patch
          8 kB
          Gary VanMatre

        Activity

          People

            gvanmatre@comcast.net Gary VanMatre
            hasant Hasan Turksoy
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: