Updating frontend to utilize new action delete endpoint.

client/coral-framework/actions/items.js

@@ -15,7 +15,7 @@ const getInit = (method, body) => {
   };
 
   const init = {method, headers};
-  if (method.toLowerCase() !== 'get') {
+  if (body) {
     init.body = JSON.stringify(body);
   }
 
@@ -23,6 +23,9 @@ const getInit = (method, body) => {
 };
 
 const responseHandler = response => {
+  if (response.status === 204) {
+    return;
+  }
   return response.ok ? response.json() : Promise.reject(`${response.status} ${response.statusText}`);
 };
 /**
@@ -241,14 +244,9 @@ export function postAction (item_id, action_type, user_id, item_type) {
 *
 */
 
-export function deleteAction (item_id, action_type, user_id, item_type) {
+export function deleteAction (action_id) {
   return () => {
-    const action = {
-      action_type,
-      user_id
-    };
-
-    return fetch(`/api/v1/${item_type}/${item_id}/actions`, getInit('DELETE', action))
+    return fetch(`/api/v1/actions/${action_id}`, {method: 'DELETE'})
       .then(responseHandler);
   };
 }

client/coral-plugin-flags/FlagButton.js

@@ -10,12 +10,13 @@ const FlagButton = ({flag, id, postAction, deleteAction, addItem, updateItem, ad
     if (!flagged) {
       postAction(id, 'flag', '123', 'comments')
         .then((action) => {
-          addItem({...action, current_user:true}, 'actions');
-          updateItem(action.item_id, action.action_type, action.id, 'comments');
+          let id = `${action.action_type}_${action.item_id}`;
+          addItem({id, current_user: action, count: flag ? flag.count + 1 : 1}, 'actions');
+          updateItem(action.item_id, action.action_type, id, 'comments');
         });
       addNotification('success', lang.t('flag-notif'));
     } else {
-      deleteAction(id, 'flag', '123', 'comments')
+      deleteAction(flagged.id)
         .then(() => {
           updateItem(id, 'flag', '', 'comments');
         });

client/coral-plugin-likes/LikeButton.js

@@ -10,11 +10,12 @@ const LikeButton = ({like, id, postAction, deleteAction, addItem, updateItem}) =
     if (!liked) {
       postAction(id, 'like', '123', 'comments')
         .then((action) => {
-          addItem({id: action.id, current_user:true, count: like ? like.count + 1 : 1}, 'actions');
-          updateItem(action.item_id, action.action_type, action.id, 'comments');
+          let id = `${action.action_type}_${action.item_id}`;
+          addItem({id, current_user: action, count: like ? like.count + 1 : 1}, 'actions');
+          updateItem(action.item_id, action.action_type, id, 'comments');
         });
     } else {
-      deleteAction(id, 'like', '123', 'comments')
+      deleteAction(liked.id)
         .then(() => {
           updateItem(like.id, 'count', like.count - 1, 'actions');
           updateItem(like.id, 'current_user', false, 'actions');

init.js

@@ -1,5 +1,5 @@
 const Setting = require('./models/setting');
-const wordlist = require('../services/wordlist');
+const wordlist = require('./services/wordlist');
 
 module.exports = () => Promise.all([
 

routes/api/index.js

@@ -9,5 +9,6 @@ router.use('/queue', require('./queue'));
 router.use('/settings', require('./settings'));
 router.use('/stream', require('./stream'));
 router.use('/user', require('./user'));
+router.use('/actions', require('./actions'));
 
 module.exports = router;

tests/client/coral-framework/store/itemActions.spec.js

@@ -173,7 +173,7 @@ describe('itemActions', () => {
       fetchMock.delete('*', {});
       return actions.deleteAction('abc', 'flag', '123', 'comments')(store.dispatch)
         .then(response => {
-          expect(fetchMock.calls().matched[0][0]).to.equal('/api/v1/comments/abc/actions');
+          expect(fetchMock.calls().matched[0][0]).to.equal('/api/v1/actions/abc');
           expect(response).to.deep.equal({});
         });
     });