Side navigation
#8200 closed bug (fixed)
Opened February 07, 2011 05:34PM UTC
Closed February 12, 2011 02:56PM UTC
Last modified February 14, 2011 01:31AM UTC
Remove undocumented $.support._scriptEval in favor of private var _scriptEval
| Reported by: | musicisair | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 1.5.1 |
| Component: | support | Version: | 1.5 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
$.support._scriptEval probably isn't need and a simple var _scriptEval should do just fine.
Unless people are improperly using $.support._scriptEval instead of $.support.scriptEval I don't see how this could break anything.
Attachments (0)
Change History (6)
Changed February 07, 2011 08:36PM UTC by comment:1
Changed February 07, 2011 10:31PM UTC by comment:2
| component: | unfiled → support |
|---|---|
| priority: | undecided → low |
Changed February 07, 2011 10:54PM UTC by comment:3
| blockedby: | → 7371 |
|---|---|
| resolution: | → wontfix |
| status: | new → closed |
Applying this patch will contradict this commit: http://bugs.jquery.com/ticket/7371 (which was a blocker)
https://github.com/jquery/jquery/blob/220a0ce1628d376ec14394c9b0be3c10f92a4cdb/src/support.js
Changed February 07, 2011 11:43PM UTC by comment:4
| resolution: | wontfix |
|---|---|
| status: | closed → reopened |
Changed February 12, 2011 02:56PM UTC by comment:5
| resolution: | → fixed |
|---|---|
| status: | reopened → closed |
Unexpose $.support._scriptEval as it's not needed. Use a private var instead. Fixes #8200.
Changeset: f0999076723d85e77d4ba73d7c464466b5a7dba1
Changed February 14, 2011 01:31AM UTC by comment:6
| milestone: | 1.next → 1.5.1 |
|---|
https://github.com/jquery/jquery/pull/225