Skip to content

Commit 007294d

Browse files
authored
Merge pull request #61687 from nextcloud/carl/node-properties
refactor(Node): Use new properties declaration in constructor
2 parents b993e52 + 51eecaf commit 007294d

1 file changed

Lines changed: 2 additions & 13 deletions

File tree

lib/private/Files/Node/Node.php

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -29,21 +29,12 @@
2929
// FIXME: this class really should be abstract (+1)
3030
class Node implements INode {
3131
/**
32-
* @var View $view
33-
*/
34-
protected $view;
35-
36-
protected IRootFolder $root;
37-
38-
/**
39-
* @param View $view
40-
* @param \OCP\Files\IRootFolder $root
4132
* @param string $path
4233
* @param FileInfo $fileInfo
4334
*/
4435
public function __construct(
45-
IRootFolder $root,
46-
$view,
36+
protected IRootFolder $root,
37+
protected View $view,
4738
protected $path,
4839
protected ?FileInfo $fileInfo = null,
4940
protected ?INode $parent = null,
@@ -52,8 +43,6 @@ public function __construct(
5243
if (Filesystem::normalizePath($view->getRoot()) !== '/') {
5344
throw new PreConditionNotMetException('The view passed to the node should not have any fake root set');
5445
}
55-
$this->view = $view;
56-
$this->root = $root;
5746
}
5847

5948
/**

0 commit comments

Comments
 (0)