From fb96d432ee4f1fcf237f56a9efda881db8b34bf0 Mon Sep 17 00:00:00 2001
From: Wyatt Johnson <wyattjoh@gmail.com>
Date: Wed, 1 Mar 2017 13:51:32 -0700
Subject: [PATCH] Moved action creation upstream

---
 graph/mutators/action.js  | 2 +-
 graph/mutators/comment.js | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/graph/mutators/action.js b/graph/mutators/action.js
index 3bc20cba4..a572a641c 100644
--- a/graph/mutators/action.js
+++ b/graph/mutators/action.js
@@ -16,7 +16,7 @@ const createAction = ({user = {}}, {item_id, item_type, action_type, group_id, m
   return ActionsService.insertUserAction({
     item_id,
     item_type,
-    user_id: group_id === 'Matched suspect word filter' ? null : user.id,
+    user_id: user.id,
     group_id,
     action_type,
     metadata
diff --git a/graph/mutators/comment.js b/graph/mutators/comment.js
index 5112ba829..d608eee57 100644
--- a/graph/mutators/comment.js
+++ b/graph/mutators/comment.js
@@ -1,6 +1,7 @@
 const errors = require('../../errors');
 
 const AssetsService = require('../../services/assets');
+const ActionsService = require('../../services/actions');
 const CommentsService = require('../../services/comments');
 
 const Wordlist = require('../../services/wordlist');
@@ -146,10 +147,11 @@ const createPublicComment = (context, commentInput) => {
           // TODO: this is kind of fragile, we should refactor this to resolve
           // all these const's that we're using like 'COMMENTS', 'FLAG' to be
           // defined in a checkable schema.
-          return context.mutators.Action.create({
+          return ActionsService.insertUserAction({
             item_id: comment.id,
             item_type: 'COMMENTS',
             action_type: 'FLAG',
+            user_id: null,
             group_id: 'Matched suspect word filter',
             metadata: {}
           })