Default val() getter expects a String returned

[Peter Beverloo]

Code (​https://github.com/jquery/jquery/blob/master/src/attributes.js#L168):

return (elem.value || "").replace(rreturn, "");

This code assumes that elem.value will return a String. This is true for most form-related elements, but <meter> and <progress> (both of which have @value and are form-associated) define their value-property as a double. Doubles do not have a replace method, resulting in a JavaScript error.

Reproduction code:

var meterElement = document.createElement ('meter'),
    jqMeter = $ (meterElement);

meterElement.value = .5;
jqMeter.val ();

Tested in Google Chrome 13 with jQuery 1.6.1.

Two proposed fixes:

1) Force elem.value to be a string:

return ((elem.value || "") + "").replace(rreturn, "");

2) Add valHooks for both <meter> as <progress> doing the cast:

jQuery.each([ "meter", "progress" ], function() {
    jQuery.valHooks[ this ] = {
        get: function( elem ) {
            // The meter and progress elements define .value as a double
            return elem.value + "";
        }
    };
});

The issue was identified by Wilfred Nas, ​http://twitter.com/#!/wnas/status/70527764922499074

[timmywil]

If value should be a number, we should probably leave it as a number. Maybe something like:

ret = elem.value;
return typeof ret === "string" ?
    ret.replace(rreturn, "") :
    ret == null ?
        "" :
        ret;

[rwaldron]

For more information about "meter" and "progress" ​http://peter.sh/examples/?/html/meter-progress.html

[rwaldron]

Cases: ​http://jsfiddle.net/rwaldron/fycvn/

[rwaldron]

After discussing this, timmywil and I agree that returning a number is the right thing to do and that type "number" should be added to the list of returns.

​http://api.jquery.com/val

[rwaldron]

#9362 is a duplicate of this ticket.

[rwldrn]

Landing pull request 382. Adds support for number values (meter,progress); Fixes #9319.

More Details:

• ​https://github.com/jquery/jquery/pull/382
• ​http://bugs.jquery.com/ticket/9319 Changeset: f82b9dddc498efdfbcd0978f8ba3e1da3b48eff3

[rwaldron]

I added "Number" to the docs entry:

entry type='method' name="val" return="String, Number, Array"

Someone with a better handle on the docs might want to add an example?