Make something amazing, for anyone, at DrupalCon. Standard prices end on August 26.
If either the editor or administrator role is missing an error will be thrown since those roles are hard-coded. A patch is coming soon that will bypass setting permissions if the roles don't exist.
Here is the patch. It also cleans up some whitespace. I also noticed that the multiple calls to user_role_grant_permissions was unnecessary. It can just take an array of permissions. I think this can be done in another issue.
Looks great. Reviewed and committed to -dev.
Automatically closed -- issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.