Skip to content

Conversation

@findstar
Copy link

append 'if else case' of function ajax_call was to support multiple checkbox value

ex)

<input type="checkbox" name="somecheck[]" ...../>
<input type="checkbox" name="somecheck[]" ...../>
<input type="checkbox" name="somecheck[]" ...../>
<input type="checkbox" name="somecheck[]" ...../>

append something of function ajax_call was to support multiple checkbox
value"
@pocesar
Copy link
Contributor

pocesar commented Oct 27, 2013

that's only when you are setting the data-phery-related of the DOM element, since inputs, selects and textareas are dealt in the serializeForm function, is that what you mean?

@findstar
Copy link
Author

yes. It make sense when 'data-phery-related'

ex)

On PHP source code

'button_simple', 'tag' => 'button', 'related' => "input[type='checkbox'].checkUser:checked")); ?>

On related option, multiple input checkbox can not catched.

Only last checkbox value retured.

So I checked phery.js file and do some modification.

2013/10/28 Paulo Cesar notifications@github.com

that's only when you are setting the data-phery-related of the DOM
element, since inputs, selects and textareas are dealt in the
serializeForm function, is that what you mean?

Reply to this email directly or view it on GitHubhttps://github.com//pull/13#issuecomment-27182685
.

¾ÈÁ¤¼ö
Jason Ahn
Linux, Server, Web, Php, Java developer.
Weather analyst

@pocesar
Copy link
Contributor

pocesar commented Oct 28, 2013

makes sense, I'll implement your code and bump the version, thanks!

@pocesar
Copy link
Contributor

pocesar commented Oct 29, 2013

test the updated library on the 7ea8798 concatFix branch and let me know. if it's working as expected, I'll merge the changes and release the new version. thanks

@findstar
Copy link
Author

Ok. working good.

Thanks your fast fix.

2013/10/29 Paulo Cesar notifications@github.com

test the updated library on the concatFix branch and let me know. if it's
working as expected, I'll merge the changes and release the new version.
thanks

Reply to this email directly or view it on GitHubhttps://github.com//pull/13#issuecomment-27269640
.

¾ÈÁ¤¼ö
Jason Ahn
Linux, Server, Web, Php, Java developer.
Weather analyst

@pocesar pocesar closed this Nov 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants