Media/Library menu URLs mismatch

  • Resolved Pixelatic

    (@davidhealy)


    10 years ago

    Not sure if this is a bug or by design… with “Display Media/Library” unchecked in MLA’s General Settings, the “Media” menu link leads to /wp-admin/admin.php?page=mla-menu while the submenu link leads to /wp-admin/upload.php?page=mla-menu

    This causes a minor issue with WP Document Revisions, which is designed to filter out Document post type attachments from displaying in the Media Library by testing for “upload.php” or “media-upload.php” in the URL.

    MLA’s “Media” menu link (“admin.php”) therefore returns an unfiltered Media Library view containing Document post type attachments, while MLA’s submenu link (“upload.php”) returns a correctly filtered Media Library view.

    As a workaround I’ve used JavaScript to replace “admin.php” with “upload.php” for MLA’s “Media” menu link.

    If you could consider using /wp-admin/upload.php?page=mla-menu for both menu links I’d be grateful – thanks for an excellent plugin.

    https://wordpress.org/plugins/media-library-assistant/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author David Lingren

    (@dglingren)

    10 years ago

    You’re welcome for the plugin, and thanks for raising this issue. In fact, this happens whenever the “mla-menu” page is the highest priority, default Media submenu.

    The “admin.php” link is a result of WordPress’ logic to handle admin submenu pages generated by plugins. It’s hard-coded in an “echo” statement in /wp-admin/menu-header.php with no “official” way to change the default.

    I was able to find a workaround based on my earlier work to hide the default “Library” submenu. I have implemented and tested the workaround and it will be in the next MLA version.

    If you’d like to test the improvements I made, please send me a note with your e-mail address and I will give you a copy by return e-mail. You can use the Contact Us page at our web site:

    Fair Trade Judaica/Contact Us

    I will leave this topic unresolved for now and post an update when the next version goes out. Thanks for your interest, your understanding and your patience.

    Plugin Author David Lingren

    (@dglingren)

    9 years, 11 months ago

    I have released MLA version 1.82, which includes the workaround to resolve this issue.

    I am marking this topic resolved, but please update it if you have any problems with the fix or further questions/suggestions. Thanks again for raising this issue and for your interest in the plugin.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Media/Library menu URLs mismatch’ is closed to new replies.