]> source.dussan.org Git - jquery.git/log
jquery.git
3 years agoTests: Fix tests for not auto-executing scripts without dataType
Michał Gołębiowski-Owczarek [Mon, 11 Jan 2021 17:20:36 +0000 (18:20 +0100)]
Tests: Fix tests for not auto-executing scripts without dataType

Two issues are fixed in testing for responses with a script Content-Type not
getting auto-executed unless an explicit `dataType: "script"` is provided:
* the test is now using a correct "text/javascript" Content-Type; it was using
  "text/html" until now which doesn't really check if the fix works
* the Node.js based version of the tests didn't account for an empty `header`
  query string parameter

Closes gh-4824
Ref gh-2432
Ref gh-2588
Ref 39cdb8c9aa0fde68f733553ba050a2ba9d86474c

3 years agoDimensions: Add offset prop fallback to FF for unreliable TR dimensions
Timmy Willison [Mon, 11 Jan 2021 16:56:08 +0000 (11:56 -0500)]
Dimensions: Add offset prop fallback to FF for unreliable TR dimensions

Firefox incorrectly (or perhaps correctly) includes table borders in computed
dimensions, but they are the only one. Workaround this by testing for it and
falling back to offset properties

Fixes gh-4529
Closes gh-4808

3 years agoCore: Report browser errors in parseXML
Michał Gołębiowski-Owczarek [Tue, 8 Dec 2020 10:22:21 +0000 (11:22 +0100)]
Core: Report browser errors in parseXML

Fixes gh-4784
Closes gh-4816

3 years agoCore: Make jQuery.isXMLDoc accept falsy input
Michał Gołębiowski-Owczarek [Mon, 7 Dec 2020 20:09:15 +0000 (21:09 +0100)]
Core: Make jQuery.isXMLDoc accept falsy input

Fixes gh-4782
Closes gh-4814

3 years agoEvent: Make focus re-triggering not focus the original element back
Michał Gołębiowski-Owczarek [Mon, 7 Dec 2020 19:28:44 +0000 (20:28 +0100)]
Event: Make focus re-triggering not focus the original element back

If during a focus handler another focus event is triggered:

```js
elem1.on( "focus", function() {
elem2.trigger( "focus" );
} );
```

due to their synchronous nature everywhere outside of IE the hack added in
gh-4279 to leverage native events causes the native `.focus()` method to be
called last for the initial element, making it steal the focus back. Since
the native method is already being called in `leverageNative`, we can skip that
final call.

This aligns with changes to the `_default` method for the `click` event that
were added when `leverageNative` was introduced there.

A side effect of this change is that now `focusin` will only propagate to the
document for the last focused element. This is a change in behavior but it also
aligns us better with how this works with native methods.

Fixes gh-4382
Closes gh-4813
Ref gh-4279

4 years agoBuild: Test on Node.js 15
Michał Gołębiowski-Owczarek [Wed, 11 Nov 2020 22:02:22 +0000 (23:02 +0100)]
Build: Test on Node.js 15

Also, run browser tests on Node 14 instead of 12.

Closes gh-4802

4 years agoEvent: Don't crash if an element is removed on blur
Michał Gołębiowski-Owczarek [Mon, 19 Oct 2020 19:17:51 +0000 (21:17 +0200)]
Event: Don't crash if an element is removed on blur

In Chrome, if an element having a `focusout` handler is blurred by
clicking outside of it, it invokes the handler synchronously. If
that handler calls `.remove()` on the element, the data is cleared,
leaving private data undefined. We're reading a property from that
data so we need to guard against this.

Fixes gh-4417
Closes gh-4799

4 years agoBuild: Explicitly exclude the queue module from the slim build
Michał Gołębiowski-Owczarek [Mon, 28 Sep 2020 16:33:33 +0000 (18:33 +0200)]
Build: Explicitly exclude the queue module from the slim build

The queue module is not present in the slim build as it depends on deferred
and our Gruntfile specifies excluding deferred should also exclude queue:
https://github.com/jquery/jquery/blob/3.5.1/Gruntfile.js#L66
This commit makes this exclusion explicit so that the queue module never
accidentally gets re-included in the slim build if it stopped importing from
the deferred module directly.

Closes gh-4793

4 years agoCore: Drop support for Edge Legacy (i.e. non-Chromium Microsoft Edge)
Michał Gołębiowski-Owczarek [Tue, 22 Sep 2020 15:49:28 +0000 (17:49 +0200)]
Core: Drop support for Edge Legacy (i.e. non-Chromium Microsoft Edge)

Drop support for Edge Legacy: the non-Chromium, EdgeHTML-based Microsoft
Edge version. Also, restrict some workarounds that were applied
unconditionally in all browsers to run only in IE now. This slightly
increases the size but reduces the performance burden on modern browsers
that don't need the workarounds.

Also, clean up some comments & remove some obsolete workarounds.

Fixes gh-4568
Closes gh-4792

4 years agoManipulation: Respect script crossorigin attribute in DOM manipulation
高灰 [Tue, 22 Sep 2020 15:30:18 +0000 (23:30 +0800)]
Manipulation: Respect script crossorigin attribute in DOM manipulation

Fixes gh-4542
Closes gh-4563

Co-authored-by: Michał Gołębiowski-Owczarek <m.goleb@gmail.com>
4 years agoTests: Recognize callbacks with dots in the Node.js mock server 4785/head
Michał Gołębiowski-Owczarek [Wed, 2 Sep 2020 16:42:52 +0000 (18:42 +0200)]
Tests: Recognize callbacks with dots in the Node.js mock server

This aligns the Node.js server with the previous PHP one in sending `mock.php`
as a callback if there's no `callback` parameter in the query string which is
triggered by a recently added test. This prevents the request crashing on that
Node.js server and printing a JS error:
```
TypeError: Cannot read property '1' of null
```

Closes gh-4764
Ref gh-4754

4 years agoTests: Skip the "jQuery.ajax() on unload" test in Safari
Michał Gołębiowski-Owczarek [Wed, 2 Sep 2020 16:04:44 +0000 (18:04 +0200)]
Tests: Skip the "jQuery.ajax() on unload" test in Safari

