WordPress.org

Ready to get started?Download WordPress

Forums

Media Tags
Media Library inline fix (5 posts)

  1. alx359
    Member
    Posted 3 years ago #

    When adding/uploading media to a post/page, in the Media Library tab there's an option to assign/remove tags in bulk from a group of checked images. That feature was not working. To make it work:

    media-tags\js\mediatags_bulk_common.js ~li 59

    if ((select_media_tags == "") && (media_tags_input == ""))
    	{
    		jQuery('#media-tags-error').html('<p>Please enter or select which Media Tags should be applied to the selected Media Items.</p>');
    		jQuery('#media-tags-error').focus();
    		return false;
    	}
    /* [alx359] redundant code. Tags not being updated in inline view because of this
    	else
    	{
    		jQuery('#media-tags-error').html('<p></p>');
    	}
    */
  2. Paul Menard
    Member
    Plugin Author

    Posted 3 years ago #

    Actually I have to disagree. The code is not redundant. The else part of the code simply clears the $media-tags-error element on the popup.

    But I do see the issue. Was not aware this was not working. Thanks.

  3. alx359
    Member
    Posted 3 years ago #

    Sorry, not proper wording used. Error handling is not redundant, think just needed to be in other place down the code.

    Thank you so much, Paul, for the fixes of this great plugin!

  4. Paul Menard
    Member
    Plugin Author

    Posted 3 years ago #

    No man! Thank you for all the bug fix code. I've been busy on client projects and have not really had a chance to do comprehensive review of bug reports or check coding I'v done in the past.

  5. alx359
    Member
    Posted 3 years ago #

    All the thank-you's go to you, Paul. I'm just humbly trying to give something back for this great giving of yours! Your plugin is great work, especially the powerful bulk parts.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags