Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#16 | metatag-vbo-integration-1624266.patch | 4.91 KB | angrytoast |
| |||
#6 | metatag-merging-metatag_vbo-into-metatag-1624266.patch | 4.13 KB | L-four |
#1 | metatag-merging metatag_vbo into metatag-1624266.patch | 4.13 KB | L-four |
Comments
Comment #1
L-four CreditAttribution: L-four commentedComment #2
L-four CreditAttribution: L-four commentedComment #4
L-four CreditAttribution: L-four commented#1: metatag-merging metatag_vbo into metatag-1624266.patch queued for re-testing.
Comment #6
L-four CreditAttribution: L-four commentedI am guessing test bot doesn't like spaces, TT _ TT
Comment #7
L-four CreditAttribution: L-four commentedComment #8
DamienMcKennaGood idea. Two things I'd like to see:
Comment #9
Jawi CreditAttribution: Jawi commentedThis would be a nice feature!
any progress on testing?
Comment #10
Dave ReidIt 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()?
Comment #11
DamienMcKennaComment #12
sahaj CreditAttribution: sahaj commentedThis is really a must-have feature for some projects. Unfortunately, I'm getting 'An illegal choice has been detected. Please contact the site administrator.' error caused by the Robots (Advanced tags) showing '0 0 0 0 0 0 0 0 0 0'.
If I left the field empty, it don't works too. Does somebody have a fix or a workaround?
(I'm using Metatag 7.x-1.10)
Comment #13
sahaj CreditAttribution: sahaj commentedI noticed the lines 36-40 in metatag_vbo.module:
and don't know if there is mistake there or in metatag module, but it don't do the job.
Any help is welcome.
Comment #14
geoffreyr CreditAttribution: geoffreyr commentedI'm looking at doing a bit of work on this. Just wondering, as far as determining a language code for localisation goes, should we derive it at entity save time, or explicitly set a language code at the form stage?
Comment #15
errand CreditAttribution: errand commentedHello sahaj,
i had the same problem when i installed this module.
You wouldn't believe me but i just changed 'checkboxes'; to 'checkbox'; resaved and of course nothing happened, BUT when i changed it back and flushed caches it worked! SO ITS WORKING
But now i'm having another issue: after updating the metatags, i have to update nodes manually because node_save doesn't update metatags...
Comment #16
angrytoast CreditAttribution: angrytoast commentedThere hasn't been much activity on this issue, so I took some of the work already done in the previous patches and addressed some feedback listed above:
This changes the metatag update logic within the bulk operation action to merge
array()
like values, such as the robots tags.I'm actually not sure how actions work with Drupal core so I was unable to test that part.
Comment #17
castelar CreditAttribution: castelar commentedPatch in #16 works for me! Really appreciate this @angrytoast @L-four and all that helped.
Comment #19
DamienMcKennaCommitted. Thanks!
Comment #20
DamienMcKenna