fix linting

assets/javascripts/discourse/lib/wizard-schema.js.es6

@@ -107,6 +107,7 @@ const action = {
       post: null,
       post_builder: null,
       post_template: null,
+      tags: null,
       skip_redirect: null,
       custom_fields: null,
       required: null,

assets/javascripts/discourse/templates/components/wizard-custom-action.hbs

@@ -84,6 +84,28 @@
     </div>
   </div>
 
+  <div class="setting full field-mapper-setting">
+    <div class="setting-label">
+      <label>{{i18n "admin.wizard.action.create_topic.tags"}}</label>
+    </div>
+
+    <div class="setting-value">
+      {{wizard-mapper
+        inputs=this.action.tags
+        property="tags"
+        onUpdate=(action "mappedFieldUpdated")
+        options=(hash
+          tagSelection="output"
+          outputDefaultSelection="tag"
+          listSelection="output"
+          wizardFieldSelection=true
+          userFieldSelection="key,value"
+          context="action"
+        )
+      }}
+    </div>
+  </div>
+
   {{#if this.action.post_builder}}
     <div class="setting full">
       <div class="setting-label">
@@ -124,28 +146,6 @@
     </div>
   </div>
 
-  <div class="setting full field-mapper-setting">
-    <div class="setting-label">
-      <label>{{i18n "admin.wizard.action.create_topic.tags"}}</label>
-    </div>
-
-    <div class="setting-value">
-      {{wizard-mapper
-        inputs=this.action.tags
-        property="tags"
-        onUpdate=(action "mappedFieldUpdated")
-        options=(hash
-          tagSelection="output"
-          outputDefaultSelection="tag"
-          listSelection="output"
-          wizardFieldSelection=true
-          userFieldSelection="key,value"
-          context="action"
-        )
-      }}
-    </div>
-  </div>
-
   <div class="setting full field-mapper-setting">
     <div class="setting-label">
       <label>{{i18n "admin.wizard.action.create_topic.visible"}}</label>

lib/custom_wizard/action.rb

@@ -104,6 +104,7 @@ class CustomWizard::Action
     end
 
     params = basic_topic_params
+    params.delete(:tags) unless user_can_tag
 
     targets = CustomWizard::Mapper.new(
       inputs: action['recipient'],
@@ -610,6 +611,10 @@ class CustomWizard::Action
       user: user
     ).perform
 
+    if tags = action_tags
+      params[:tags] = tags
+    end
+
     params[:raw] = action['post_builder'] ?
       mapper.interpolate(
         action['post_template'],
@@ -632,10 +637,6 @@ class CustomWizard::Action
       params[:category] = category
     end
 
-    if tags = action_tags
-      params[:tags] = tags
-    end
-
     if public_topic_fields.any?
       public_topic_fields.each do |field|
         unless action[field].nil? || action[field] == ""
@@ -837,4 +838,9 @@ class CustomWizard::Action
       @log.join('; ')
     )
   end
+  def user_can_tag
+    allowed_groups =
+      SiteSetting.pm_tags_allowed_for_groups.split('|').map(&:to_i)
+    user.groups.pluck(:id).any? { |group| allowed_groups.include?(group) }
+  end
 end