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.In testing out http://drupal.org/node/153998, I tried the following experiment:
1. Enable all Drupal core modules in 5.x.
2. Enable all possible permissions for a particular role.
3. Upgrade to Drupal 6.x.
In doing so, I found that some permissions were mysteriously removed for no apparent reason.
One is the "edit any book content" permission (formerly known as "edit book content" in 5.x), which was due to a simple mistake in book.install... this is fixed in the attached patch.
The other is the "administer locales" permission in 5.x. It looks like it's been split up into two permissions in 6.x ("administer languages" and "translate interface"), but nothing gets changed on the DB update so neither of these get enabled. I would think both of these should get enabled for any role that previously had "adminster locales", but I may be wrong about this because I don't know much about the new language system... For now I haven't included it in the attached patch. Suggestions from anyone who knows more about this are welcome... thanks.
| Comment | File | Size | Author |
|---|---|---|---|
| restore_book_permissions.patch | 762 bytes | David_Rothstein | |
Comments
Comment #1
catchJust realised you're the same person as on the other issue so no need to cross link :)
We'll also need an upgrade path from rc1 - rc2. Leaving at CNR, since the other issue is likely to have a database update anyway, could possibly merge this into that.
Comment #2
pwolanin CreditAttribution: pwolanin commentedlooks like someone missed the book module when they changed the permission names
Comment #3
catchActually it looks like a hunk didn't get committed from the final patch here: http://drupal.org/node/120960#new
Comment #4
pwolanin CreditAttribution: pwolanin commentedhow the hell do hunks of patches get missed?! This is about the 3rd time in the last couple weeks.
Comment #5
Dries CreditAttribution: Dries commentedI've committed this patch; while that fixes the book permission, we might still want to look at the other permissions?
Comment #6
pwolanin CreditAttribution: pwolanin commented@Dries - I took a quick look through the original patch, and seemed like the poll module changes got in correctly.
Comment #7
Gábor HojtsyYes, both locale permissions can be given to those who had the previous permission. Seems like that one is missing here.
Comment #8
Gábor HojtsyMerged the remaining RC1 - RC2 update of the book permissions and the locale update to http://drupal.org/node/153998 so marking this fixed for the earlier committed patch.
Comment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.