The test has been already skipped in Chrome as it dropped support for such
requests and now Safari has joined the squad.

This will resolve AJAX test errors we've had for a while in Safari 13 & iOS 13.

Closes gh-4779

4 years agoBuild: Make the import/no-unused-modules ESLint rule work in WebStorm
Michał Gołębiowski-Owczarek [Wed, 2 Sep 2020 15:24:55 +0000 (17:24 +0200)]
Build: Make the import/no-unused-modules ESLint rule work in WebStorm

When run via WebStorm, the root path against which paths in the config of the
`import/no-unused-modules` ESLint rule are resolved is the path where the ESLint
config file that defines the rule lies, i.e. `src`. When run via the command
line, it's usually the root folder of the jQuery repository. This pattern
intends to catch both.

Note that we cannot specify two patterns here:
```js
[ "src/*.js", "*.js" ]
```
as they're analyzed individually and the rule crashes if a pattern cannot be
matched.

Closes gh-4777

4 years agoAttributes: Drop the `toggleClass(boolean|undefined)` signature
Michał Gołębiowski-Owczarek [Tue, 1 Sep 2020 08:42:03 +0000 (10:42 +0200)]
Attributes: Drop the `toggleClass(boolean|undefined)` signature

The behavior of this signature is not intuitive, especially if classes are
manipulated via other ways between `toggleClass` calls.

Fixes gh-3388
Closes gh-4766

4 years agoAjax: Make responseJSON work for erroneous same-domain JSONP requests
Michał Gołębiowski-Owczarek [Mon, 31 Aug 2020 22:02:44 +0000 (00:02 +0200)]
Ajax: Make responseJSON work for erroneous same-domain JSONP requests

Don't use a script tag for JSONP requests unless for cross-domain requests
or if scriptAttrs are provided. This makes the `responseJSON` property available
in JSONP error callbacks.

This fixes a regression from jQuery 3.5.0 introduced in gh-4379 which made
erroneous script responses to not be executed to follow native behavior.

The 3.x-stable branch doesn't need this fix as it doesn't use script tags for
regular async requests.

Closes gh-4778
Ref gh-4771
Ref gh-4773
Ref gh-4379

4 years agoEvent: Remove the event.which shim
Michał Gołębiowski-Owczarek [Wed, 26 Aug 2020 12:10:33 +0000 (14:10 +0200)]
Event: Remove the event.which shim

All supported browsers implement this property by themselves. The shim was only
needed for IE <9.

Fixes gh-3235
Closes gh-4765
Ref gh-4755

4 years agoAjax: Execute JSONP error script responses
Dallas Fraser [Tue, 25 Aug 2020 19:41:06 +0000 (15:41 -0400)]
Ajax: Execute JSONP error script responses

Issue gh-4379 was meant to be a bug fix but the JSONP case is a bit special:
under the hood it's a script but it simulates JSON responses in an environment
without a CORS setup and sending JSON payloads on error responses is quite
typical there.

This commit makes JSONP error responses still execute the payload. The regular
script error responses continue to be skipped.

Fixes gh-4771
Closes gh-4773

4 years agoAjax: Avoid CSP errors in the script transport for async requests
Michał Gołębiowski-Owczarek [Tue, 25 Aug 2020 19:28:30 +0000 (21:28 +0200)]
Ajax: Avoid CSP errors in the script transport for async requests

Until now, the AJAX script transport only used a script tag to load scripts
for cross-domain requests or ones with `scriptAttrs` set. This commit makes
it also used for all async requests to avoid CSP errors arising from usage
of inline scripts. This also makes `jQuery.getScript` not trigger CSP errors
as it uses the AJAX script transport under the hood.

For sync requests such a change is impossible and that's what `jQuery._evalUrl`
uses. Fixing that is tracked in gh-1895.

The commit also makes other type of requests using the script tag version of the
script transport set its type to "GET", namely async scripts & ones with
`scriptAttrs` set in addition to the existing cross-domain ones.

Fixes gh-3969
Closes gh-4763

4 years agoTests: Remove an unused local variable
Wonhyoung Park [Thu, 13 Aug 2020 11:24:30 +0000 (20:24 +0900)]
Tests: Remove an unused local variable

Closes gh-4769

4 years agoBuild: Append .eslintignore paths to grunt eslint paths
Ed Sanders [Tue, 21 Apr 2020 11:43:52 +0000 (12:43 +0100)]
Build: Append .eslintignore paths to grunt eslint paths

This allows us to turn off the `quiet` option which was suppressing warnings.
We can also set `maxWarnings` to 0 now that aren't any.

Closes gh-4689

4 years agoAjax: Drop the json to jsonp auto-promotion logic
Michał Gołębiowski-Owczarek [Mon, 27 Jul 2020 17:15:57 +0000 (19:15 +0200)]
Ajax: Drop the json to jsonp auto-promotion logic

Previously, `jQuery.ajax` with `dataType: 'json'` with a provided callback was
automatically converted to a jsonp request unless one also specified
`jsonp: false`. Today the preferred way of interacting with a cross-domain
backend is CORS which works in all browsers jQuery 4 will support.

Auto-promoting JSON requests to JSONP ones introduces a security issue as the
developer may be unaware they're not just downloading data but executing code
from a remote domain.

This commit disables the auto-promoting logic.

BREAKING CHANGE: to trigger a JSONP request, it's now required to specify
`dataType: "jsonp"`; previously some requests with `dataType: "json"` were
auto-promoted to JSONP.

Fixes gh-1799
Fixes gh-3376
Closes gh-4754

4 years agoBuild: Use the US spelling of "favor"
Necmettin Karakaya [Wed, 22 Jul 2020 14:12:54 +0000 (22:12 +0800)]
Build: Use the US spelling of "favor"

Closes gh-4752

4 years agoBuild: Fix commitplease husky config
Beatriz Rezener [Mon, 20 Jul 2020 17:06:39 +0000 (14:06 -0300)]
Build: Fix commitplease husky config

Fixes gh-4735
Closes gh-4737

4 years agoBuild: Update dependencies
Michał Gołębiowski-Owczarek [Wed, 15 Jul 2020 14:17:41 +0000 (16:17 +0200)]
Build: Update dependencies

