diff options
4 files changed, 11 insertions, 5 deletions
diff --git a/apps/dav/lib/CalDAV/Activity/Provider/Base.php b/apps/dav/lib/CalDAV/Activity/Provider/Base.php index 99ad903f247..ee9c6716130 100644 --- a/apps/dav/lib/CalDAV/Activity/Provider/Base.php +++ b/apps/dav/lib/CalDAV/Activity/Provider/Base.php @@ -160,7 +160,7 @@ abstract class Base implements IProvider { } return [ - 'type' => 'group', + 'type' => 'user-group', 'id' => $gid, 'name' => $this->groupDisplayNames[$gid], ]; diff --git a/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php b/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php index 37a56f88042..9c4f5eeb47c 100644 --- a/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php +++ b/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php @@ -182,7 +182,7 @@ class BaseTest extends TestCase { */ public function testGenerateGroupParameter($gid) { $this->assertEquals([ - 'type' => 'group', + 'type' => 'user-group', 'id' => $gid, 'name' => $gid, ], $this->invokePrivate($this->provider, 'generateGroupParameter', [$gid])); diff --git a/apps/files_sharing/lib/Activity/Providers/Groups.php b/apps/files_sharing/lib/Activity/Providers/Groups.php index 9a8f7164c55..10f52882aeb 100644 --- a/apps/files_sharing/lib/Activity/Providers/Groups.php +++ b/apps/files_sharing/lib/Activity/Providers/Groups.php @@ -150,7 +150,7 @@ class Groups extends Base { } return [ - 'type' => 'group', + 'type' => 'user-group', 'id' => $gid, 'name' => $this->groupDisplayNames[$gid], ]; diff --git a/lib/private/RichObjectStrings/Validator.php b/lib/private/RichObjectStrings/Validator.php index 55a50fc2a48..29980885cf8 100644 --- a/lib/private/RichObjectStrings/Validator.php +++ b/lib/private/RichObjectStrings/Validator.php @@ -71,11 +71,17 @@ class Validator implements IValidator { foreach ($matches[1] as $parameter) { if (!isset($parameters[$parameter])) { throw new InvalidObjectExeption('Parameter is undefined'); - } else { - $this->validateParameter($parameters[$parameter]); } } } + + foreach ($parameters as $parameter) { + if (!\is_array($parameter)) { + throw new InvalidObjectExeption('Parameter is malformed'); + } + + $this->validateParameter($parameter); + } } /** |