Refactor #8721: Replace all uses of jQuery(item).attr('id') with item.id - Foreman

Actions

Refactor #8721

closed

Replace all uses of jQuery(item).attr('id') with item.id

Added by Tomer Brisker over 9 years ago. Updated almost 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Performance

Target version:

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/2020

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

No need to use jQuery when we don't have to.
Using plain item.id is about 100X faster:
http://jsperf.com/jquery-item-attr-id-vs-item-id

Actions

Also available in: Atom PDF