jonathansp / jquery-serializefields Goto Github PK

View Code? Open in Web Editor NEW

13.0 13.0 4.0 446 KB

jQuery plugin to serialize form elements into a JSON object using fieldsets.

License: GNU General Public License v2.0

HTML 18.06% JavaScript 81.94%

javascript jquery jquery-plugin

jquery-serializefields's People

Contributors

Stargazers

Watchers

Forkers

wprelic xixiyiying chaparo

jquery-serializefields's Issues

Add support to select and multiple select inputs.

Look that test:

QUnit.test("When form has select input it should get value from selected", function(assert) {

    // given
    var form = '<form>'+
    '<select name="carlist">'+
    ' <option value="volvo" selected>Volvo</option>'+
    ' <option value="saab">Saab</option>'+
    ' <option value="opel">Opel</option>'+
    '</select>'+
    '</form>';

    // when
    var result = $(form).serializeFields()

    // then
    assert.equal(result.carlist, "volvo", "Property carlist has value Volvo!")
});

IE 11 support

Not sure if its something to do with my setup, or if there is an actual issue with IE 11 support but..

the serialiserializeFields method gathers all the data into a JSON object, but doesn't group by the fieldset names. All the keys are at the same level.

Anybody had experience with this? Cheers

P.S. a client uses IE not me!

Problem when parse check box.

When parsing checkbox input it get value even if checkbox is not checked. I think the best solution for that case is:

When checkbox is not checked their value is ignored. Like here:
https://github.com/marioizquierdo/jquery.serializeJSON#include-unchecked-checkboxes

See that test case:

QUnit.test( "When form has checkbox it should get value only from 'checked'", function( assert ) {
  // given
  var form = '<form>'+
             '  <input type="checkbox" name="check1" value="true" checked/>'+
             '  <input type="checkbox" name="check2" value="true"/>'+
             '  <input type="checkbox" name="check3" value="true"/>'+
             '</form>';

  // when
  var result = $(form).serializeFields()

  // then
  assert.equal(result.check1, "true","Property check1 has value!")
  assert.ok(!wasDefined(result.check2),"Property check2 was not defined!")
  assert.ok(!wasDefined(result.check3),"Property check3 was not defined!")
});

Create new grunt task for npm publish

It should make sure that all tests are ok before executing npm publish.

Add support to html5 input types

https://developer.mozilla.org/en/docs/Web/HTML/Element/Input

Add Bower instructions

Add bower instructions

bower install jquery-serializeFields

More details can be found here:
http://bower.io/docs/creating-packages/#register

Remove parse function from inside $.fn.serializeFields.

Tag update to 0.0.4 version

Please update tag for last version. That way bower can show the right version.
👍

Problem when get value from radio input

It get the last value from a Radio input.

Look this test case:

QUnit.test("When form has radio button input it should get value from checked", function(assert) {

    // given
    var form = '<form>'+
    +'<input type="radio" name="sex" value="male" checked>Male'
    +'<input type="radio" name="sex" value="female">Female'
    +'</form>';

    // when
    var result = $(form).serializeFields()

    // then
    assert.equal(result.sex, "male", "Property sex has value male!")
});

Is there any reason to ignore Textarea on default config ?

jquery-serializeFields/src/jquery-serializeFields.js

Line 46 in ee59ee7

field_selector: "input:enabled,select:enabled",

Create a way to ignore selected fields.

Create a way to ignore fields by tag like:

    <fieldset name="contact">
        <input type="text" name="email" value="[email protected]" />
        <fieldset name="phones">
            <input type="text" name="primary" value="+55 51 123456789" />
            <input type="text" name="mobile" value="+55 51 987654987" />
            <!-- ignore this field -->
            <input data-ignored-field="true" hidden type="text" name="my_ignored_field" value="nothing" />
        </fieldset>
    </fieldset>

Thxs