Skip to content

Commit

Permalink
[BUGFIX] Make WebserverType an optional argument
Browse files Browse the repository at this point in the history
There is no downside in making the argument optional.
Although the API is marked as internal, it is in use
in third party code, which would break within a bugfix release.

Logically the API makes total sense with the argument
being optional. If the webserver type is not known or not
provided, no additional file is created.

Resolves: #101868
Releases: main, 12.4
Change-Id: Ie559b8685fc582681befe09ff769724f13639862
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80894
Reviewed-by: Stefan Bürk <stefan@buerk.tech>
Tested-by: Stefan Bürk <stefan@buerk.tech>
Tested-by: core-ci <typo3@b13.com>
  • Loading branch information
helhum authored and sbuerk committed Sep 7, 2023
1 parent c95ec9e commit 6d7483a
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -31,7 +31,7 @@ class DefaultFactory
*
* @return StructureFacadeInterface
*/
public function getStructure(WebserverType $webserverType)
public function getStructure(WebserverType $webserverType = WebserverType::Other)
{
$rootNode = new RootNode($this->getDefaultStructureDefinition($webserverType), null);
return new StructureFacade($rootNode);
Expand Down

0 comments on commit 6d7483a

Please sign in to comment.