DynaFaces.ViewState initialize function - collect Form Data Check

[glassfishrobot]

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. ...

Environment

Operating System: All Platform: All

Affected Versions

[current]

[glassfishrobot]

• Issue Imported From: https://github.com/javaee/jsf-extensions/issues/81
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To: @glassfishrobot
• Original Issue Closed By:@glassfishrobot

[glassfishrobot]

@glassfishrobot Commented Reported by rogerk

[glassfishrobot]

@glassfishrobot Commented @edburns said: Thanks for catching this. r=edburns. Please fix it on the HEAD.

[glassfishrobot]

@glassfishrobot Commented rogerk said: 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();

[glassfishrobot]

@glassfishrobot Commented Was assigned to rogerk

[glassfishrobot]

@glassfishrobot Commented This issue was imported from java.net JIRA JSF_EXTENSIONS-81

[glassfishrobot]

@glassfishrobot Commented Marked as fixed on Thursday, September 27th 2007, 4:30:35 am