I'm attaching a patch to the UPGRADE.txt file to add the following statement in the D6->D7 upgrade instructions:
"In addition to updating to the latest available version of Drupal 6.x core,
you must also upgrade all of your contributed modules for Drupal to their
latest Drupal 6.x versions."
The instructions currently explain the need to update to the latest version of Drupal 6.x but are not explicit about the need to update to the latest version for contrib modules specifically. It is especially important to point this out for D6->D7 upgrades because many module developers are not including the D6 hook_update_N() functions for their modules in their .install files. This is happening much more frequently than it was with the D5->D6 upgrades. (It's probably happening because D7 introduced a more radical change to the database API, so module developers would have to substantially rewrite their hook_update_N() functions to get them working in D7 and are (perhaps understandably) reluctant to do so.
Comment | File | Size | Author |
---|---|---|---|
#13 | upgrade-1167996-13.patch | 612 bytes | BWPanda |
#9 | upgrade-1167996-9.patch | 4.51 KB | drupal_was_my_past |
#5 | upgrade-1167996-5.patch | 4.54 KB | drupal_was_my_past |
#3 | upgrade-1167996-3.patch | 4.54 KB | drupal_was_my_past |
#1 | UPGRADE.txt_needs_additional_instruction-1167996-1.patch | 5.03 KB | Sheldon Rampton |
Comments
Comment #1
Sheldon Rampton CreditAttribution: Sheldon Rampton commentedHere's the patch.
Comment #2
jhodgdonThere are changes in your patch that are not related to this fix (which I think is a good idea). Please make a patch that doesn't make the other changes (such as removing the mention of the .zip format, which we would please like to keep in there). I think that's the only one...
I also think it would be helpful to mention how you go about updating the modules? Maybe not...
And when you add a patch, please set the status to "needs review" so that both the test bot and people know it's time to review it.
And finally, this technically needs to be patched against D8 and then applied to both d8 and d7. Hmmm. The D8 upgrade guide probably needs to have its numbers updated first. Sigh.
Thanks!
Comment #3
drupal_was_my_past CreditAttribution: drupal_was_my_past commentedRe-roll patch from #1.
Comment #4
jhodgdonLooks pretty good... One mention of 6.x crept in though:
Comment #5
drupal_was_my_past CreditAttribution: drupal_was_my_past commentedGood catch @jhodgdon. Fixed in the attached patch.
Comment #6
jhodgdonLooks good, thanks!
Comment #7
Sheldon Rampton CreditAttribution: Sheldon Rampton commentedOf course the wording will have to be changed back to 6.x when the D8 patch is backported to D7.
Comment #8
Dries CreditAttribution: Dries commentedCommitted to 8.x. Moving to 7.x-dev for backporting.
Comment #9
drupal_was_my_past CreditAttribution: drupal_was_my_past commentedPatch for 7.x.
Comment #10
jhodgdonThanks, looks ready to me!
Comment #11
webchickLooks great!
Committed and pushed to 7.x. Thanks!
Comment #13
BWPanda CreditAttribution: BWPanda commentedWhile upgrading a site from 6.x to 7.x, I noticed that one of the 7.x references wasn't changed to 6.x in UPGRADE.txt. The attached patch fixes this.
Comment #14
jhodgdonThanks for finding and fixing this error! I've committed this change to 7.x.