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.
Motivation
After updating to new code I had situation that shouldn't be: BOTCHA is enabled and Moopapi is not. That is why BOTCHA was looked like it doesn't work at all.
Solution
We should add hook_updateN, which performs necessary operations: enable Moopapi and flush all caches.
Comments
Comment #1
PatchRanger CreditAttribution: PatchRanger commentedThis hook is useless, because it is an installation requirement that we can not work around: user must download all dependencies first, then enable them. Closing as "works as designed" since there is nothing to do with it.
Comment #2
PatchRanger CreditAttribution: PatchRanger commentedWe can't download and enable Moopapi instead of user - it is a fact.
But we should flush caches to empower the changes to take place.
If we don't do it - all admin pages dissappear, which is definitely confusing.
Comment #3
PatchRanger CreditAttribution: PatchRanger commentedFixed for D7: http://drupalcode.org/project/botcha.git/commit/b26a83a.
Comment #4
PatchRanger CreditAttribution: PatchRanger commentedPorted by http://drupalcode.org/project/botcha.git/commit/8f52991.