JQuery .proxy deprecated and QMIGRATE: Boolean attribute 'disabled' is not set to its lowercased name in Jquery 4.0

[neo2k23]

Hi

I am using the latest Test jQuery Updates plugin to test jquery 4.0 and i get the following jquery migrate warnings when trying to insert a image from the media library in a plugin setting.

		JQMIGRATE: jQuery.proxy() is deprecated jquery-migrate-4.0.0-beta.1.js:136:28
		console.trace() JQMIGRATE: jQuery.proxy() is deprecated jquery-migrate-4.0.0-beta.1.js:142:13
			jQuery 3
			Uploader http://server.tst/wp-includes/js/plupload/wp-plupload.js?ver=7.0-alpha-61387:74
			ready http://server.tst/wp-includes/js/media-views.js?ver=7.0-alpha-61387:9608

and this one.

		console.trace() JQMIGRATE: Boolean attribute 'disabled' is not set to its lowercased name jquery-migrate-4.0.0-beta.1.js:142:13
			jQuery 8
			render http://server.tst/wp-includes/js/media-views.js?ver=7.0-alpha-61387:1337
			Backbone 5
			refresh http://server.tst/wp-includes/js/media-views.js?ver=7.0-

Please fix. Thank you

[neo2k23]

I found one fix

changing line 1337 and line 5794 in media-views.js from attr to prop fixes that warning message.

    this.$el.prop( 'disabled', model.disabled );

changing line 74 in wp-upload.js to

	this[ key ] = this[ key ].bind( this );

fixes the .proxy warning but i am not sure that is the correct fix as .bind does not set guid on the bound function instances so that proxied event handlers can be removed without storing the instance of the proxied function.

[azaozz]

Replacing attr with prop should be straightforward. Thinking many of these were fixed earlier but seems there are more places where this is needed.

Not completely sure about $.proxy too. Needs more testing.

Trac ticket: https://core.trac.wordpress.org/ticket/64425

• Updated attr to use prop.
• Use bind instead of proxy with a comment to update the function used with namespace.

[hbhalodia]

Hi @neo2k23 @azaozz, I have added the PR for the same. As said, we need to test the bind related changes, because we do not know in what context it the function can be instantiated.

Thanks,

[neo2k23]

@hbhalodia @azaozz i scanned the current beta version wp 7 for all .proxy instances and there are in total 4 files using .proxy

​https://share.zight.com/jkuNXqA8

medialeement-and-player.js
tinymce.min.js
wp-upload.js
wp-tinymce.js

It was already clear to me that just replacing .proxy with .bind could cause issues as .bind does not set guid on the bound function instances so that proxied event handlers can be removed without storing the instance of the proxied function.

Thanks all and have a great XMas.

[azaozz]

Replying to neo2k23:

medialeement-and-player.js
tinymce.min.js
wp-upload.js
wp-tinymce.js

Most of these are (older) external libraries, not sure if they can be/should be patched by WP. At the same time it seems ​there aren't plans upstream to remove support for $.proxy for now, so can probably leave it as-is. Can fix wp-upload.js though.

Thanks all and have a great XMas.

Thanks, you too!