Remove the Flash and Silverlight options from Plupload

[azaozz]

Plupload includes Flash and Silverlight as fallbacks when the browser doesn't support HTML 5.0. Since we dropped support for old IE, all currently supported browsers use HTML 5.0. Even if Plupload is used in an old unsupported browser, it will revert to using "HTML 4" (a simple file field).

[azaozz]

In 41755.patch:

• Modify moxie.js and plupload.js: remove support for the Silverlight and Flash runtimes and hardcore html5,html4 for the runtimes setting.
• Add the above non-minified files to wp-includes/js/plupload as they are different than the originals.
• Delete the Flash and Silverlight executables.

Tested in latest Firefox, Edge, and Chrome, also in Edge emulation of IE11, 10, and 8 (just in case). All seems to be working properly.

[azaozz]

In 41755.2.patch: also add plupload.flash.swf and plupload.silverlight.xap to $_old_files to be deleted when upgrading.

The patch also includes the ​last GPLv2 version of Plupload.

[azaozz]

In 41328:

Plupload:

• Update to 2.1.9.
• Include the non-minified plupload.js and moxie.js.
• Remove support for Flash and Silverlight, the available runtime options are html5 and html4.
• Delete plupload.flash.swf and plupload.silverlight.xap.

Fixes #41755.

[azaozz]

Committed this to make it easier for testing, and also to be able to do other fixes for plupload.js and moxie.js. Feel free to reopen if any bugs/errors.

[gmariani405]

Enabled debugging of plupload and moxie

[gmariani405]

Added a switch to be able to actually debug with the individual scripts when SCRIPT_DEBUG is enabled.

[azaozz]

Replying to gmariani405:

Thanks for the patch!

Was thinking about this too. If we want it, would probably be better to drop plupload.full.min.js and keep the 'plupload' dependency on moxie.js, something like:

$scripts->add( 'moxie', "/wp-includes/js/plupload/moxie$suffix.js", array(), '1.3.5' );
$scripts->add( 'plupload', "/wp-includes/js/plupload/plupload$suffix.js", array( 'moxie' ), '2.1.9' );

Not really sure if that's needed. Plupload is an external package/library. There are changes to the original files, so we added our versions as non-minified (readable) files, but we aren't going to "actively develop" it in core.

[afercia]

When running grunt precommit I get 1 test failure:

1) Tests_Admin_IncludesUpdateCore::test_new_files_are_not_in_old_files_array_compiled with data set #559 ('wp-includes/js/plupload/plupload.js')
Failed asserting that file "/ ... /wordpress-develop/public_html/build/wp-includes/js/plupload/plupload.js" does not exist.

I guess because this change re-introduces wp-includes/js/plupload/plupload.js which is listed in the $_old_files since WP 3.9 ? See [28085]

[netweb]

Replying to afercia:

When running grunt precommit I get 1 test failure:

1) Tests_Admin_IncludesUpdateCore::test_new_files_are_not_in_old_files_array_compiled with data set #559 ('wp-includes/js/plupload/plupload.js')
Failed asserting that file "/ ... /wordpress-develop/public_html/build/wp-includes/js/plupload/plupload.js" does not exist.

I guess because this change re-introduces wp-includes/js/plupload/plupload.js which is listed in the $_old_files since WP 3.9 ? See [28085]

Note the path @afercia of the file is in the build folder, removing your local /build folder via rm -rf build/ or running grunt will clean out the build folder and remove that file that is currently causes the test to fail. Tests should pass fine after that.

[afercia]

@netweb thanks, I've tried that but grunt precommit builds the JS files and then runs the test, so it still fails :)

[netweb]

Replying to afercia:

@netweb thanks, I've tried that but grunt precommit builds the JS files and then runs the test, so it still fails :)

Right you are :+1: , I hadn't re-run grunt build myself after removing the the build/ folder ;)

[azaozz]

Replying to afercia:

I guess because this change re-introduces wp-includes/js/plupload/plupload.js which is listed in the $_old_files since WP 3.9 ? See [28085]

Good catch, thanks! Will remove that file from $_old_files.

grunt precommit throws weird errors here, will need to fix it somehow (it requires "global" svn to be installed, but many apps (IDEs, Tortoisesvn, etc.) have "local" copies, and sometimes the svn versions differ and... all gets really messy).

[azaozz]

In 41332:

Plupload: remove the reinstated plupload.js from the list of old files.

Props afercia.
See #41755.

[azaozz]

In 41570:

Plupload: minify and load plupload.js and moxie.js separately to make them easier to debug.

Props gmariani405, azaozz.
Fixes #41755.

[azaozz]

In 41571:

Plupload: also delete plupload.full.min.js when updating.

See #41755.

[westonruter]

In 42041:

Tests: Remove references to deleted JS files in QUnit tests.

Amends [41570], [40584].
See #41755, #40690.