From 4e7de7a687bf0b84a3e869bc228dbef65cbc136c Mon Sep 17 00:00:00 2001
From: hylom <hylom@users.sourceforge.jp>
Date: Thu, 25 May 2017 21:19:11 +0900
Subject: [PATCH]  * Web::Controller::API::Submission: fix error messages

---
 src/newslash_web/lib/Newslash/Web/Controller/API/Submission.pm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/newslash_web/lib/Newslash/Web/Controller/API/Submission.pm b/src/newslash_web/lib/Newslash/Web/Controller/API/Submission.pm
index 839b77a1..a021a800 100644
--- a/src/newslash_web/lib/Newslash/Web/Controller/API/Submission.pm
+++ b/src/newslash_web/lib/Newslash/Web/Controller/API/Submission.pm
@@ -15,7 +15,7 @@ sub get {
     my $submissions = $c->model('submissions');
     my $submission = $submissions->select(submission_id => $subid);
     if (!$submission) {
-        $c->render(json => { err => 1, message => "submission_not_found" });
+        $c->render(json => { err => 1, message => "not_found" });
         $c->rendered(404);
         return;
     }
@@ -33,7 +33,7 @@ sub post {
 
     # check permission
     if (!$user->{permissions}->{submission}) {
-        $c->render(json => { err => 1, message => "not_allowed" });
+        $c->render(json => { err => 1, reason => "not_allowed", message => "not_allowed" });
         $c->rendered(403);
         return;
     }
@@ -72,10 +72,11 @@ sub post {
         else {
             #if ($submissions->last_errorno && $submissions->last_errorno == 1062) { #ER_DUP_ENTRY
             if (defined $sub_id) {
+                $c->render(json => { err => 1, reason => "duplicated_post", message => "duplicated_post" });
                 $c->rendered(409);
                 return;
             }
-            $c->render(json => { err => 1, message => $submissions->last_error });
+            $c->render(json => { err => 1, reason => "server_error", message => $submissions->last_error });
             $c->rendered(500);
         }
     }
-- 
2.11.0