Problem/Motivation

When the Role Test Accounts module is enabled through configuration management (for example: `drush cim`), the module generates notices in the command line output.

[warning] array_values() expects parameter 1 to be array, null given RoleTestAccountsManager.php:99
 [warning] array_values() expects parameter 1 to be array, null given RoleTestAccountsManager.php:106
 [warning] array_diff(): Expected parameter 1 to be an array, null given RoleTestAccountsManager.php:109
 [warning] Invalid argument supplied for foreach() RoleTestAccountsManager.php:109
 [warning] array_diff(): Expected parameter 1 to be an array, null given RoleTestAccountsManager.php:113
 [warning] Invalid argument supplied for foreach() RoleTestAccountsManager.php:113

Steps to reproduce

Enable role_test_accounts through configuration management

Proposed resolution

Implement sanity checks to prevent notices when role_test_accounts is enabled through configuration management

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#3 3223273-2.patch934 bytesidebr

Comments

idebr created an issue. See original summary.

idebr’s picture

Issue summary: View changes
idebr’s picture

Status: Active » Needs review
StatusFileSize
new934 bytes

Attached patch implements sanity checks to prevent notices when role_test_accounts is enabled through configuration management

  • 9ce8305 committed on 8.x-1.x
    Issue #3223273 by idebr: Notices when role_test_accounts is enabled...
idebr’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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