refactor(naming): cms_thread -> thread, thread -> thread_item (#350)

Author: mydearxym

cover/excoveralls.json

@@ -28,7 +28,6 @@ defmodule GroupherServer.CMS.Delegate.Helper do
         %User{} = user,
         type \\ :article
       ) do
-    IO.inspect("hello?")
     supported_emotions = get_supported_mentions(type)
 
     new_entries =

lib/groupher_server/cms/delegates/helper.ex

@@ -55,7 +55,6 @@ defmodule GroupherServerWeb.Resolvers.CMS do
   end
 
   def paged_articles(_root, ~m(thread filter)a, %{context: %{cur_user: user}}) do
-    IO.inspect(filter, label: "the filter")
     CMS.paged_articles(thread, filter, user)
   end
 

lib/groupher_server_web/resolvers/cms_resolver.ex

@@ -36,7 +36,7 @@ defmodule GroupherServerWeb.Schema.Helper.Queries do
       @desc unquote("get paged #{action}ed users of an article")
       field unquote(:"#{action}ed_users"), :paged_users do
         arg(:id, non_null(:id))
-        arg(:thread, :cms_thread, default_value: :post)
+        arg(:thread, :thread, default_value: :post)
         arg(:filter, non_null(:paged_filter))
 
         middleware(M.PageSizeProof)

lib/groupher_server_web/schema/Helper/queries.ex

@@ -72,7 +72,7 @@ defmodule GroupherServerWeb.Schema.Account.Mutations do
     field :add_to_collect, :collect_folder do
       arg(:article_id, non_null(:id))
       arg(:folder_id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
 
       middleware(M.Authorize, :login)
       resolve(&R.Accounts.add_to_collect/3)
@@ -82,7 +82,7 @@ defmodule GroupherServerWeb.Schema.Account.Mutations do
     field :remove_from_collect, :collect_folder do
       arg(:article_id, non_null(:id))
       arg(:folder_id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
 
       middleware(M.Authorize, :login)
       resolve(&R.Accounts.remove_from_collect/3)

lib/groupher_server_web/schema/account/account_mutations.ex

@@ -66,7 +66,7 @@ defmodule GroupherServerWeb.Schema.CMS.Misc do
     value(:radar)
   end
 
-  enum :cms_thread do
+  enum :thread do
     value(:post)
     value(:job)
     value(:user)
@@ -251,19 +251,19 @@ defmodule GroupherServerWeb.Schema.CMS.Misc do
 
   @desc "common filter for upvoted articles"
   input_object :upvoted_articles_filter do
-    field(:thread, :cms_thread)
+    field(:thread, :thread)
     pagination_args()
   end
 
   @desc "common filter for collect folders"
   input_object :collect_folders_filter do
-    field(:thread, :cms_thread)
+    field(:thread, :thread)
     pagination_args()
   end
 
   @desc "common filter for collect articles"
   input_object :collected_articles_filter do
-    field(:thread, :cms_thread)
+    field(:thread, :thread)
     pagination_args()
   end
 

lib/groupher_server_web/schema/cms/cms_misc.ex

@@ -105,7 +105,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     field :partial_tags, list_of(:tag) do
       arg(:community_id, :id)
       arg(:community, :string)
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:all, :boolean, default_value: false)
 
       resolve(&R.CMS.get_tags/3)
@@ -115,7 +115,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     field :paged_article_comments, :paged_article_comments do
       arg(:id, non_null(:id))
       arg(:mode, :article_comments_mode, default_value: :replies)
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:filter, :comments_filter)
 
       middleware(M.PageSizeProof)
@@ -125,7 +125,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     @desc "get paged article comments participators"
     field :paged_article_comments_participators, :paged_users do
       arg(:id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:filter, :paged_filter)
 
       middleware(M.PageSizeProof)
@@ -144,7 +144,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     @desc "get paged comments"
     field :paged_comments, :paged_comments do
       arg(:id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:filter, :comments_filter)
 
       middleware(M.PageSizeProof)
@@ -154,7 +154,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     @desc "get paged comments participators"
     field :paged_comments_participators, :paged_users do
       arg(:id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:filter, :paged_filter)
 
       middleware(M.PageSizeProof)
