[stable23] Forbid tagging readonly files #44326
Conversation
![backportbot[bot]](https://github.com/avatars/in/60201?s=60&v=4){kind=small}
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
25 supports 7.4 Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
come-nc
changed the title
![github-advanced-security[bot]](https://github.com/avatars/in/57789?s=60&v=4){kind=small}
| @@ -60,10 +60,19 @@ | |||
| $tagMapper, | |||
| $userSession, | |||
| $groupManager, | |||
| function ($name) { | |||
| function ($name): bool { | |||
Check notice
Code scanning / Psalm
MissingClosureParamType Note
| $nodes = \OC::$server->getUserFolder()->getById((int)$name); | ||
| return !empty($nodes); | ||
| } | ||
| }, | ||
| function ($name): bool { |
Check notice
Code scanning / Psalm
MissingClosureParamType Note
| } | ||
| }, | ||
| function ($name): bool { | ||
| $nodes = \OC::$server->getUserFolder()->getById((int)$name); |
Check notice
Code scanning / Psalm
DeprecatedMethod Note
| } | ||
| }, | ||
| function ($name): bool { | ||
| $nodes = \OC::$server->getUserFolder()->getById((int)$name); |
Check notice
Code scanning / Psalm
PossiblyNullReference Note
23 still supports PHP 7.3 Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
|
apps/dav/tests/unit/SystemTag/SystemTagsObjectMappingCollectionTest.php
Outdated
Show resolved
Hide resolved
| @@ -77,7 +86,10 @@ | |||
| $tagMapper, | |||
| $userSession, | |||
| $groupManager, | |||
| $entityExistsFunction | |||
| $entityExistsFunction, | |||
| function ($name) { | |||
Check notice
Code scanning / Psalm
MissingClosureParamType Note
… 7.3 Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
come-nc
force-pushed
the
backport/44309/stable23
branch
from
355a6f1
to
a4473ce
Compare
Backport of PR #44309 / #44297