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.
Since Google+ has been shut down as of April 2019, does the Metatag: Google+ submodule serve any purpose?
If not, it could be removed.
Remaining tasks
- Update stored configuration.
- Update per-entity values.
Update script to uninstall the module
Comment | File | Size | Author |
---|---|---|---|
#37 | metatag-n3065441-37.patch | 3.16 KB | DamienMcKenna |
#32 | metatag-n3065441-32.patch | 4.07 KB | DamienMcKenna |
#27 | metatag-n3065441-27.patch | 19.29 KB | DamienMcKenna |
#26 | metatag-n3065441-26.patch | 17.39 KB | DamienMcKenna |
#25 | metatag-n3065441-25.patch | 15.86 KB | DamienMcKenna |
Issue fork metatag-3065441
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
DamienMcKennaAgreed.
Comment #3
DamienMcKennaRemoving modules gets tricky. We'll need to have an update script to uninstall the module, remove all values from configuration and entities, and blanking out some of the files, which we'll actually remove later.
Comment #4
DamienMcKennaComment #6
eelkeblokIn the issue fork I added two update hooks to the main module, the second of which is incomplete. I suppose we will also need a post_update hook to clean up content (make sure we only save config or content when it was actually updated, since this sort of stuff can take a while).
Comment #7
DamienMcKennaThis is a good start, thank you.
The second update script needs to be refined to:
I'm not sure how D8 handles things being deleted before the respective plugins are uninstalled, I'd be concerned at it blowing up sites. As such I think we should make the module "hidden" and physically delete the files later.
Comment #8
DamienMcKennaComment #9
DamienMcKennaLet's get this into Metatag 2.0.0.
Comment #10
DamienMcKennaComment #12
DamienMcKennaWIP patch based upon the PR. This needs work to update the configuration and update the per-entity values.
Comment #13
DamienMcKennaComment #14
DamienMcKennaNeeds work to finish the update scripts.
Comment #15
DamienMcKennaWIP on the update scripts and test coverage. Skipping test coverage, for now.
Comment #16
DamienMcKennaWorking on this.
Comment #17
DamienMcKennaMore progress.
Comment #18
DamienMcKennaSo the update scripts don't work if metatag_google_plus has been removed already, so we might need to delete the files later in order to do the update script. Either that or move the update scripts to 8.x-1.x and delete the files in 2.0.x. Decisions, decisions.
Comment #19
DamienMcKennaDoes this patch apply cleanly?
Comment #20
solideogloria CreditAttribution: solideogloria commentedThe patch applies. However, I no longer use this module, so I have no way to test if the update script works.
Comment #21
DamienMcKennaOh, the patch doesn't apply because of the fixture file. Dangit.
Comment #23
DamienMcKennaI committed the fixture change separately so that the tests can run.
Comment #24
DamienMcKennaIn order to avoid the conflict between configuration schema requirements and removing the files that contain the necessary schema, we might just have to skip testing the configuration changes. Or move the uninstall to 8.x-1.x and then remove the files in 2.0.x.
Comment #25
DamienMcKennaBecause we can't guarantee that everyone will have updated to the newest 8.x-1.x release before upgrading to 2.0.x, we cannot assume that the update scripts would have been executed before the files disappeared. Therefore we need to hide the module in v2 and then plan for them to be removed in v3. *sigh*
However, some of the tags could be removed.
Comment #26
DamienMcKennaI can remove all of the tags and just leave the info file in place, that seems to work.
Comment #27
DamienMcKennaRemoving even more bits!
Comment #29
DamienMcKennaCommitted.
Comment #30
DamienMcKennaComment #31
DamienMcKennaNeed some follow-up to make the post-update script names to be generic so they can be reused for other tags that are being removed.
Comment #32
DamienMcKennaRenaming stuff.
Comment #34
DamienMcKennaCommitted.
Comment #37
DamienMcKennaNeeded to add more meta tags to the removal list.
Comment #39
DamienMcKennaAll done.