diff --git a/lib/classes/JsonApi/Routes/Blubber/CommentsCreate.php b/lib/classes/JsonApi/Routes/Blubber/CommentsCreate.php
index a1402b4b98a3a9376626bc8cceffe651993214b2..3548645bf35748a6cce6454cb87818057c998a60 100644
--- a/lib/classes/JsonApi/Routes/Blubber/CommentsCreate.php
+++ b/lib/classes/JsonApi/Routes/Blubber/CommentsCreate.php
@@ -46,7 +46,7 @@ class CommentsCreate extends JsonApiController
return $this->getCreatedResponse($comment);
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
if (empty(self::arrayGet($json, 'data.attributes.content'))) {
return 'Comment should not be empty.';
diff --git a/lib/classes/JsonApi/Routes/Blubber/CommentsUpdate.php b/lib/classes/JsonApi/Routes/Blubber/CommentsUpdate.php
index a3ef571cf0858873e80bbec1cf6cc28ea83451eb..8f83246b3f1b86081b70d6c6db484725538e5ed4 100644
--- a/lib/classes/JsonApi/Routes/Blubber/CommentsUpdate.php
+++ b/lib/classes/JsonApi/Routes/Blubber/CommentsUpdate.php
@@ -44,7 +44,7 @@ class CommentsUpdate extends JsonApiController
return $this->getCodeResponse(204);
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
if (empty(self::arrayGet($json, 'data.attributes.content'))) {
return 'Comment should not be empty.';
diff --git a/lib/classes/JsonApi/Routes/Blubber/ThreadsCreate.php b/lib/classes/JsonApi/Routes/Blubber/ThreadsCreate.php
index d503157b93516b55e4c7d93ba3c018e0e12bb2d1..b5bc9435e301e44b03cb3335d24568313f7fccd7 100644
--- a/lib/classes/JsonApi/Routes/Blubber/ThreadsCreate.php
+++ b/lib/classes/JsonApi/Routes/Blubber/ThreadsCreate.php
@@ -51,7 +51,7 @@ class ThreadsCreate extends JsonApiController
return $this->getCreatedResponse($thread);
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
if (Schema::TYPE !== self::arrayGet($json, 'data.type')) {
return 'Missing or wrong type.';
diff --git a/lib/classes/JsonApi/Routes/ConfigValues/ConfigValuesUpdate.php b/lib/classes/JsonApi/Routes/ConfigValues/ConfigValuesUpdate.php
index 97411404a6b0e271abadcf425b218a6afa4716f0..cf46134eb00085ba414bf6f51ae460bffbf331a2 100644
--- a/lib/classes/JsonApi/Routes/ConfigValues/ConfigValuesUpdate.php
+++ b/lib/classes/JsonApi/Routes/ConfigValues/ConfigValuesUpdate.php
@@ -41,7 +41,7 @@ class ConfigValuesUpdate extends JsonApiController
return $this->getContentResponse($resource);
}
- protected function validateResourceDocument($json, $resource)
+ protected function validateResourceDocument($json, $data)
{
if (!self::arrayHas($json, 'data')) {
return 'Missing `data` member at document´s top level.';
@@ -55,7 +55,7 @@ class ConfigValuesUpdate extends JsonApiController
return 'Document must have an `id`.';
}
- if (self::arrayGet($json, 'data.id') !== $this->generateId($resource)) {
+ if (self::arrayGet($json, 'data.id') !== $this->generateId($data)) {
return 'Mismatch between URI parameter and document `id`.';
}
diff --git a/lib/classes/JsonApi/Routes/CourseMemberships/CourseMembershipsUpdate.php b/lib/classes/JsonApi/Routes/CourseMemberships/CourseMembershipsUpdate.php
index 6bf71eadd06a6c40700d1e8d238ba7b5ecdd6845..4d0218e93fde1501102ee3323efefa4c9aa29b5c 100644
--- a/lib/classes/JsonApi/Routes/CourseMemberships/CourseMembershipsUpdate.php
+++ b/lib/classes/JsonApi/Routes/CourseMemberships/CourseMembershipsUpdate.php
@@ -70,7 +70,7 @@ class CourseMembershipsUpdate extends JsonApiController
return $membership;
}
- protected function validateResourceDocument($json, \CourseMember $membership)
+ protected function validateResourceDocument($json, $data)
{
if (CourseMemberSchema::TYPE !== self::arrayGet($json, 'data.type')) {
return 'Missing or wrong type.';
@@ -93,7 +93,11 @@ class CourseMembershipsUpdate extends JsonApiController
return 'Attribute `visible` must be either `yes` or `no`';
}
- if ('no' === $visible && in_array($membership->status, ['tutor', 'dozent'])) {
+ if (
+ 'no' === $visible
+ && $data instanceof \CourseMember
+ && in_array($data->status, ['tutor', 'dozent'])
+ ) {
return 'Users of status `tutor` or `dozent` must remain visible.';
}
}
diff --git a/lib/classes/JsonApi/Routes/Courseware/CoursewareInstancesUpdate.php b/lib/classes/JsonApi/Routes/Courseware/CoursewareInstancesUpdate.php
index fff4608cdebffa3453d6618ee3fe9c419725ced4..c5ef250ba5245c6a9e4a6a977c5af7777c4119b0 100644
--- a/lib/classes/JsonApi/Routes/Courseware/CoursewareInstancesUpdate.php
+++ b/lib/classes/JsonApi/Routes/Courseware/CoursewareInstancesUpdate.php
@@ -40,7 +40,7 @@ class CoursewareInstancesUpdate extends JsonApiController
/**
* @SuppressWarnings(PHPMD.UnusedFormalParameters)
*/
- protected function validateResourceDocument($json, $resource)
+ protected function validateResourceDocument($json, $data)
{
if (!self::arrayHas($json, 'data')) {
return 'Missing `data` member at document´s top level.';
@@ -61,7 +61,7 @@ class CoursewareInstancesUpdate extends JsonApiController
}
$blockTypes = array_map(function ($blockType) {
return $blockType::getType();
- }, $resource->getBlockTypes());
+ }, $data->getBlockTypes());
foreach ($favoriteBlockTypes as $favoriteBlockType) {
if (!in_array($favoriteBlockType, $blockTypes)) {
return 'Attribute `favorite-block-types` contains an invalid block type.';
@@ -79,7 +79,7 @@ class CoursewareInstancesUpdate extends JsonApiController
if (!is_string($editingPermissionLevel)) {
return 'Attribute `editing-permission-level` must be a string.';
}
- if (!$resource->isValidEditingPermissionLevel($editingPermissionLevel)) {
+ if (!$data->isValidEditingPermissionLevel($editingPermissionLevel)) {
return 'Attribute `editing-permission-level` contains an invalid value.';
}
}
@@ -87,7 +87,7 @@ class CoursewareInstancesUpdate extends JsonApiController
if (self::arrayHas($json, 'data.attributes.certificate-settings')) {
$certificateSettings = self::arrayGet($json, 'data.attributes.certificate-settings');
- if (!$resource->isValidCertificateSettings($certificateSettings)) {
+ if (!$data->isValidCertificateSettings($certificateSettings)) {
return 'Attribute `certificate-settings` contains an invalid value.';
}
}
@@ -95,7 +95,7 @@ class CoursewareInstancesUpdate extends JsonApiController
if (self::arrayHas($json, 'data.attributes.reminder-settings')) {
$reminderSettings = self::arrayGet($json, 'data.attributes.reminder-settings');
- if (!$resource->isValidReminderSettings($reminderSettings)) {
+ if (!$data->isValidReminderSettings($reminderSettings)) {
return 'Attribute `reminder-settings` contains an invalid value.';
}
}
@@ -103,7 +103,7 @@ class CoursewareInstancesUpdate extends JsonApiController
if (self::arrayHas($json, 'data.attributes.reset-progress-settings')) {
$resetProgressSettings = self::arrayGet($json, 'data.attributes.reset-progress-settings');
- if (!$resource->isValidResetProgressSettings($resetProgressSettings)) {
+ if (!$data->isValidResetProgressSettings($resetProgressSettings)) {
return 'Attribute `reset-progress-settings` contains an invalid value.';
}
}
diff --git a/lib/classes/JsonApi/Routes/Courseware/StructuralElementsUpdate.php b/lib/classes/JsonApi/Routes/Courseware/StructuralElementsUpdate.php
index 33ba9a3f9bc69907d21a261e9272af03906c2931..febe3cc8480b4db3c1f94f8efdb967483ec5d54f 100644
--- a/lib/classes/JsonApi/Routes/Courseware/StructuralElementsUpdate.php
+++ b/lib/classes/JsonApi/Routes/Courseware/StructuralElementsUpdate.php
@@ -39,7 +39,7 @@ class StructuralElementsUpdate extends JsonApiController
/**
* @SuppressWarnings(PHPMD.UnusedFormalParameters)
*/
- protected function validateResourceDocument($json, $resource)
+ protected function validateResourceDocument($json, $data)
{
if (!self::arrayHas($json, 'data')) {
return 'Missing `data` member at document´s top level.';
@@ -55,7 +55,7 @@ class StructuralElementsUpdate extends JsonApiController
if (self::arrayHas($json, 'data.relationships.parent')) {
// Sonderfall: Wurzel hat kein parent und kann auch nicht verändert werden
- if ($resource->isRootNode()) {
+ if ($data->isRootNode()) {
if (null !== self::arrayGet($json, 'data.relationships.parent.data')) {
return 'Cannot modify `parent` of a root node.';
}
@@ -70,7 +70,7 @@ class StructuralElementsUpdate extends JsonApiController
// keine Schleifen
if (
in_array(
- $resource->id,
+ $data->id,
array_merge(
[$parent->id],
array_map(function ($ancestor) {
diff --git a/lib/classes/JsonApi/Routes/Courseware/UnitsUpdate.php b/lib/classes/JsonApi/Routes/Courseware/UnitsUpdate.php
index 762fb0437e81a86eda650ea66ad0c721073642cd..75956c4c2ccb01ee26372015e6c917b3688230c9 100644
--- a/lib/classes/JsonApi/Routes/Courseware/UnitsUpdate.php
+++ b/lib/classes/JsonApi/Routes/Courseware/UnitsUpdate.php
@@ -43,7 +43,7 @@ class UnitsUpdate extends JsonApiController
/**
* @SuppressWarnings(PHPMD.UnusedFormalParameters)
*/
- protected function validateResourceDocument($json, $resource)
+ protected function validateResourceDocument($json, $data)
{
if (!self::arrayHas($json, 'data')) {
return 'Missing `data` member at document´s top level.';
diff --git a/lib/classes/JsonApi/Routes/Files/FileRefsUpdate.php b/lib/classes/JsonApi/Routes/Files/FileRefsUpdate.php
index 27ddabab7b1f7653a438574ea8ad12ff0642a076..79000cc7445c919defec21820232c80a12c02cb8 100644
--- a/lib/classes/JsonApi/Routes/Files/FileRefsUpdate.php
+++ b/lib/classes/JsonApi/Routes/Files/FileRefsUpdate.php
@@ -66,9 +66,9 @@ class FileRefsUpdate extends JsonApiController
}
}
- protected function validateResourceDocument($json, $fileRef)
+ protected function validateResourceDocument($json, $data)
{
- if ($err = $this->validateFileRefResourceObject($json, $fileRef)) {
+ if ($err = $this->validateFileRefResourceObject($json, $data)) {
return $err;
}
}
diff --git a/lib/classes/JsonApi/Routes/Files/FoldersUpdate.php b/lib/classes/JsonApi/Routes/Files/FoldersUpdate.php
index afb43ace897a44d68ba1c9c9946e9be80a4f6e3b..f089c3d9f2a62e4600f79074fa641b6ca5af9482 100644
--- a/lib/classes/JsonApi/Routes/Files/FoldersUpdate.php
+++ b/lib/classes/JsonApi/Routes/Files/FoldersUpdate.php
@@ -66,9 +66,9 @@ class FoldersUpdate extends JsonApiController
}
}
- protected function validateResourceDocument($json, \FolderType $folder)
+ protected function validateResourceDocument($json, $data)
{
- if ($err = $this->validateFolderResourceObject($json, $folder)) {
+ if ($err = $this->validateFolderResourceObject($json, $data)) {
return $err;
}
}
diff --git a/lib/classes/JsonApi/Routes/Forum/AbstractEntriesCreate.php b/lib/classes/JsonApi/Routes/Forum/AbstractEntriesCreate.php
index 204414b711e463d8e1dd79d590bcc72a8918b966..734b9d8a6cff69d8c55631b0bef31da82b513b39 100644
--- a/lib/classes/JsonApi/Routes/Forum/AbstractEntriesCreate.php
+++ b/lib/classes/JsonApi/Routes/Forum/AbstractEntriesCreate.php
@@ -11,7 +11,7 @@ abstract class AbstractEntriesCreate extends JsonApiController
{
use ValidationTrait;
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
$content = self::arrayHas($json, 'data.attributes.title');
if (empty($content)) {
diff --git a/lib/classes/JsonApi/Routes/Forum/ForumCategoriesCreate.php b/lib/classes/JsonApi/Routes/Forum/ForumCategoriesCreate.php
index bbb492e5231769ebae508eb4bf457b322a0f5562..280e437e0238bcbde620c1517cde568a67b135e5 100644
--- a/lib/classes/JsonApi/Routes/Forum/ForumCategoriesCreate.php
+++ b/lib/classes/JsonApi/Routes/Forum/ForumCategoriesCreate.php
@@ -51,7 +51,7 @@ class ForumCategoriesCreate extends JsonApiController
return $category;
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
$title = self::arrayGet($json, 'data.attributes.title', '');
if (empty($title)) {
diff --git a/lib/classes/JsonApi/Routes/Forum/ForumCategoriesUpdate.php b/lib/classes/JsonApi/Routes/Forum/ForumCategoriesUpdate.php
index 961f17552419051b346b00c1c63299866ef3770e..e09339efde261a4a29054a7e8f883ac88ecbd20d 100644
--- a/lib/classes/JsonApi/Routes/Forum/ForumCategoriesUpdate.php
+++ b/lib/classes/JsonApi/Routes/Forum/ForumCategoriesUpdate.php
@@ -53,7 +53,7 @@ class ForumCategoriesUpdate extends JsonApiController
}
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
$title = self::arrayGet($json, 'data.attributes.title', '');
if (empty($title)) {
diff --git a/lib/classes/JsonApi/Routes/Forum/ForumEntriesUpdate.php b/lib/classes/JsonApi/Routes/Forum/ForumEntriesUpdate.php
index 1165ff2d84636b1557d68550267e86d15e5f99f1..21eeba4c0dfb0ae2dd8c5b04edaadbe105c96ff1 100644
--- a/lib/classes/JsonApi/Routes/Forum/ForumEntriesUpdate.php
+++ b/lib/classes/JsonApi/Routes/Forum/ForumEntriesUpdate.php
@@ -60,7 +60,7 @@ class ForumEntriesUpdate extends JsonApiController
return $entry;
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
$title = self::arrayGet($json, 'data.attributes.title');
$content = self::arrayGet($json, 'data.attributes.content');
diff --git a/lib/classes/JsonApi/Routes/Messages/MessageUpdate.php b/lib/classes/JsonApi/Routes/Messages/MessageUpdate.php
index 5dcde5b455834990159cb61472ed40d3f6c4994b..add12e4633bbf901ac1cddb140df5e4c87f3722d 100644
--- a/lib/classes/JsonApi/Routes/Messages/MessageUpdate.php
+++ b/lib/classes/JsonApi/Routes/Messages/MessageUpdate.php
@@ -37,7 +37,7 @@ class MessageUpdate extends JsonApiController
return $this->getContentResponse($message);
}
- protected function validateResourceDocument($json)
+ protected function validateResourceDocument($json, $data)
{
if (!self::arrayHas($json, 'data')) {
return 'Missing `data` member at message´s top level.';
diff --git a/lib/classes/JsonApi/Routes/News/NewsUpdate.php b/lib/classes/JsonApi/Routes/News/NewsUpdate.php
index 269c4501e01212e61a6faaa26ea24014ffbfc754..6b26c43f8b83bbc7dc9a58a5f9132859460b847a 100644
--- a/lib/classes/JsonApi/Routes/News/NewsUpdate.php
+++ b/lib/classes/JsonApi/Routes/News/NewsUpdate.php
@@ -77,7 +77,7 @@ class NewsUpdate extends JsonApiController
return $news;
}
- protected function validateResourceDocument($json, \StudipNews $news)
+ protected function validateResourceDocument($json, $data)
{
if (NewsSchema::TYPE !== self::arrayGet($json, 'data.type')) {
return 'Missing or wrong type.';