Merge pull request #149 from juvenn/master

修复 beforeSave 没有正确返回对象的 bug
leancloud · Apr 13, 2017 · f1c9219 · f1c9219
2 parents 1047caa + 4eab60c
commit f1c9219
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,4 +1,10 @@
 
+0.5.6 发布日期：2017-04-13
+----
+
+* 修复: 用户定义的 class hook 不需要返回对象，
+    而是由中间件来返回被修改的对象
+
 0.5.5 发布日期：2017-04-12
 ----
 

diff --git a/src/LeanCloud/Client.php b/src/LeanCloud/Client.php
@@ -23,7 +23,7 @@ class Client {
     /**
      * Client version
      */
-    const VERSION = '0.5.5';
+    const VERSION = '0.5.6';
 
     /**
      * API Endpoints for Regions

diff --git a/src/LeanCloud/Engine/LeanEngine.php b/src/LeanCloud/Engine/LeanEngine.php
@@ -522,9 +522,8 @@ private function dispatchHook($className, $hookName, $body) {
         } else if (strpos($hookName, "after") === 0) {
             $this->renderJSON(array("result" => "ok"));
         } else {
-            $outObj = $result;
             // Encode result object to type-less literal JSON
-            $this->renderJSON($outObj->toJSON());
+            $this->renderJSON($obj->toJSON());
         }
     }
 

diff --git a/test/engine/index.php b/test/engine/index.php
@@ -58,7 +58,6 @@
 
 Cloud::beforeSave("TestObject", function($obj, $user) {
     $obj->set("__testKey", 42);
-    return $obj;
 });
 
 Cloud::afterSave("TestObject", function($obj, $user) {