Skip to content

Commit 16bdb01

Browse files
committed
bug #15 Added missing return-statement in DefaultConfiguration. (daFish)
This PR was merged into the 1.0.x-dev branch. Discussion ---------- Added missing return-statement in DefaultConfiguration. This adds `return $this` to the `fixPermissionsWith*`-methods which were missing previously. Commits ------- 5d6c0ca Added missing return-statement in DefaultConfiguration.
2 parents c66db26 + 5d6c0ca commit 16bdb01

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

src/Configuration/DefaultConfiguration.php

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -289,24 +289,32 @@ public function fixPermissionsWithChmod(string $mode = '0777') : self
289289
{
290290
$this->permissionMethod = 'chmod';
291291
$this->permissionMode = $mode;
292+
293+
return $this;
292294
}
293295

294296
public function fixPermissionsWithChown(string $webServerUser) : self
295297
{
296298
$this->permissionMethod = 'chown';
297299
$this->permissionUser = $webServerUser;
300+
301+
return $this;
298302
}
299303

300304
public function fixPermissionsWithChgrp(string $webServerGroup) : self
301305
{
302306
$this->permissionMethod = 'chgrp';
303307
$this->permissionGroup = $webServerGroup;
308+
309+
return $this;
304310
}
305311

306312
public function fixPermissionsWithAcl(string $webServerUser) : self
307313
{
308314
$this->permissionMethod = 'acl';
309315
$this->permissionUser = $webServerUser;
316+
317+
return $this;
310318
}
311319

312320
// Relative to the project root directory

0 commit comments

Comments
 (0)