Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
3 Jun 2012 at 18:42 UTC
Updated:
4 Jan 2014 at 02:05 UTC
Jump to comment: Most recent, Most recent file
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 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 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_7011in 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 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