Bug Tracker

Modify ↓

Ticket #1309 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

className.has(undefined) and className.add(undefined) should fail gracefully

Reported by: keithorama Owned by: brandon
Priority: minor Milestone: 1.1.4
Component: core Version: 1.1.3
Keywords: Cc:
Blocking: Blocked by:

Description

it looks like className.remove() handles the case where the parameter is undefined. The has() and add() methods blow up. Would be nice if they didn't

Attachments

jquery.diff Download (1.2 KB) - added by keithorama 6 years ago.
patch to check for non-string input on className.*

Change History

Changed 6 years ago by keithorama

patch to check for non-string input on className.*

comment:1 Changed 6 years ago by brandon

  • Owner set to brandon

comment:2 Changed 6 years ago by brandon

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in Rev [2436].

comment:3 Changed 6 years ago by brandon

  • Version changed from 1.1.2 to 1.1.3
  • Milestone changed from 1.1.3 to 1.1.4

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.