TomodomoCo / total-slider

A WordPress plugin for creating, editing and removing ‘slides’ with text and images (for a homepage, for example).
GNU General Public License v2.0
7 stars 2 forks source link

Upload not possible in Chrome (Win8) #53

Closed debiesven closed 10 years ago

debiesven commented 11 years ago

When I upload an image and I want to Set it as background, than I get the following error and a blank page in the lightbox. I use Wordpress 3.5.

Uncaught TypeError: Cannot call method 'split' of undefined interface.min.js:74 window.send_to_editor interface.min.js:74 (anonymous function)

screenshot screenshot-2

PeterUpfold commented 11 years ago

Sven,

Thanks for reporting this issue.

Can you enable SCRIPT_DEBUG in your wp-config.php file like so, and reproduce the error again?

define('SCRIPT_DEBUG', true);

This enables the non-minified JavaScript file, so that we can see more clearly where this error is occurring.

Thanks!

debiesven commented 11 years ago

Hi,

When I enable script_debug I get the error as on the screenshot. screenshot-1

PeterUpfold commented 11 years ago

I have done a test with a new install of WordPress 3.5, Total Slider 1.1.1 and the latest Chrome on Windows 8. I can't seem to find any issues with uploading — either uploading a new image, or selecting one from the gallery work as expected.

Perhaps this suggests that there may be a conflict or incompatibility between Total Slider and another plugin that you are using on your particular site.

What plugins and themes do you have on your site?

debiesven commented 11 years ago

This are the plugins I have:

And I have the following theme:

PeterUpfold commented 11 years ago

Apologies for the very long delay in getting back to you about this.

This appears to be an incompatibility with the Simple Grid Theme Responsive 2.0, because that theme is throwing an E_NOTICE in certain admin pages. This message is confusing the uploader and preventing the upload from completing.

I managed to work around this error by modifying the mytheme_add_admin() function in themes/simplegridthemes/settings.php to prevent the E_NOTICE from showing up.

In this mytheme_add_admin() function, replace the line:

if ( $_GET['page'] == basename(__FILE__) ) {

with:

if ( array_key_exists('page', $_GET) && $_GET['page'] == basename(__FILE__) ) {

This avoids the E_NOTICE, and when I make this change, I can successfully upload files via the Total Slider admin interface.

If this issue is still relevant for you, please let me know if it works!

chrisvanpatten commented 10 years ago

Closing this issue. If the proposed fix doesn't work, let us know.