diff options
| -rw-r--r-- | core/Command/Group/Add.php | 5 | ||||
| -rw-r--r-- | lib/private/Group/Database.php | 6 |
2 files changed, 9 insertions, 2 deletions
diff --git a/core/Command/Group/Add.php b/core/Command/Group/Add.php index eb2ecd0c533..f2ee6195a44 100644 --- a/core/Command/Group/Add.php +++ b/core/Command/Group/Add.php @@ -70,8 +70,9 @@ class Add extends Base { $group = $this->groupManager->createGroup($gid); $output->writeln('Created group "' . $group->getGID() . '"'); - if ($input->hasOption('display-name')) { - $group->setDisplayName($input->getOption('display-name')); + $displayName = trim((string) $input->getOption('display-name')); + if ($displayName !== '') { + $group->setDisplayName($displayName); } } } diff --git a/lib/private/Group/Database.php b/lib/private/Group/Database.php index 21febce8e66..a616146db6d 100644 --- a/lib/private/Group/Database.php +++ b/lib/private/Group/Database.php @@ -109,6 +109,7 @@ class Database extends ABackend $builder = $this->dbConn->getQueryBuilder(); $result = $builder->insert('groups') ->setValue('gid', $builder->createNamedParameter($gid)) + ->setValue('displayname', $builder->createNamedParameter($gid)) ->execute(); } catch(UniqueConstraintViolationException $e) { $result = 0; @@ -451,6 +452,11 @@ class Database extends ABackend $this->fixDI(); + $displayName = trim($displayName); + if ($displayName === '') { + $displayName = $gid; + } + $query = $this->dbConn->getQueryBuilder(); $query->update('groups') ->set('displayname', $query->createNamedParameter($displayName)) |