Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[user:create] Ensure $roles is an array when creating users. #4134

Merged
merged 1 commit into from
Sep 6, 2019

Conversation

ndouglas
Copy link
Contributor

This prevents an error from being thrown when constructing the success message. See #4133.

This prevents an error from being thrown when constructing the success message.  See hechoendrupal#4133.
@ndouglas ndouglas changed the title Ensure $roles is an array when creating users. [user:create] Ensure $roles is an array when creating users. Aug 20, 2019
@enzolutions
Copy link
Contributor

Thank you @ndouglas

@enzolutions enzolutions merged commit b84ad5c into hechoendrupal:master Sep 6, 2019
enzolutions pushed a commit that referenced this pull request Oct 10, 2019
* Update services.yml

* Fix Travis tests

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* 1

* Ensure $roles is an array when creating users. (#4134)

This prevents an error from being thrown when constructing the success message.  See #4133.

* Generate revisional entity content is broken (#4139)

* Add bundle permissions for content entity (#4138)

* Add command option has-bundle-permissions and use on entity.

* Add (Entity}Permissions.php file.

* Add permission_callback to permissions.yml file.

* Added fix from #4139 as it hurts.

* Add own permissions checks.

* Add --default option to database:add (#4143)

This option allows to set the database as the default one instead of
adding a new database in addition to the default one.

See: #4142

* Update version to 1.9.3 (#4144)

* Update version to 1.9.3

* Remove Module/InstallCommand::moduleRequirement

* Remove
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants