Add options to create/remove groups via occ#10334
Add options to create/remove groups via occ#10334denismosolov merged 1 commit intonextcloud:masterfrom
Conversation
|
Nice stuff! |
rullzer
left a comment
rullzer
left a comment
There was a problem hiding this comment.
We require you to sign off on your commits.
git commit --amend -s
Should do the trick
Also, could you run:
bash build/autoloaderchecker.sh
To make sure the autoloader is up to date?
| @@ -0,0 +1,67 @@ | |||
| <?php | |||
| /** | |||
There was a problem hiding this comment.
Could you make the file strict?
declare(strict_types=1);
core/Command/Group/Add.php
Outdated
| return 1; | ||
| } else { | ||
| $this->groupManager->createGroup($groupName); | ||
| $group = $this->groupManager->get($groupName); |
There was a problem hiding this comment.
createGroup already returns the group no need for the get.
denismosolov
changed the title
denismosolov
force-pushed
the
group-add
branch
3 times, most recently
from
eb6bb4e to
2cb8f22
Compare
denismosolov
changed the title
denismosolov
added
3. to review
Signed-off-by: Denis Mosolov <denismosolov@gmail.com>
denismosolov
force-pushed
the
group-add
branch
from
78b48e4 to
0b18e2c
Compare
(ignore the xdebug warning, this is an issue on my setup) |
|
Too late to add this to NC14 IMO, so tagging this for 15. |
|
Thanks guys! |
#10334 (comment) -> @MorrisJobke @rullzer shall we revert this change? This is a new feature and we're past feature freeze. |
|
I'm fine with having this. Isolated new commands. |
|
@denismosolov In the future please wait for the approval for merge by others if it is stated like this in the comments. We are currently shortly before the RC 1 release and want to have this stable. |
|
For this one it's fine. |
|
Sorry for the accident. I didn't know what to do after getting 2 reviews. Just clicked the merge button because UI alowed me to do that. |
4 participants
Add options to create and remove groups via command line.
Requested in #8317
Syntax:
occ group:add groupidcreates a new group, groupid - the new groups nameocc group:delete groupidremoves group, cannot remove the group named 'admin'@tterranigma , @EpeR1 what do you think?
Signed-off-by: Denis Mosolov denismosolov@gmail.com