Ref. #1556248: metatag_ui was not found and #1281624: Merge metatag_ui back into metatag.module.

Metatag_ui no longer exists. This is causing confusion for some users. I'm wondering if we can remove the info and module files of the _ui module entirely.

CommentFileSizeAuthor
#1 metatag-1778286-1-remove-ui-module.patch686 bytesalberto56

Comments

alberto56’s picture

alberto56 commented
Title: Now that metatag_ui no longer exists, can the code be removed (to avoid confusion)? » Now that metatag_ui no longer exists, remove module_ui code (to avoid confusion)
Category: support » feature
Status: Active » Needs review
StatusFileSize
new metatag-1778286-1-remove-ui-module.patch686 bytes

Here's a patch that does this.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Postponed

Thanks, but I want to wait until after v1.0 to remove the files, there are still lots of installations that have the module.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes
Status: Postponed » Fixed

Committed. Thanks alberto56!

  • DamienMcKenna committed de64eee on 7.x-1.x 
    Issue #1778286 by alberto56: Removed the deprecated metatag_ui module.
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI I re-committed the patch as I'd forgotten to give you the proper attribution. Sorry about that.

  • DamienMcKenna committed d61e6ca on 7.x-1.x 
    Revert "Issue #1778286 by alberto56: Removed the deprecated metatag_ui...
  • DamienMcKenna committed e4ca69a on 7.x-1.x authored by alberto56
    Issue #1778286 by alberto56: Removed the deprecated metatag_ui module.
alberto56’s picture

alberto56 commented

Thanks! Keep up the good work,

Cheers,

Albert.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.