Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter Seth Carstens

    (@sethcarstens)

    the issue:
    https://jqueryui.com/upgrade-guide/1.10/#removed-ui-isover-and-ui-isoveraxis-methods

    The solution:

    function umm_admin_enqueue($hook) {
    	if( 'users.php' != $hook && 'user-meta-manager' != $_GET['page'])
    		return;
    	wp_enqueue_script( 'jquery-ui-sortable');
    }
    add_action( 'admin_enqueue_scripts', 'umm_admin_enqueue' );

    Plugin Author Jason Lau

    (@jason-lau)

    Hi Seth,

    Thanks! I will release a fix with the next version, which is in the works.

    Plugin Author Jason Lau

    (@jason-lau)

    This is fixed in version 3.0.5. It wasn’t a js-related issue, but rather a php-related one.

    Thanks for reporting!

    Thread Starter Seth Carstens

    (@sethcarstens)

    I’m quite confident its JS related. As you can see above, this is a Javscript library that is not longer loaded in jquiry UI core. The error, is a javascript error. The fix, is done in php…. but its not a PHP error, unless there was ALSO a PHP error with your update functions…..

    Thread Starter Seth Carstens

    (@sethcarstens)

    it does appear to be fixed though, regardless of how you did it.

    Also I wanted to take a moment to say thank you for the plugin, you did amazing work.

    Plugin Author Jason Lau

    (@jason-lau)

    The php problem was UMM was not correctly loading the array of sorted meta keys to display. The sort order was updated properly in the database, but not in the UI.

    UMM loaded it’s own version of jQuery UI, which was bundled with the plugin. The js error you’re referring to probably occurred if jQuery UI was loaded from WP core instead of UMM.

    To prevent it from happening to others in the future, I deprecated the UMM version of jQuery UI and now load it from the WP core version. This prevents multiple instances of jQuery UI.

    Thanks, I’m glad you find the plugin useful. It’s even more useful without bugs! ??

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Sort is broken…’ is closed to new replies.