@@ -165,7 +165,7 @@ defmodule GroupherServerWeb.Schema.CMS.Queries do
     # TODO: remove
     field :comments, :paged_comments do
       arg(:id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:filter, :comments_filter)
 
       middleware(M.PageSizeProof)

lib/groupher_server_web/schema/cms/cms_queries.ex

@@ -202,7 +202,7 @@ defmodule GroupherServerWeb.Schema.CMS.Types do
     timestamp_fields()
   end
 
-  object :thread do
+  object :thread_item do
     field(:id, :id)
     field(:title, :string)
     field(:raw, :string)
@@ -232,7 +232,7 @@ defmodule GroupherServerWeb.Schema.CMS.Types do
     field(:index, :integer)
     field(:logo, :string)
     field(:author, :user, resolve: dataloader(CMS, :author))
-    field(:threads, list_of(:thread), resolve: dataloader(CMS, :threads))
+    field(:threads, list_of(:thread_item), resolve: dataloader(CMS, :threads))
     field(:categories, list_of(:category), resolve: dataloader(CMS, :categories))
 
     @desc "total count of post contents"
@@ -443,7 +443,7 @@ defmodule GroupherServerWeb.Schema.CMS.Types do
   end
 
   object :paged_threads do
-    field(:entries, list_of(:thread))
+    field(:entries, list_of(:thread_item))
     pagination_fields()
   end
 

lib/groupher_server_web/schema/cms/cms_types.ex

@@ -8,7 +8,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Comment do
     @desc "write a comment"
     field :create_article_comment, :article_comment do
       # TODO use thread and force community pass-in
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:id, non_null(:id))
       arg(:content, non_null(:string))
       # arg(:mention_users, list_of(:ids))
@@ -82,7 +82,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Comment do
     field :create_comment, :comment do
       # TODO use thread and force community pass-in
       arg(:community, non_null(:string))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:id, non_null(:id))
       arg(:body, non_null(:string))
       arg(:mention_users, list_of(:ids))
@@ -97,7 +97,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Comment do
     @desc "update a comment"
     field :update_comment, :comment do
       arg(:id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:body, non_null(:string))
 
       # TDOO: use a comment resolver
@@ -107,7 +107,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Comment do
 
     @desc "delete a comment"
     field :delete_comment, :comment do
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
       arg(:id, non_null(:id))
 
       middleware(M.Authorize, :login)
@@ -123,7 +123,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Comment do
     @desc "reply a exsiting comment"
     field :reply_comment, :comment do
       arg(:community, non_null(:string))
-      arg(:thread, non_null(:cms_thread), default_value: :post)
+      arg(:thread, non_null(:thread), default_value: :post)
       arg(:id, non_null(:id))
       arg(:body, non_null(:string))
       arg(:mention_users, list_of(:ids))

lib/groupher_server_web/schema/cms/mutations/comment.ex

@@ -76,7 +76,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Community do
     end
 
     @desc "create independent thread"
-    field :create_thread, :thread do
+    field :create_thread, :thread_item do
       arg(:title, non_null(:string))
       arg(:raw, non_null(:string))
       arg(:index, :integer, default_value: 0)
@@ -131,7 +131,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Community do
       arg(:title, non_null(:string))
       arg(:color, non_null(:rainbow_color_enum))
       arg(:community_id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
 
       middleware(M.Authorize, :login)
       middleware(M.PassportLoader, source: :community)
@@ -147,7 +147,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Community do
       # arg(:color, non_null(:rainbow_color_enum))
       arg(:color, non_null(:rainbow_color_enum))
       arg(:community_id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
 
       middleware(M.Authorize, :login)
       middleware(M.PassportLoader, source: :community)
@@ -160,7 +160,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Community do
     field :delete_tag, :tag do
       arg(:id, non_null(:id))
       arg(:community_id, non_null(:id))
-      arg(:thread, :cms_thread, default_value: :post)
+      arg(:thread, :thread, default_value: :post)
 
       middleware(M.Authorize, :login)
       middleware(M.PassportLoader, source: :community)