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.
As noticed by hefox in this issue: #1920342-8: Feature implementation of roles/permissions is broken we should not write roles to features code without checking whether they still exist in the database.
The current code builds the exported code solely from the data it receives from features, but this data might not correspond with the current state of the database.
Note: Please note that you will need to use this patch together with the patch from #1265168-38: Rebuild the file list properly when a feature is enabled or disabled in order to make Features work correctly for OG.
Comment | File | Size | Author |
---|---|---|---|
#1 | og-Do_not_export_nonexisting_roles-2021673-1.patch | 1.68 KB | xatoo |
Comments
Comment #1
xatoo CreditAttribution: xatoo commentedComment #3
xatoo CreditAttribution: xatoo commented#1: og-Do_not_export_nonexisting_roles-2021673-1.patch queued for re-testing.
Comment #5
jlapp CreditAttribution: jlapp commented#1: og-Do_not_export_nonexisting_roles-2021673-1.patch queued for re-testing.
Comment #6
jlapp CreditAttribution: jlapp commentedThe patch in #1 worked perfectly for me. The issue I was experiencing was:
I see that the patch is passing all tests now. Is there any reason this should not be committed?
Comment #7
xatoo CreditAttribution: xatoo commentedWell, one reason for not committing: Somebody needs to RTCB it. :)
Comment #8
gbaudoin CreditAttribution: gbaudoin commentedThis also fixed my issue.
Comment #8.0
gbaudoin CreditAttribution: gbaudoin commentedAdded warning about the need for an extra patch
Comment #9
guillaumev CreditAttribution: guillaumev commentedThis also fixed my issue.
Comment #10
aurelianzaha CreditAttribution: aurelianzaha at jobiqo - job board technology commentedHi,
Strange, a very old patch and still not committed. the patch works for me as well..
Comment #11
izmeez CreditAttribution: izmeez commentedThe patch in this issue includes a hunk that is also changed in #987568: Sortable roles by weight thus they conflict with each other. Once committed the other will need to be re-rolled.