QE-518 dev: isValidPatch op handler languageSettings (nothing to chec…

…k) (#3623)
LimeSurvey · Nov 20, 2023 · 4e6034b · 4e6034b
1 parent 3133716
commit 4e6034b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/application/libraries/Api/Command/V1/SurveyPatch/OpHandlerLanguageSettingsUpdate.php b/application/libraries/Api/Command/V1/SurveyPatch/OpHandlerLanguageSettingsUpdate.php
@@ -56,7 +56,7 @@ public function canHandle(OpInterface $op): bool
      *          "entity": "languageSetting",
      *          "op": "update",
      *          "id": {
-     *              "sid": 123456,
+     *              "sid": 123456,  //todo do we still need the sid here?
      *              "language": "de"
      *          },
      *          "props": {
@@ -72,7 +72,7 @@ public function canHandle(OpInterface $op): bool
      *          "entity": "languageSetting",
      *          "op": "update",
      *          "id": {
-     *              "sid": 123456
+     *              "sid": 123456  //todo do we still need the sid here?
      *          },
      *          "props": {
      *              "de": {
@@ -153,7 +153,7 @@ private function getTransformedProps(OpInterface $op, ?array $props)
      */
     public function isValidPatch(OpInterface $op): bool
     {
-        // TODO: Implement isValidPatch() method.
+        // getTransformedProps will throw an exception if the patch is not valid
         return true;
     }
 }