Ticket #7876 (closed Bug: fixed)

Opened 4 years ago

Last modified 18 months ago

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:

Description

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

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

Change History

comment:1 Changed 4 years ago by Saare

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

Changed 3 years ago by j.swiderski

comment:2 Changed 3 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 19 months ago by j.swiderski

#10480 was marked as duplicate.

comment:4 Changed 19 months ago by pjasiun

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

comment:5 Changed 19 months ago by pjasiun

  • Milestone set to CKEditor 4.2

comment:6 Changed 19 months ago by pjasiun

in branch/ticket #10281

comment:7 Changed 19 months ago by pjasiun

  • Status changed from review to assigned

comment:8 Changed 18 months ago by a.nowodzinski

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