jsf-extensions
  1. jsf-extensions
  2. JSF_EXTENSIONS-81

DynaFaces.ViewState initialize function - collect Form Data Check

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: runtime
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      81

      Description

      In com_sun_facesw_ajax.js:

      DynaFaces.ViewState

      initialize: function ....
      ...
      if (('void' != collectPostDataType && 'undefined' != collectPostDataType) ||
      ('void' != inputsType && 'undefined' != collectPostDataType)) {
      // Just get the state data.
      ...

      I believe should be:

      if (('void' != collectPostDataType && 'undefined' != collectPostDataType) ||
      ('void' != inputsType && 'undefined' != inputsType)) {
      // Just get the state data.
      ...

        Activity

        Hide
        Ed Burns added a comment -

        Thanks for catching this. r=edburns. Please fix it on the HEAD.

        Show
        Ed Burns added a comment - Thanks for catching this. r=edburns. Please fix it on the HEAD.
        Hide
        rogerk added a comment -

        Checked in mods:

        Index: com_sun_faces_ajax-max.js
        ===================================================================
        — com_sun_faces_ajax-max.js (revision 468)
        +++ com_sun_faces_ajax-max.js (working copy)
        @@ -505,7 +505,7 @@
        var collectPostDataType = typeof this.options.collectPostData;
        var inputsType = typeof this.options.inputs;
        if (('void' != collectPostDataType && 'undefined' != collectPostDataType) ||

        • ('void' != inputsType && 'undefined' != collectPostDataType)) {
          + ('void' != inputsType && 'undefined' != inputsType)) {
          // Just get the state data.
          var viewState = DynaFaces.$(DynaFaces.gViewState);
          t = viewState.tagName.toLowerCase();
          @@ -661,7 +661,7 @@
          var collectPostDataType = typeof this.options.collectPostData;
          var inputsType = typeof this.options.inputs;
          if (('void' != collectPostDataType && 'undefined' != collectPostDataType) ||
        • ('void' != inputsType && 'undefined' != collectPostDataType)) {
          + ('void' != inputsType && 'undefined' != inputsType)) {
          // Just get the state data.
          var viewState = DynaFaces.$(DynaFaces.gViewState);
          t = viewState.tagName.toLowerCase();
        Show
        rogerk added a comment - Checked in mods: Index: com_sun_faces_ajax-max.js =================================================================== — com_sun_faces_ajax-max.js (revision 468) +++ com_sun_faces_ajax-max.js (working copy) @@ -505,7 +505,7 @@ var collectPostDataType = typeof this.options.collectPostData; var inputsType = typeof this.options.inputs; if (('void' != collectPostDataType && 'undefined' != collectPostDataType) || ('void' != inputsType && 'undefined' != collectPostDataType)) { + ('void' != inputsType && 'undefined' != inputsType)) { // Just get the state data. var viewState = DynaFaces.$(DynaFaces.gViewState); t = viewState.tagName.toLowerCase(); @@ -661,7 +661,7 @@ var collectPostDataType = typeof this.options.collectPostData; var inputsType = typeof this.options.inputs; if (('void' != collectPostDataType && 'undefined' != collectPostDataType) || ('void' != inputsType && 'undefined' != collectPostDataType)) { + ('void' != inputsType && 'undefined' != inputsType)) { // Just get the state data. var viewState = DynaFaces.$(DynaFaces.gViewState); t = viewState.tagName.toLowerCase();

          People

          • Assignee:
            rogerk
            Reporter:
            rogerk
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: