Changeset 22872

Timestamp:

11/27/2012 04:17:53 PM (12 years ago)

Author:

ryan

Message:

Always send nocache_headers() for admin-ajax.php. This prevents iOS from caching ajax calls. ​http://stackoverflow.com/questions/12506897/is-safari-on-ios-6-caching-ajax-results

Turn off multi selection uploads for mobile devices. Currently Plupload cannot upload multiple files in iOS Safari. Only the first file makes it to the queue.

Empty wp on unload to work around caching in iOS Safari.

Props azaozz, miqrogroove, nacin

fixes #22552

Location:

trunk

Files:

• wp-admin/admin-ajax.php (modified) (1 diff)
• wp-admin/includes/media.php (modified) (1 diff)
• wp-includes/js/media-models.js (modified) (1 diff)
• wp-includes/media.php (modified) (1 diff)

trunk/wp-admin/admin-ajax.php

@@ -37 +37 @@
 
 send_nosniff_header();
+nocache_headers();
 
 do_action( 'admin_init' );

trunk/wp-admin/includes/media.php

@@ -1509 +1509 @@
 );
 
+// Multi-file uploading doesn't currently work in iOS Safari,
+// single-file allows the built-in camera to be used as source for images
+if ( wp_is_mobile() )
+    $plupload_init['multi_selection'] = false;
+
 $plupload_init = apply_filters( 'plupload_init', $plupload_init );
 

trunk/wp-includes/js/media-models.js

@@ -810 +810 @@
     });
 
+    // Clean up. Prevents mobile browsers caching
+    $(window).on('unload', function(){
+        window.wp = null;
+    });
+
 }(jQuery));

trunk/wp-includes/media.php

@@ -1255 +1255 @@
         'urlstream_upload'    => true,
     );
+    
+    // Multi-file uploading doesn't currently work in iOS Safari,
+    // single-file allows the built-in camera to be used as source for images
+    if ( wp_is_mobile() )
+        $defaults['multi_selection'] = false;
 
     $defaults = apply_filters( 'plupload_default_settings', $defaults );