Currently, the 'editor' role is created via config in the panopoly_core module.

However, this can cause some problems with Features:

  1. We don't want reverting panopoly_core to reset the permissions to their original state
  2. Even though it's fine for a profile to override a config yaml from a module, there's a long-standing Features bug that makes Features act strangely when you try to do it: #2668562: Not detecting overrides in profile config files

Instead, creating it programmatically in hook_install() will eliminate these problems.

CommentFileSizeAuthor
#2 panopoly-editor-role-3380561-2.patch2.44 KBdsnopek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
FileSize
2.44 KB

Here's the patch!

dsnopek’s picture

Status: Needs review » Fixed

Committed

  • dsnopek committed c8a33a9a on 8.x-2.x
    Issue #3380561: Create 'editor' role in code (rather than config) to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.