This also resolves a security warning from GitHub about a vulnerable `request`
version - the new `testswarm` package version depends on a fixed `request`.

Closes gh-4732

4 years agobuild: set up periodic code scanning analysis
Timmy Willison [Thu, 25 Jun 2020 21:32:02 +0000 (17:32 -0400)]
build: set up periodic code scanning analysis

4 years agoManipulation: Avoid concatenating strings in buildFragment
Michał Gołębiowski-Owczarek [Wed, 10 Jun 2020 14:13:22 +0000 (16:13 +0200)]
Manipulation: Avoid concatenating strings in buildFragment

Concatenating HTML strings in buildFragment is a possible security risk as it
creates an opportunity of escaping the concatenated wrapper. It also makes it
impossible to support secure HTML wrappers like
[trusted types](https://web.dev/trusted-types/). It's safer to create wrapper
elements using `document.createElement` & `appendChild`.

The previous way was needed in jQuery <4 because IE <10 doesn't accept table
parts set via `innerHTML`, even if the element which contents are set is
a proper table element, e.g.:
```js
tr.innerHTML = "<td></td>";
```
The whole structure needs to be passed in one HTML string. jQuery 4 drops
support for IE <11 so this is no longer an issue; in older version we'd have
to duplicate the code paths.

IE <10 needed to have `<option>` elements wrapped in
`<select multiple="multiple">` but we no longer need that on master which
makes the `document.createElement` way shorter as we don't have to call
`setAttribute`.

All these improvements, apart from making logic more secure, decrease the
gzipped size by 58 bytes.

Closes gh-4724
Ref gh-4409
Ref angular/angular.js#17028

Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
4 years agoDocs: Update Frequently Reported Issues in the GitHub issue template
Michał Gołębiowski-Owczarek [Mon, 8 Jun 2020 18:25:11 +0000 (20:25 +0200)]
Docs: Update Frequently Reported Issues in the GitHub issue template

The issue about selectors with '#' being broken is old and no longer
frequently reported so this commit removes it from the list. On the other
hand, we're now getting lots of reports about the security fix in jQuery 3.5.0
that was also a breaking change: gh-4642. This one is now mentioned in the
list.

Closes gh-4728
Ref gh-4642

4 years agoBuild:Event: Make sure all source modules' exports are used (#4648)
Michał Gołębiowski-Owczarek [Tue, 2 Jun 2020 11:45:08 +0000 (13:45 +0200)]
Build:Event: Make sure all source modules' exports are used (#4648)

To achieve that, use `eslint-plugin-import`'s `no-unused-modules` rule.

Also, explicitly import `event/trigger.js` from `jquery.js`; so far it was
only imported from ajax.js, making it mistakenly skipped in the
`custom:slim,-deprecated` build.

4 years agoDeprecated: Remove jQuery.trim
Michał Gołębiowski-Owczarek [Mon, 18 May 2020 21:20:38 +0000 (23:20 +0200)]
Deprecated: Remove jQuery.trim

The API has been deprecated in 3.5.0 so it can be removed in 4.0.0.

Ref gh-4461
Closes gh-4695

4 years agoRelease: Remove an unused chalk dependency
Michał Gołębiowski-Owczarek [Mon, 18 May 2020 20:45:04 +0000 (22:45 +0200)]
Release: Remove an unused chalk dependency

Chalk was used for a Sizzle version check that's no longer there on `master`.

Closes gh-4712

4 years agoBuild: Update eslint-config-jquery, fix linting violations
Michał Gołębiowski-Owczarek [Mon, 18 May 2020 20:25:49 +0000 (22:25 +0200)]
Build: Update eslint-config-jquery, fix linting violations

Closes gh-4696
Ref jquery/eslint-config-jquery#15
Ref jquery/eslint-config-jquery#16

4 years agoTests: Remove remaining obsolete jQuery.cache references
Michał Gołębiowski-Owczarek [Mon, 18 May 2020 16:43:01 +0000 (18:43 +0200)]
Tests: Remove remaining obsolete jQuery.cache references

PR gh-4586 removed some of those but not all.

Closes gh-4715
Ref gh-4586

4 years agoDocs: Change JS Foundation mentions to OpenJS Foundation
Michał Gołębiowski-Owczarek [Mon, 18 May 2020 16:41:32 +0000 (18:41 +0200)]
Docs: Change JS Foundation mentions to OpenJS Foundation

Closes gh-4711

4 years agoDocs: add SECURITY.md, show security email address
Timmy Willison [Tue, 12 May 2020 18:33:45 +0000 (14:33 -0400)]
Docs: add SECURITY.md, show security email address

4 years agoBuild: Followups after introducing ES modules compiled via Rollup
Michał Gołębiowski-Owczarek [Tue, 5 May 2020 12:30:14 +0000 (14:30 +0200)]
Build: Followups after introducing ES modules compiled via Rollup

This commit cleans up a few comments & configurations that are out of date
after the migration to ES modules backed by a Rollup-based compilation.

Also, de-indent AMD modules. This will preserve a more similar
structure to the one on 3.x-stable where the body of the main `define`
wrapper is not indented.

Closes gh-4705

4 years agoCSS: Include `show`, `hide` & `toggle` methods in the jQuery slim build
Michał Gołębiowski-Owczarek [Tue, 5 May 2020 12:16:41 +0000 (14:16 +0200)]
CSS: Include `show`, `hide` & `toggle` methods in the jQuery slim build

The `show()`, `hide()` & `toggle()` methods were included in the 3.x jQuery
slim build. The jQuery master build accidentally started to exclude them as
they were only imported in the effects module and the new Rollup-based build
system follows the module dependency graph when excluding modules.

To resolve the issue, import the `css/showHide.js` file directly in the main
`jquery.js` file.

Closes gh-4704
Ref jquery/jquery-migrate#346

4 years agoBuild: Correct code indentations based on jQuery Style Guide
Wonseop Kim [Tue, 5 May 2020 08:49:27 +0000 (17:49 +0900)]
Build: Correct code indentations based on jQuery Style Guide

1. Correct code indentations based on jQuery Style Guide
   (contribute.jquery.org/style-guide/js/#spacing).
2. Add rules to "src/.eslintrc.json" to enable "enforcing consistent
   indentation", with minimal changes to the current code.

Closes gh-4672

4 years agoTests: Workaround failures in recent XSS tests in iOS 8 - 12
Michał Gołębiowski-Owczarek [Thu, 30 Apr 2020 19:25:29 +0000 (21:25 +0200)]
Tests: Workaround failures in recent XSS tests in iOS 8 - 12

iOS 8-12 parses `<noembed>` tags differently, executing this code. This is no
different to native behavior on that OS, though, so just accept it.

Ref gh-4685
Closes gh-4694

4 years agoDocs: Fix typos
Pierre Grimaud [Wed, 29 Apr 2020 19:18:23 +0000 (21:18 +0200)]
Docs: Fix typos

Closes gh-4686

4 years agoTests: Add tests for recently fixed manipulation XSS issues
Michał Gołębiowski-Owczarek [Wed, 29 Apr 2020 14:39:04 +0000 (16:39 +0200)]
Tests: Add tests for recently fixed manipulation XSS issues

Closes gh-4685
Ref gh-4642
Ref gh-4647

4 years agoBuild: Reduce the slim build header comment & jQuery.fn.jquery
Michał Gołębiowski-Owczarek [Mon, 27 Apr 2020 20:23:59 +0000 (22:23 +0200)]
Build: Reduce the slim build header comment & jQuery.fn.jquery

So far, the slim build was expanded to its full exclusion list, generating the
following `jQuery.fn.jquery`:
```
v4.0.0-pre -ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-deprecated/ajax-event-alias,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready
```

This commit changes it to just `v4.0.0-pre slim`. Only the pure slim build is
treated this way, any modification to it goes through the old expansion; e.g.
for `custom:slim,-deprecated` we get the following `jQuery.fn.jquery`:
```
v4.0.0-pre -deprecated,-deprecated/ajax-event-alias,-deprecated/event,-ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready
```

Since the version string is also put in the jQuery header comment, it also got
smaller.

Also, the logic to skip including the commit hash in the header comment - when
provided through the COMMIT environment variable which we do in Jenkins - in
minified builds headers has been applied to builds with exclusions as well.

Closes gh-4649

4 years agoTests: Use only one focusin/out handler per matching window & document
Michał Gołębiowski-Owczarek [Mon, 27 Apr 2020 19:37:06 +0000 (21:37 +0200)]
Tests: Use only one focusin/out handler per matching window & document

Backport tests from a jQuery 3.x fix that's not needed on `master`.

Also, fix the "focusin from an iframe" test to actually verify the behavior
from commit 1cecf64e5aa415367a7dae0b55c2dd17b591442d - the commit that
introduced the regression - to make sure we don't regress on either front.

The main part of the modified test was checking that focusin handling in an
iframe works and that's still checked. The test was also checking that it
doesn't propagate to the parent document, though, and, apparently, in IE it
does. This one test is now blacklisted in IE.

(cherry picked from 9e15d6b469556eccfa607c5ecf53b20c84529125)
(cherry picked from 1a4f10ddc37c34c6dc3a451ee451b5c6cf367399)

Ref gh-4652
Ref gh-4656
Closes gh-4657

4 years agoBuild: Move ESLint max-len disable-directive to dist/.eslintrc.json
Ed S [Mon, 27 Apr 2020 19:29:13 +0000 (20:29 +0100)]
Build: Move ESLint max-len disable-directive to dist/.eslintrc.json

This disable-directive only applies to the built version, so put
it in /dist. This avoids a warning about an unused directive in the
source version.

Closes gh-4676

4 years agoTests: Fix flakiness in the "jQuery.ajax() - JSONP - Same Domain" test
Michał Gołębiowski-Owczarek [Mon, 27 Apr 2020 18:22:39 +0000 (20:22 +0200)]
Tests: Fix flakiness in the "jQuery.ajax() - JSONP - Same Domain" test

The "jQuery.ajax() - JSONP - Same Domain" test is firing a request with
a duplicate "callback" parameter, something like (simplified):
```
mock.php?action=jsonp&callback=jQuery_1&callback=jQuery_2
```

There was a difference in how the PHP & Node.js implementations of the jsonp
action in the mock server handled situations like that. The PHP implementation
was using the latest parameter while the Node.js one was turning it into an
array but the code didn't handle this situation. Because of how JavaScript
stringifies arrays, while the PHP implementation injected the following code:
```js
jQuery_2(payload)
```
the Node.js one was injecting the following one:
```js
jQuery_1,jQuery_2(payload)
```
This is a comma expression in JavaScript; it so turned out that in the majority
of cases both callbacks were identical so it was more like:
```js
jQuery_1,jQuery_1(payload)
```
which evaluates to `jQuery_1(payload)` when `jQuery_1` is defined, making the
test go as expected. In many cases, though, especially on Travis, the callbacks
were different, triggering an `Uncaught ReferenceError` error & requiring
frequent manual re-runs of Travis builds.

This commit fixes the logic in the mock Node.js server, adding special handling
for arrays.

Closes gh-4687

4 years agoDocs: Update the link to the jsdom repository
Michał Gołębiowski-Owczarek [Sat, 25 Apr 2020 18:47:25 +0000 (20:47 +0200)]
Docs: Update the link to the jsdom repository

Closes gh-4684

4 years agoBuild: Test on Node.js 14, stop testing on Node.js 8 & 13
Michał Gołębiowski-Owczarek [Thu, 23 Apr 2020 11:24:35 +0000 (13:24 +0200)]
Build: Test on Node.js 14, stop testing on Node.js 8 & 13

Closes gh-4678

4 years agoBuild: Enable reportUnusedDisableDirectives in ESLint
Ed S [Mon, 20 Apr 2020 17:01:20 +0000 (18:01 +0100)]
Build: Enable reportUnusedDisableDirectives in ESLint

This forbids unnecessary `eslint-disable` comments.

Ref gh-4095
Closes gh-4520

4 years agoDocs: Use https for hyperlinks in README
Jonathan [Fri, 17 Apr 2020 09:46:49 +0000 (11:46 +0200)]
Docs: Use https for hyperlinks in README

Closes gh-4673

4 years agoAjax: Overwrite s.contentType with content-type header value, if any
Christian Wenz [Mon, 6 Apr 2020 19:15:55 +0000 (21:15 +0200)]
Ajax: Overwrite s.contentType with content-type header value, if any

This fixes the issue of "%20" in POST data being replaced with "+"
even for requests with content-type different from
"application/x-www-form-urlencoded", e.g. for "application/json".

Fixes gh-4119
Closes gh-4650

Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Michał Gołębiowski-Owczarek <m.goleb@gmail.com>
4 years agoManipulation: Make jQuery.htmlPrefilter an identity function
Michał Gołębiowski-Owczarek [Mon, 16 Mar 2020 20:49:29 +0000 (21:49 +0100)]
Manipulation: Make jQuery.htmlPrefilter an identity function

Closes gh-4642

4 years agoBuild: Resolve Travis config warnings
Michał Gołębiowski-Owczarek [Fri, 13 Mar 2020 16:16:07 +0000 (17:16 +0100)]
Build: Resolve Travis config warnings

Travis reports warnings in our config:
* root: deprecated key sudo (The key `sudo` has no effect anymore.)
* root: missing os, using the default linux
* root: key matrix is an alias for jobs, using jobs

They are all now resolved.

Closes gh-4636

4 years agoData:Event:Manipulation: Prevent collisions with Object.prototype
Michał Gołębiowski-Owczarek [Mon, 2 Mar 2020 22:02:42 +0000 (23:02 +0100)]
Data:Event:Manipulation: Prevent collisions with Object.prototype

Make sure events & data keys matching Object.prototype properties work.
A separate fix for such events on cloned elements was added as well.

Fixes gh-3256
Closes gh-4603

4 years agoRelease: Use an in-repository dist README fixture
Michał Gołębiowski-Owczarek [Mon, 2 Mar 2020 21:42:38 +0000 (22:42 +0100)]
Release: Use an in-repository dist README fixture

Use a dist README fixture kept in the jQuery repository instead of modifying
an existing one. This makes the jQuery repository the single source of truth
when it comes to jQuery releases and it makes it easier to make changes to
README without worrying how it will affect older jQuery lines.

The commit also ES6ifies build/release.js & build/release/dist.js

Closes gh-4614

4 years agoBuild: Enable ESLint one-var rule for var declarations in browser code
Michał Gołębiowski-Owczarek [Mon, 2 Mar 2020 21:25:35 +0000 (22:25 +0100)]
Build: Enable ESLint one-var rule for var declarations in browser code

Node.js code is written more & more commonly in ES6+ so it doesn't make sense
to enable it there. There are many violations in test code so it's disabled
there as well.

Closes gh-4615

4 years agoTests: Pass a number of necessary done() calls to assert.async()
Michał Gołębiowski-Owczarek [Mon, 2 Mar 2020 21:15:06 +0000 (22:15 +0100)]
Tests: Pass a number of necessary done() calls to assert.async()

It is no longer needed to create `done` wrappers in tests that require
multiple async operations to complete.

Closes gh-4633

4 years agoBuild: Add Christian Oliff to .mailmap & AUTHORS.txt
Michał Gołębiowski-Owczarek [Mon, 24 Feb 2020 18:10:32 +0000 (19:10 +0100)]
Build: Add Christian Oliff to .mailmap & AUTHORS.txt

Closes gh-4613

4 years agoCore: Fire iframe script in its context, add doc param in globalEval
Michał Gołębiowski-Owczarek [Mon, 10 Feb 2020 18:17:22 +0000 (19:17 +0100)]
Core: Fire iframe script in its context, add doc param in globalEval

1. Support passing custom document to jQuery.globalEval; the script will be
   invoked in the context of this document.
2. Fire external scripts appended to iframe contents in that iframe context;
   this was already supported & tested for inline scripts but not for external
   ones.

Fixes gh-4518
Closes gh-4601

4 years agoEvent: remove jQuery.event.global
Michał Gołębiowski-Owczarek [Mon, 10 Feb 2020 18:13:09 +0000 (19:13 +0100)]
Event: remove jQuery.event.global

jQuery.event.global has been write-only in the jQuery source for the past few
years; reading from it was removed in c2d6847de09a52496f78baebc04f317e11ece6d2
when fixing the trac-12989 bug.

Closes gh-4602

4 years agoDocs: Remove a mention of the event/alias.js module from README
Michał Gołębiowski-Owczarek [Mon, 27 Jan 2020 18:22:05 +0000 (19:22 +0100)]
Docs: Remove a mention of the event/alias.js module from README

The file contents now lie in deprecated/event.js so the README reference
is no longer correct.

Ref gh-4572
Closes gh-4599

4 years agoBuild: Lint the minified jQuery file as well - a Gruntfile fix
Michał Gołębiowski-Owczarek [Mon, 27 Jan 2020 18:21:23 +0000 (19:21 +0100)]
Build: Lint the minified jQuery file as well - a Gruntfile fix

While we have absolutely no style-related expectations to our minified file,
we do care that it's valid ES 5.1. This is now verified.

Fixes gh-3075
Ref gh-4594
Closes gh-4598

4 years agoAjax: Deprecate AJAX event aliases, inline event/alias into deprecated
Michał Gołębiowski-Owczarek [Tue, 21 Jan 2020 13:12:35 +0000 (14:12 +0100)]
Ajax: Deprecate AJAX event aliases, inline event/alias into deprecated

A new `src/deprecated` directory makes it possible to exclude some deprecated
APIs from a custom build when their respective "parent" module is excluded
without keeping that module outside of the `src/deprecated` directory or
the `src/deprecated.js` file.

Closes gh-4572

4 years agoCSS: Remove the opacity CSS hook
Michał Gołębiowski-Owczarek [Tue, 21 Jan 2020 13:11:06 +0000 (14:11 +0100)]
CSS: Remove the opacity CSS hook

The consequence is `.css( "opacity" )` will now return an empty string for
detached elements in standard-compliant browsers and "1" in IE & the legacy
Edge. That behavior is shared by most other CSS properties which we're not
normalizing either.

Closes gh-4593

4 years agoBuild: Lint the minified jQuery file as well
Michał Gołębiowski-Owczarek [Tue, 21 Jan 2020 12:51:03 +0000 (13:51 +0100)]
Build: Lint the minified jQuery file as well

While we have absolutely no style-related expectations to our minified file,
we do care that it's valid ES 5.1. This is now verified.

Fixes gh-3075
Closes gh-4594

4 years agoBuild: Add intuitive names to Travis jobs
Michał Gołębiowski-Owczarek [Mon, 20 Jan 2020 18:19:08 +0000 (19:19 +0100)]
Build: Add intuitive names to Travis jobs

Otherwise it's hard to see at a glance that a particular job is running
on Firefox ESR, for example.

Closes gh-4596

4 years agoCore: Exclude callbacks & deferred modules in the slim build as well
Michał Gołębiowski-Owczarek [Mon, 20 Jan 2020 17:58:23 +0000 (18:58 +0100)]
Core: Exclude callbacks & deferred modules in the slim build as well

So far, the slim build only excluded ajax & effects modules. As many web apps
right now rely on native Promises, often with a polyfill for legacy browsers,
deferred & callbacks modules are not that useful for sites that already exclude
ajax & effects modules.

This decreases the gzipped minified size of the slim module by 1760 bytes,
to 19706 bytes (below 20k!).

Closes gh-4553

4 years agoAttributes: Refactor val(): don't strip carriage return, isolate IE workarounds
Michał Gołębiowski-Owczarek [Mon, 13 Jan 2020 18:25:01 +0000 (19:25 +0100)]
Attributes: Refactor val(): don't strip carriage return, isolate IE workarounds

Before this change, `val()` was stripping out carriage return characters from
the returned value. No test has relied on that. The logic was different for
option elements as its custom defined hook was omitting this stripping logic.

This commit gets rid of the carriage return removal and isolates the IE-only
select val getter to be skipped in other browsers.

Closes gh-4585

4 years agoTests: Remove obsolete jQuery data tests
Michał Gołębiowski-Owczarek [Mon, 13 Jan 2020 18:23:01 +0000 (19:23 +0100)]
Tests: Remove obsolete jQuery data tests

The tests relied on `jQuery.cache` so they only ever worked in jQuery 1.x.

Closes gh-4586

4 years agoAttributes: Don't set the type attr hook at all outside of IE
Michał Gołębiowski-Owczarek [Mon, 13 Jan 2020 18:22:08 +0000 (19:22 +0100)]
Attributes: Don't set the type attr hook at all outside of IE

This removes a needless function call in modern browsers.

Closes gh-4587

4 years agoBuild: Make dev mode work in Karma again, serve source files from disk
Michał Gołębiowski-Owczarek [Tue, 7 Jan 2020 23:35:55 +0000 (00:35 +0100)]
Build: Make dev mode work in Karma again, serve source files from disk

PR gh-4550 added support for running ES modules & AMD tests via Karma. This
required reading the `esmodules` & `amd` props from both `QUnit.config` &
`QUnit.urlParams`. By picking these two properties manually, the `dev` one
stopped being respected while ones handled directly by QUnit were fine (like
`hidepassed`). Instead of maintaining the full list of options, the code now
iterates over QUnit URL config and handles the fallbacks in a more generic way.

Apart from that, all jQuery source & test files are now read directly from disk
instead of being cached by Karma so that one can run `grunt karma:chrome-debug`
& work on a fix without restarting that Karma run after each change. A similar
effect could have been achieved by setting `autoWatch` to `true` but then the
main Karma page runs tests in an iframe by default when
`grunt karma:chrome-debug` is run instead of relying on the current debug flow.

Closes gh-4574
Ref gh-4550

4 years agoBuild:Tests: Fix custom build tests, verify on Travis
Michał Gołębiowski-Owczarek [Tue, 7 Jan 2020 22:59:08 +0000 (23:59 +0100)]
Build:Tests: Fix custom build tests, verify on Travis

This commit fixes unit tests for the following builds:

1. The no-deprecated build: `custom:-deprecated`
2. The current slim build: `custom:-ajax,-effects`
3. The future (#4553) slim build: `custom:-ajax,-callbacks,-deferred,-effects`

It also adds separate Travis jobs for the no-deprecated & slim builds.

Closes gh-4577

4 years agoDocs: Update links to EdgeHTML issues to go through Web Archive
Michał Gołębiowski-Owczarek [Tue, 7 Jan 2020 22:45:41 +0000 (23:45 +0100)]
Docs: Update links to EdgeHTML issues to go through Web Archive

With Microsoft going Chromium with Edge, its old EdgeHTML issues were all
removed. :(

Closes gh-4584

4 years agoBuild: Create a `grunt custom:slim` alias for the Slim build (#4578)
Michał Gołębiowski-Owczarek [Tue, 7 Jan 2020 15:42:49 +0000 (16:42 +0100)]
Build: Create a `grunt custom:slim` alias for the Slim build (#4578)

Closes gh-4578

4 years agoSelector: Remove the "a:enabled" workaround for Chrome <=77
Michał Gołębiowski-Owczarek [Mon, 16 Dec 2019 18:43:38 +0000 (19:43 +0100)]
Selector: Remove the "a:enabled" workaround for Chrome <=77

Remove the workaround for a broken `:enabled` pseudo-class on anchor elements
in Chrome <=77. These versions of Chrome considers anchor elements with the
`href` attribute as matching `:enabled`.

Closes gh-4569

4 years agoDocs: direct users to GitHub docs for cloning the repo
Timmy Willison [Mon, 16 Dec 2019 18:37:16 +0000 (13:37 -0500)]
Docs: direct users to GitHub docs for cloning the repo

Ref gh-4556
Close gh-4571

4 years agoBuild: Make Karma work in ES modules mode
Michał Gołębiowski-Owczarek [Mon, 16 Dec 2019 18:33:49 +0000 (19:33 +0100)]
Build: Make Karma work in ES modules mode

Also, run such a suite in CI to make sure modules are working as expected
when used directly.

Closes gh-4550

4 years agoBuild: Auto-convert sources to AMD
Michał Gołębiowski-Owczarek [Mon, 9 Dec 2019 19:00:44 +0000 (20:00 +0100)]
Build: Auto-convert sources to AMD

jQuery source has been migrated in gh-4541 from AMD to ES modules. To maintain
support for consumers of our AMD modules, this commits adds a task transpiling
the ES modules sources in `src/` to AMD in `amd/`.

A "Load with AMD" checkbox was also restored to the QUnit setup. Note that,
contrary to jQuery 3.x, AMD files need to be generated via `grunt amd` or
`grunt` as sources are not authored in ECMAScript modules. To achieve a similar
no-compile experience during jQuery 4.x testing, use the new "Load as modules"
checkbox which works in all supported browsers except for IE & Edge (the
legacy, EdgeHTML-based one).

Ref gh-4541
Closes gh-4554

4 years agoEvent: Only attach events to objects that accept data - for real
Michał Gołębiowski-Owczarek [Mon, 9 Dec 2019 18:50:14 +0000 (19:50 +0100)]
Event: Only attach events to objects that accept data - for real

There was a check in jQuery.event.add that was supposed to make it a noop
for objects that don't accept data like text or comment nodes. The problem was
the check was incorrect: it assumed `dataPriv.get( elem )` returns a falsy
value for an `elem` that doesn't accept data but that's not the case - we get
an empty object then. The check was changed to use `acceptData` directly.

Fixes gh-4397
Closes gh-4558

4 years agoDocs: Change OS X to macOS in README
Christian Oliff [Tue, 3 Dec 2019 11:35:48 +0000 (20:35 +0900)]
Docs: Change OS X to macOS in README

macOS has been around for long enough to update the naming here.

Closes gh-4552

4 years agoBuild: Fix the Windows build
Michał Gołębiowski-Owczarek [Mon, 2 Dec 2019 18:55:19 +0000 (19:55 +0100)]
Build: Fix the Windows build

This commit gets rid of rollup-plugin-hypothetical in favor of a simpler
inline Rollup plugin that fits our need and is compatible with Windows.

Fixes gh-4548
Closes gh-4549

4 years agoBuild: Require extensions for ES6 imports, prevent import cycles
Michał Gołębiowski-Owczarek [Mon, 25 Nov 2019 19:16:53 +0000 (20:16 +0100)]
Build: Require extensions for ES6 imports, prevent import cycles

jQuery source is now authored in ECMAScript modules. Native browser support
for them requires full file names including extensions. Rollup works even
if import paths don't specify extensions, though, so one import slipped
through without such an extension, breaking native browser import of
src/jquery.js.

A new ESLint rule using eslint-plugin-import prevents us from regressing
on that front.

Also, eslint-plugin-import's no-cycle rule is used to avoid import cycles.

Closes gh-4544
Ref gh-4541
Ref 075320149ae30a5c593c06b2fb015bdf033e0acf

5 years agoBuild: Fix the import path to serialize.js from ajax.js
Michał Gołębiowski-Owczarek [Tue, 19 Nov 2019 14:18:27 +0000 (15:18 +0100)]
Build: Fix the import path to serialize.js from ajax.js

5 years agoSelector: Make empty attribute selectors work in IE again
Michał Gołębiowski-Owczarek [Mon, 18 Nov 2019 21:10:55 +0000 (22:10 +0100)]
Selector: Make empty attribute selectors work in IE again

qSA in IE 11/Edge often (but not always) don't find elements with an empty
name attribute selector (`[name=""]`). Detect that & fall back to Sizzle
traversal.

Interestingly, IE 10 & older don't seem to have the issue.

Fixes gh-4435
Closes gh-4510

5 years agoCore: Migrate from AMD to ES modules 🎉
Michał Gołębiowski-Owczarek [Mon, 18 Nov 2019 20:15:03 +0000 (21:15 +0100)]
Core: Migrate from AMD to ES modules 🎉

Migrate all source AMD modules to ECMAScript modules. The final bundle
is compiled by a custom build process that uses Rollup under the hood.

Test files themselves are still loaded via RequireJS as that has to work in
IE 11.

Tests can now be run in "Load as modules" mode which replaces the previous
"Load with AMD" option. That option of running tests doesn't work in IE
and Edge as it requires support for dynamic imports.

Some of the changes required by the migration:
* check `typeof` of `noGlobal` instead of using the variable directly
  as it's not available when modules are used
* change the nonce module to be an object as ECMASscript module exports
  are immutable
* remove some unused exports
* import `./core/parseHTML.js` directly in `jquery.js` so that it's not
  being cut out when the `ajax` module is excluded in a custom compilation

Closes gh-4541

5 years agoTests: Skip a "width/height on a table row with phantom borders" test in Firefox
Michał Gołębiowski-Owczarek [Mon, 28 Oct 2019 19:38:33 +0000 (20:38 +0100)]
Tests: Skip a "width/height on a table row with phantom borders" test in Firefox

Firefox 70 & newer fail this test but the issue there is more profound - Firefox
doesn't subtract borders from table row computed widths.

Closes gh-4537
Ref jquery/jquery#4529
Ref https://bugzilla.mozilla.org/show_bug.cgi?id=1590837
Ref w3c/csswg-drafts#4444

(cherry picked from commit c79e1d5fefc50b1df0a1c2ca3f06b567e79c0f9b)

5 years agoTests: Don't test synchronous XHR on unload in Chrome
Michał Gołębiowski-Owczarek [Mon, 28 Oct 2019 19:27:49 +0000 (20:27 +0100)]
Tests: Don't test synchronous XHR on unload in Chrome

Chrome 78 dropped support for synchronous XHR requests inside of
beforeunload, unload, pagehide, and visibilitychange event handlers.
See https://bugs.chromium.org/p/chromium/issues/detail?id=952452

Closes gh-4536

(cherry picked from commit c5b48c8caa58e7b73164ac033bf726a072903708)

5 years agoBuild: Run tests on Travis only on browsers defined in the config
Michał Gołębiowski-Owczarek [Tue, 22 Oct 2019 18:49:37 +0000 (20:49 +0200)]
Build: Run tests on Travis only on browsers defined in the config

The environmental variable BROWSERS was being created but it wasn't read in the
list of browsers to pass to Karma.

Closes gh-4532

5 years agoBuild: Run tests on Firefox ESR as well
Michał Gołębiowski-Owczarek [Tue, 22 Oct 2019 18:20:53 +0000 (20:20 +0200)]
Build: Run tests on Firefox ESR as well

Closes gh-4530

(cherry picked from commit 0a73b94a21d4c30b5598b95923dc73d640e07b99)

5 years agoBuild: Run tests on Node.js 13 in addition to 8, 10 & 12
Michał Gołębiowski-Owczarek [Tue, 22 Oct 2019 18:03:36 +0000 (20:03 +0200)]
Build: Run tests on Node.js 13 in addition to 8, 10 & 12

Closes gh-4528

5 years agoBuild: Run tests on Travis on FirefoxHeadless as well
Michał Gołębiowski-Owczarek [Mon, 21 Oct 2019 17:06:39 +0000 (19:06 +0200)]
Build: Run tests on Travis on FirefoxHeadless as well

Also, run them on both ChromeHeadless & FirefoxHeadless locally on
`grunt karma:main`.

Plus, so far, the chrome addons were installed for all the jobs, even
the ones that weren't used for browser testing. Changing that makes
those jobs faster.

Closes gh-4524

5 years agoSelector: Use shallow document comparisons in uniqueSort
Michał Gołębiowski-Owczarek [Mon, 21 Oct 2019 17:04:48 +0000 (19:04 +0200)]
Selector: Use shallow document comparisons in uniqueSort

IE/Edge sometimes crash when comparing documents between frames using the strict
equality operator (`===` & `!==`). Funnily enough, shallow comparisons
(`==` & `!=`) work without crashing.

The change to shallow comparisons in `src/selector.js` was done in gh-4471 but
relevant changes in `src/selector/uniqueSort.js` were missed. Those changes
have landed in Sizzle in jquery/sizzle#459.

Fixes gh-4441
Closes gh-4512
Ref gh-4471
Ref jquery/sizzle#459

5 years agoDocs: Update most URLs to HTTPS
Michał Gołębiowski-Owczarek [Mon, 21 Oct 2019 17:03:48 +0000 (19:03 +0200)]
Docs: Update most URLs to HTTPS

Closes gh-4511

5 years agoSelector: Add a test for throwing on post-comma invalid selectors
Michał Gołębiowski-Owczarek [Mon, 21 Oct 2019 17:03:03 +0000 (19:03 +0200)]
Selector: Add a test for throwing on post-comma invalid selectors

Sizzle's PR jquery/sizzle#456 introduced a test catching not throwing on
badly-escaped identifiers by Firefox 3.6-5. Unfortunately, it was placed just
before a test Opera 10-11 failed, making Opera fail quicker and not adding
a post-comma invalid selector to rbuggyQSA.

The issue was fixed in jquery/sizzle#463. This jQuery commit backports the test
that Sizzle PR added as no workarounds are needed in browsers jQuery supports.

Closes gh-4516
Ref jquery/sizzle#456
Ref jquery/sizzle#463

5 years agoTests: Stop using jQuery.find in tests
Michał Gołębiowski-Owczarek [Mon, 21 Oct 2019 17:02:22 +0000 (19:02 +0200)]
Tests: Stop using jQuery.find in tests

This prepares us for possibly hiding jQuery.find in jQuery 4.0.

Closes gh-4517

5 years agoCSS: Workaround buggy getComputedStyle on table rows in IE/Edge
Michał Gołębiowski-Owczarek [Mon, 14 Oct 2019 16:41:35 +0000 (18:41 +0200)]
CSS: Workaround buggy getComputedStyle on table rows in IE/Edge

Fixes gh-4490
Closes gh-4506

5 years agoSelector: Make selectors with leading combinators use qSA again
Michał Gołębiowski-Owczarek [Mon, 14 Oct 2019 16:28:19 +0000 (18:28 +0200)]
Selector: Make selectors with leading combinators use qSA again

An optimization added in jquery/sizzle#431 skips the temporary IDs for selectors
not using child or descendant combinators. For sibling combinators, though, this
pushes a selector with a leading combinator to qSA directly which crashes and
falls back to a slower Sizzle route.

This commit makes selectors with leading combinators not skip the selector
rewriting. Note that after jquery/jquery#4454 & jquery/sizzle#453, all modern
browsers other than Edge leverage the :scope pseudo-class, avoiding temporary
id attributes.

Closes gh-4509
Ref jquery/sizzle#431

5 years agoBuild: Require strict mode in Node.js scripts via ESLint
Michał Gołębiowski-Owczarek [Tue, 8 Oct 2019 22:17:55 +0000 (00:17 +0200)]
Build: Require strict mode in Node.js scripts via ESLint

So far, only browser-based JS files were required to be in strict mode (in the
function form). This commit adds such a requirement to Node.js scripts where
the global form is preferred. All Node.js scripts in sloppy mode were
converted to strict mode.

Closes gh-4499

5 years agoManipulation:Selector: Use the nodeName util where possible to save size
Michał Gołębiowski-Owczarek [Tue, 8 Oct 2019 20:41:59 +0000 (22:41 +0200)]
Manipulation:Selector: Use the nodeName util where possible to save size

Saves 20 bytes.

Closes gh-4504

5 years agoDocs: Convert link to Homebrew from HTTP to HTTPS
Christian Oliff [Mon, 7 Oct 2019 06:45:40 +0000 (15:45 +0900)]
Docs: Convert link to Homebrew from HTTP to HTTPS

`http://brew.sh/` -> `https://brew.sh/`

Closes gh-4501

5 years agoBuild: Support jquery-release --dry-run flag
Michał Gołębiowski-Owczarek [Sat, 5 Oct 2019 16:48:27 +0000 (18:48 +0200)]
Build: Support jquery-release --dry-run flag

Without this change passing `--dry-run` to jquery-release still pushes to the
jquery-dist repository which is dangerous as one can assume `--dry-run` to be
safe from external side effects.

Close gh-4498

5 years agoBuild: Stop copying src/core.js to dist on release
Michał Gołębiowski-Owczarek [Fri, 4 Oct 2019 14:13:14 +0000 (16:13 +0200)]
Build: Stop copying src/core.js to dist on release

File `src/core.js` has started erroneously being copied to `dist/` in gh-2981.

Fixes gh-4489
Closes gh-4492
Ref gh-2979
Ref gh-2981