Merge pull request otale#243 from otale/develop

☔️ fix article delete fail [bug]
wangq125 · Sep 19, 2017 · e6b0f48 · e6b0f48
2 parents d79f957 + cc6182e
commit e6b0f48
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/...ale/exception/GolbalExceptionResolve.java → ...ale/exception/GolbalExceptionHandler.java b/...ale/exception/GolbalExceptionResolve.java → ...ale/exception/GolbalExceptionHandler.java
@@ -1,8 +1,8 @@
 package com.tale.exception;
 
-import com.blade.exception.ExceptionResolve;
 import com.blade.ioc.annotation.Bean;
-import com.blade.mvc.hook.Signature;
+import com.blade.mvc.WebContext;
+import com.blade.mvc.handler.DefaultExceptionHandler;
 import com.blade.mvc.ui.RestResponse;
 import com.blade.validator.exception.ValidateException;
 import lombok.extern.slf4j.Slf4j;
@@ -14,16 +14,17 @@
  */
 @Slf4j
 @Bean
-public class GolbalExceptionResolve implements ExceptionResolve {
+public class GolbalExceptionHandler extends DefaultExceptionHandler {
 
     @Override
-    public boolean handle(Exception e, Signature signature) {
+    public void handle(Exception e) {
         if (e instanceof ValidateException) {
             ValidateException validateException = (ValidateException) e;
-            String msg = validateException.getErrMsg();
-            signature.response().json(RestResponse.fail(msg));
-            return false;
+            String            msg               = validateException.getErrMsg();
+            WebContext.response().json(RestResponse.fail(msg));
+        } else {
+            super.handle(e);
         }
-        return true;
     }
+
 }
diff --git a/src/main/java/com/tale/service/ContentsService.java b/src/main/java/com/tale/service/ContentsService.java
@@ -125,11 +125,10 @@ public void updateArticle(Contents contents) {
      */
     public void delete(int cid) {
         Optional<Contents> contents = this.getContents(cid + "");
-        contents.ifPresent(content -> Base.atomic(() -> {
+        contents.ifPresent(content -> {
             new Contents().delete(cid);
             new Relationships().delete("cid", cid);
-            return true;
-        }));
+        });
     }
 
     /**