Adds Administrator role and Content editor role recipes
Closes #3454605
Merge request reports
Activity
added 1 commit
- 91255f94 - Adds Administrator and Content editor role recipes
added 1 commit
- 0dca6137 - Removes standard_roles recipe and updates admin and editor to use ensure_exists
- Resolved by Jim Birch
- Resolved by Jim Birch
added 1 commit
- 1d7640e7 - Adds 'access content', moves node permissions to standard recipe.
130 130 $expected_list = $comparer->getEmptyChangelist(); 131 131 // We expect core.extension to be different because standard is no longer 132 132 // installed. 133 $expected_list['update'] = ['core.extension']; 133 $expected_list['update'] = ['core.extension', 'user.role.content_editor']; 134 134 $this->assertSame($expected_list, $comparer->getChangelist()); - Comment on lines 130 to 134
I tried to figure out if anything update in permissions(remove/add even w.r.t to logic or not) in this file /core/recipes/content_editor_role/recipe.yml
then 'user.role.content_editor' config included in the change list: $this->configImporter()->getStorageComparer()->getChangelist();
To pass the test w.r.t to condition assertSame(), included 'user.role.content_editor' as well
not sure may be need to update assertSame() condition, either added change 'll be fine, any other concerns?
Edited by Pooja Sharma We actually don't want to change this line. The thing we need to do is figure out why
user.role.content_editor
is changing, and alter our recipes in such a way that it doesn't get changed. The point of the Standard recipe set is that it should generate a site that is identical to the Standard profile.
added 1 commit
- 7e6c91c6 - issue/3454605: reverted the changes in test file.