jenkins-bot has submitted this change and it was merged. Change subject: Fix undefined $user in OpenStackNovaProject::deleteMember ......................................................................
Fix undefined $user in OpenStackNovaProject::deleteMember Bug: T147716 Change-Id: Ie1f809375d66237fce85f61202a4e6f5e1eb4605 (cherry picked from commit 548194e2f3dcb2df8f8f87b565cbf5c469eb7979) --- M nova/OpenStackNovaProject.php 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Hashar: Looks good to me, approved jenkins-bot: Verified diff --git a/nova/OpenStackNovaProject.php b/nova/OpenStackNovaProject.php index 8f938ac..bfedd7d 100644 --- a/nova/OpenStackNovaProject.php +++ b/nova/OpenStackNovaProject.php @@ -381,6 +381,7 @@ $ldap->printDebug( "Failed to remove $username from " . $sudoer->getSudoerName(), NONSENSITIVE ); } } + $user = new OpenStackNovaUser( $username ); $ldap->printDebug( "Successfully removed $user->userDN from $this->projectname", NONSENSITIVE ); $this->deleteRoleCaches( $username ); $this->editArticle(); -- To view, visit https://gerrit.wikimedia.org/r/314855 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1f809375d66237fce85f61202a4e6f5e1eb4605 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/OpenStackManager Gerrit-Branch: REL1_27 Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com> Gerrit-Reviewer: Alex Monk <a...@wikimedia.org> Gerrit-Reviewer: Andrew Bogott <abog...@wikimedia.org> Gerrit-Reviewer: Chad <ch...@wikimedia.org> Gerrit-Reviewer: Chasemp <r...@wikimedia.org> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com> Gerrit-Reviewer: Reedy <re...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits