#2652 closed enhancement (fixed)
val() returns an empty string in IE6/7
Reported by: | kowcik | Owned by: | flesler |
---|---|---|---|
Priority: | minor | Milestone: | 1.3 |
Component: | core | Version: | 1.2.6 |
Keywords: | val, select selected | Cc: | |
Blocked by: | Blocking: |
Description
See the attached example.
Attachments (2)
Change History (8)
Changed 15 years ago by
comment:1 Changed 15 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
So.. the options don't have a 'value' attribute, it's expected that you get and empty string as return.
I'll close as invalid, but please reopen if you have anything else to say, do include more details when you do that (better test case and tell us what you expect the return to be, etc).
comment:2 Changed 15 years ago by
Resolution: | invalid |
---|---|
Status: | closed → reopened |
In IE val() returns empty strings, whereas in Firefox the value is returned also when there's no value attributed. This should the desired behaviour, because val seems to be an abstraction to access inputs values as they would be submitted to the server. Also, there is a specific line in jQuery code that tries to retrieve values from options, but for some reason this value is not returned later. Generally option.text is the right property to read from in IE.
//val function source code if ( option.selected ) { // Get the specifc value for the option value = jQuery.browser.msie && !option.attributes.value.specified ? option.text : option.value;
comment:3 Changed 15 years ago by
Owner: | set to flesler |
---|---|
Status: | reopened → new |
comment:4 Changed 15 years ago by
Status: | new → assigned |
---|
comment:5 Changed 15 years ago by
Milestone: | 1.2.4 → 1.3 |
---|---|
need: | Patch → Commit |
Priority: | major → minor |
Type: | bug → enhancement |
Version: | 1.2.3 → 1.2.6 |
val() is meant to be used on selects, not on options. If you call it on a select, it will return the value of the selected options, or an array if it's "multiple".
Still.. I pulled out a patch, I'll ask for opinions to see if it should be applied or not.
Changed 15 years ago by
Attachment: | val[5790].diff added |
---|
comment:6 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed and added some tests at [5793].
Test case from jQuery documentation