Draft API new topic

adaur · adaur · commit 9fb09b7be72e · 2016-04-07T21:18:47.000+08:00
diff --git a/featherbb/Controller/Api/Topic.php b/featherbb/Controller/Api/Topic.php
@@ -32,7 +32,7 @@ public function newTopic($req, $res, $args)
 
         $is_admmod = $this->model->checkPermissions($cur_posting, null, $args['id']);
 
-        if (is_bool($is_admmod)) {
+        if (!is_bool($is_admmod)) {
             return $is_admmod;
         }
 
@@ -78,5 +78,9 @@ public function newTopic($req, $res, $args)
 
             return Router::redirect(Router::pathFor('postApi', ['id' => $new['pid']]));
         }
+        else {
+            die(var_dump($errors));
+            return json_encode($errors, JSON_PRETTY_PRINT);
+        }
     }
 }
diff --git a/featherbb/Model/Api/Topic.php b/featherbb/Model/Api/Topic.php
@@ -11,6 +11,7 @@
 
 use FeatherBB\Core\Database as DB;
 use FeatherBB\Core\Error;
+use FeatherBB\Core\Track;
 use FeatherBB\Core\Utils;
 
 class Topic extends Api
@@ -110,6 +111,7 @@ public function check_errors_before_post($fid, $errors)
 
         // If it's a new topic
         if ($fid) {
+            // todo: can't get req_subject
             $subject = Utils::trim(Input::post('req_subject'));
 
             if (ForumSettings::get('o_censoring') == '1') {
@@ -307,9 +309,11 @@ public function insert_topic($post, $fid)
             ->set($topic['update']);
         $topic = $topic->save();
 
-        $this->search->update_search_index('post', $new['pid'], $post['message'], $post['subject']);
+        $search = new \FeatherBB\Core\Search();
 
-        Forum::update($fid);
+        $search->update_search_index('post', $new['pid'], $post['message'], $post['subject']);
+
+        \FeatherBB\Model\Forum::update($fid);
 
         return $new;
     }
diff --git a/featherbb/routes.php b/featherbb/routes.php
@@ -186,7 +186,7 @@
     Route::get('/user/{id:\d+}', '\FeatherBB\Controller\Api\User:display')->setName('userApi');
     Route::get('/forum/{id:\d+}', '\FeatherBB\Controller\Api\Forum:display')->setName('forumApi');
     Route::get('/topic/{id:\d+}', '\FeatherBB\Controller\Api\Topic:display')->setName('topicApi');
-    Route::post('/new/topic/forum-id/{id:\d+}', '\FeatherBB\Controller\Api\Topic:newTopic')->setName('newTopicApi');
+    Route::post('/topic/{id:\d+}', '\FeatherBB\Controller\Api\Topic:newTopic')->setName('newTopicApi');
     Route::get('/post/{id:\d+}', '\FeatherBB\Controller\Api\Post:display')->setName('postApi');
 })->add(new JsonHeader);
 

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public function newTopic($req, $res, $args)`
`32`	`32`
`33`	`33`	`$is_admmod = $this->model->checkPermissions($cur_posting, null, $args['id']);`
`34`	`34`
`35`		`- if (is_bool($is_admmod)) {`
	`35`	`+ if (!is_bool($is_admmod)) {`
`36`	`36`	`return $is_admmod;`
`37`	`37`	`}`
`38`	`38`
`@@ -78,5 +78,9 @@ public function newTopic($req, $res, $args)`
`78`	`78`
`79`	`79`	`return Router::redirect(Router::pathFor('postApi', ['id' => $new['pid']]));`
`80`	`80`	`}`
	`81`	`+ else {`
	`82`	`+ die(var_dump($errors));`
	`83`	`+ return json_encode($errors, JSON_PRETTY_PRINT);`
	`84`	`+ }`
`81`	`85`	`}`
`82`	`86`	`}`