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.
When Drupal 7.13 was released adding forum_update_7011(), it looks like it was left outside the updates-7.x-extra documentation group.
This patch fixes that.
Comment | File | Size | Author |
---|---|---|---|
#4 | 1615312-4.patch | 8.13 KB | dsdeiz |
forum-updates-7x-extra.patch | 657 bytes | David_Rothstein | |
Comments
Comment #1
jhodgdonUmmm... OK... That group doesn't have any description:
http://api.drupal.org/api/drupal/modules!system!system.install/group/upd...
but I guess it is the updates that are between 7.x versions, as opposed to:
http://api.drupal.org/api/drupal/modules!system!system.install/group/upd...
Are there other updates that should be added to this group, along with this patch?
Comment #2
David_Rothstein CreditAttribution: David_Rothstein commentedLooks like there might be a bunch. I just tried installing Drupal 7.0, enabling all core modules, then updating to Drupal 7.x and running drush updatedb. Here's what I got:
Comment #3
jhodgdonThat is an excellent way to check! All of those should therefore be in the "extra updates" group, and we should also make sure that none of those are in the "6-7 updates" group. (URLs on api.drupal.org are in comment #1).
That sounds like a good Novice patch project...
Comment #4
dsdeiz CreditAttribution: dsdeiz commentedThis seems to be already on the "Extra system updates for 7.x" group.
Sorry, I'm a little bit confused. I see
forum_update_7011
in between the "@addtogroup updates-7.x-extra" and "@} End of "addtogroup..." but I'm not sure why it's not listed here.Seems like these are already listed on the group.
Attached my first attempt.
Comment #5
dsdeiz CreditAttribution: dsdeiz commentedComment #6
jhodgdonRegarding forum_update_7011, it looks like your patch fixed the problem there correctly.
Regarding field.module, it looks like 7002 is already listed on http://api.drupal.org/api/drupal/modules!system!system.install/group/upd...
Regarding menu.module - I agree these are already listed. The list in #2 was the list of all that *should* be listed -- I don't think David was trying to say they were all missing, just that this was the list of what should be there and we need to make a patch to make sure they're all there.
Anyway... I haven't checked this patch completely yet, but it looks like you have the right idea. I'll try to give it a more complete review soon unless someone else does first. Thanks!
Comment #7
jhodgdonIt all checked out OK -- thanks for the patch! Committed to 7.x.
Follow-up issue:
#1688020: The Extra System Updates group has no description and wrong name