Opened 6 years ago

Closed 4 years ago

#7876 closed Bug (fixed)

jQuery Adapter val() function handles multiple elements incorrectly

Reported by: mridgway Owned by: pjasiun
Priority: Normal Milestone: CKEditor 4.2
Component: General Version: 3.1
Keywords: Cc:


My understanding is that if you have a jQuery object that contains multiple form elements, when you call val() it will return the first element's value. (I haven't found anything in the documentation that says this specifically, but it seems to be the case)

The jQuery adapter loops through each of the wrapped elements calling val(), replacing the returned result each time. This ends up returning the last element's value.

I'm using an older version of the code, but a quick comparison of the trunk's source indicates that it has the same problem.

Attachments (1)

test.html (2.2 KB) - added by j.swiderski 5 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by Saare

  • Keywords jquery removed
  • Status changed from new to confirmed
  • Version changed from 3.4.1 to 3.1

Changed 5 years ago by j.swiderski

comment:2 Changed 5 years ago by j.swiderski

I'm attaching test case file. I'm not sure if this is what @mridgway meant but if I understood the description correctly then the problem does not exist anymore.

Can anyone confirm, give his point of view?

comment:3 Changed 4 years ago by j.swiderski

#10480 was marked as duplicate.

comment:4 Changed 4 years ago by pjasiun

  • Owner set to pjasiun
  • Status changed from confirmed to review

comment:5 Changed 4 years ago by pjasiun

  • Milestone set to CKEditor 4.2

comment:6 Changed 4 years ago by pjasiun

in branch/ticket #10281

comment:7 Changed 4 years ago by pjasiun

  • Status changed from review to assigned

comment:8 Changed 4 years ago by a.nowodzinski

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy