Download & Extend

Add support views bulk operations for mass updating metatags

Project:Metatag
Version:7.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:L-four
Status:needs work

Issue Summary

I made this module http://drupal.org/sandbox/L-four/1603672 for one of my clients so they could quickly update there metatag using views bulk operations.

I made this Issue to merging this views bulk operations functionality with the core metatag module.

Comments

#1

Title:Add support views bulk operations for mas updating metatags» Add support views bulk operations for mass updating metatags
AttachmentSizeStatusTest resultOperations
metatag-merging metatag_vbo into metatag-1624266.patch4.13 KBIdleFAILED: [[SimpleTest]]: [MySQL] Fetch test patch: failed to retrieve [metatag-merging metatag_vbo into metatag-1624266.patch] from [drupal.org].View details | Re-test

#2

Status:active» needs review

#3

Status:needs review» needs work

The last submitted patch, metatag-merging metatag_vbo into metatag-1624266.patch, failed testing.

#4

Status:needs work» needs review

#1: metatag-merging metatag_vbo into metatag-1624266.patch queued for re-testing.

#5

Status:needs review» needs work

The last submitted patch, metatag-merging metatag_vbo into metatag-1624266.patch, failed testing.

#6

I am guessing test bot doesn't like spaces, TT _ TT

AttachmentSizeStatusTest resultOperations
metatag-merging-metatag_vbo-into-metatag-1624266.patch4.13 KBIdlePASSED: [[SimpleTest]]: [MySQL] 15 pass(es).View details | Re-test

#7

Status:needs work» needs review

#8

Status:needs review» needs work

Good idea. Two things I'd like to see:

  • Please test it using the basic Drupal admin pages to ensure the actions work without the VBO module.
  • Rewrite it as a patch to the main Metatag module.

#9

This would be a nice feature!

any progress on testing?

#10

It would be nice if this could re-use metatag_metatags_form() which handles all the logic of filtering out values which match the default configuration for the entity. I feel like this patch is potentially duplicating a lot of code. Also agreed that this should be easy to support from the main module. Does VBO support placing its hooks inside a sub-file like metatag.vbo.inc using hook_hook_info()?

nobody click here