Skip to content

Commit 2ca49f8

Browse files
authored
Merge pull request #6102 from nextcloud/enh/noid/fix-cmd-options
DockerActionManager: fix setting CMD options for collabora
2 parents 2942af2 + de053c9 commit 2ca49f8

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

php/src/Data/ConfigurationManager.php

-4
Original file line numberDiff line numberDiff line change
@@ -976,10 +976,6 @@ public function GetAdditionalCollaboraOptions() : string {
976976
return $config['collabora_additional_options'];
977977
}
978978

979-
public function GetAdditionalCollaboraOptionsArray() : array {
980-
return explode(' ', $this->GetAdditionalCollaboraOptions());
981-
}
982-
983979
public function DeleteAdditionalCollaboraOptions() : void {
984980
$config = $this->GetConfig();
985981
$config['collabora_additional_options'] = '';

php/src/Docker/DockerActionManager.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -581,7 +581,7 @@ public function CreateContainer(Container $container) : void {
581581
// Special things for the collabora container which should not be exposed in the containers.json
582582
} elseif ($container->GetIdentifier() === 'nextcloud-aio-collabora') {
583583
if ($this->configurationManager->GetAdditionalCollaboraOptions() !== '') {
584-
$requestBody['HostConfig']['Config']['Cmd'] = $this->configurationManager->GetAdditionalCollaboraOptionsArray();
584+
$requestBody['Cmd'] = [$this->configurationManager->GetAdditionalCollaboraOptions()];
585585
}
586586
}
587587

0 commit comments

Comments
 (0)