wip

assets/javascripts/discourse/components/wizard-custom-field.js.es6

@@ -41,7 +41,8 @@ export default Component.extend({
     if (this.isDropdown) {
       options.wizardFieldSelection = 'key,value';
       options.listSelection = 'assignment';
-      options.inputTypes = 'pair,assignment';
+      options.inputTypes = 'association,assignment';
+      options.singular = true;
       options.pairConnector = 'association';
       options.keyPlaceholder = 'admin.wizard.key';
       options.valuePlaceholder = 'admin.wizard.value';

assets/javascripts/discourse/components/wizard-mapper-connector.js.es6

@@ -1,10 +1,10 @@
 import Component from "@ember/component";
-import { lt } from '@ember/object/computed';
+import { gt } from '@ember/object/computed';
 import { computed } from "@ember/object";
 
 export default Component.extend({
-  classNameBindings: [':mapper-connector', ':mapper-block', 'single'],
+  classNameBindings: [':mapper-connector', ':mapper-block', 'hasMultiple::single'],
-  single: lt('connectors.length', 2),
+  hasMultiple: gt('connectors.length', 1),
   connectorLabel: computed(function() {
     let key = this.connector;
     let path = this.inputTypes ? `input.${key}.name` : `connector.${key}`;

assets/javascripts/discourse/components/wizard-mapper-input.js.es6

@@ -5,13 +5,14 @@ import Component from "@ember/component";
 import { observes } from "discourse-common/utils/decorators";
 
 export default Component.extend({
-  classNameBindings: [':mapper-input', 'type'],
+  classNameBindings: [':mapper-input', 'inputType'],
   inputType: alias('input.type'),
   isConditional: equal('inputType', 'conditional'),
   isAssignment: equal('inputType', 'assignment'),
-  isPair: equal('inputType', 'pair'),
+  isAssociation: equal('inputType', 'association'),
+  isValidation: equal('inputType', 'validation'),
   hasOutput: or('isConditional', 'isAssignment'),
-  hasPairs: or('isConditional', 'isPair'),
+  hasPairs: or('isConditional', 'isAssociation', 'isValidation'),
   connectors: computed(function() { return connectorContent('output', this.input.type, this.options) }),
   inputTypes: computed(function() { return inputTypesContent(this.options) }),
     

assets/javascripts/discourse/components/wizard-mapper-selector.js.es6

@@ -1,6 +1,6 @@
 import { alias, or, gt } from "@ember/object/computed";
 import { computed } from "@ember/object";
-import { default as discourseComputed, observes } from "discourse-common/utils/decorators";
+import { default as discourseComputed, observes, on } from "discourse-common/utils/decorators";
 import { getOwner } from 'discourse-common/lib/get-owner';
 import { defaultSelectionType, selectionTypes } from '../lib/wizard-mapper'; 
 import { snakeCase, selectKitContent } from '../lib/wizard';
@@ -72,7 +72,7 @@ export default Component.extend({
     return showTypes ? 'chevron-down' : 'chevron-right';
   },
   
-  @observes('options.@each', 'inputType')
+  @observes('inputType')
   resetActiveType() {
     this.set('activeType', defaultSelectionType(this.selectorType, this.options));
   },

assets/javascripts/discourse/components/wizard-mapper.js.es6

@@ -1,22 +1,24 @@
 import { getOwner } from 'discourse-common/lib/get-owner';
-import { on } from 'discourse-common/utils/decorators';
 import { newInput, selectionTypes } from '../lib/wizard-mapper';
-import { default as discourseComputed, observes } from 'discourse-common/utils/decorators';
+import { default as discourseComputed, observes, on } from 'discourse-common/utils/decorators';
+import { gt } from "@ember/object/computed";
 import Component from "@ember/component";
 import { A } from "@ember/array";
 
 export default Component.extend({
   classNames: 'wizard-mapper',
+  hasInput: gt('inputs.length', 0),
   
-  @discourseComputed('inputs.[]', 'options.singular')
+  @discourseComputed('options.singular', 'hasInput')
-  canAdd(inputs, singular) {
+  canAdd(singular, hasInput) {
-    return !singular || !inputs || inputs.length < 1;
+    return !singular || !hasInput;
   },
   
-  @discourseComputed('options.@each')
+  @discourseComputed('options.@each.inputType')
   inputOptions(options) {
     let result = {
       inputTypes: options.inputTypes || 'conditional,assignment',
+      inputConnector: options.inputConnector || 'or',
       pairConnector: options.pairConnector || null,
       outputConnector: options.outputConnector || null,
       context: options.context || null
@@ -45,7 +47,9 @@ export default Component.extend({
         this.set('inputs', A());
       }
 
-      this.get('inputs').pushObject(newInput(this.inputOptions));
+      this.get('inputs').pushObject(
+        newInput(this.inputOptions, this.inputs.length)
+      );
     },
 
     remove(input) {

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

@@ -50,7 +50,7 @@ function buildMappedJson(inputs) {
     if (present(inpt.output)) {
       input.output = inpt.output;
       input.output_type = snakeCase(inpt.output_type);
-      input.connector = inpt.connector;
+      input.output_connector = inpt.output_connector;
     }
     
     if (present(inpt.pairs)) {
@@ -74,8 +74,7 @@ function buildMappedJson(inputs) {
     }
         
     if ((input.type === 'assignment' && present(input.output)) ||
-        (input.type === 'conditional' && present(input.pairs)) ||
+        present(input.pairs)) {
-        (input.type === 'pair' && present(input.pairs))) {
       
       result.push(input);
     }
@@ -146,20 +145,15 @@ function buildStepJson(object) {
   };
 }
 
-function mappedProperty(property, value) {
+function castCase(property, value) {
   return property.indexOf('_type') > -1 ? camelCase(value) : value;
 }
 
-function buildObject(json, type) {
+function buildProperty(json, property, type) {  
-  let params = {
+  if (mapped(property, type) && present(json[property])) {
-    isNew: false
-  }
-  
-  Object.keys(json).forEach(prop => {
-    if (mapped(prop, type) && present(json[prop])) {
     let inputs = [];
     
-      json[prop].forEach(inputJson => {
+    json[property].forEach(inputJson => {
       let input = {}
       
       Object.keys(inputJson).forEach(inputKey => {
@@ -171,7 +165,7 @@ function buildObject(json, type) {
             let pair = {};
             
             Object.keys(pairJson).forEach(pairKey => {
-                pair[pairKey] = mappedProperty(pairKey,  pairJson[pairKey]);
+              pair[pairKey] = castCase(pairKey,  pairJson[pairKey]);
             });
             
             pair.pairCount = pairCount;
@@ -183,7 +177,7 @@ function buildObject(json, type) {
           
           input.pairs = pairs;
         } else {
-            input[inputKey] = mappedProperty(inputKey,  inputJson[inputKey]);
+          input[inputKey] = castCase(inputKey,  inputJson[inputKey]);
         }
       });
       
@@ -192,10 +186,19 @@ function buildObject(json, type) {
       );
     });
     
-      params[prop] = A(inputs);
+    return A(inputs);
   } else {
-      params[prop] = json[prop];
+    return json[property];
   }
+}
+
+function buildObject(json, type) {
+  let params = {
+    isNew: false
+  }
+  
+  Object.keys(json).forEach(prop => {
+    params[prop] = buildProperty(json, prop, type)
   });
   
   return EmberObject.create(params);
@@ -228,7 +231,7 @@ function buildProperties(json) {
     props.existingId = true;
 
     properties.wizard.forEach((p) => {
-      props[p] = json[p];
+      props[p] = buildProperty(json, p, 'wizard');
       
       if (wizardHasAdvanced(p, json[p])) {
         props.showAdvanced = true;
@@ -242,7 +245,7 @@ function buildProperties(json) {
         };
         
         properties.step.forEach((p) => {
-          stepParams[p] = stepJson[p];
+          stepParams[p] = buildProperty(stepJson, p, 'wizard');;
                     
           if (stepHasAdvanced(p, stepJson[p])) {
             stepParams.showAdvanced = true;

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

@@ -34,16 +34,19 @@ const connectors = {
     'greater',
     'less',
     'greater_or_equal',
-    'less_or_equal'
+    'less_or_equal',
+    'regex'
   ]
 }
 
 function defaultConnector(connectorType, inputType, opts = {}) {
   if (opts[`${connectorType}Connector`]) return opts[`${connectorType}Connector`];  
-  if (inputType === 'assignment' && connectorType === 'output') return 'set';
   if (inputType === 'conditional' && connectorType === 'output') return 'then';
   if (inputType === 'conditional' && connectorType === 'pair') return 'equal';
-  if (inputType === 'pair') return 'equal';
+  if (inputType === 'assignment' && connectorType === 'output') return 'set';
+  if (inputType === 'association' && connectorType === 'pair') return 'association';
+  if (inputType === 'validation' && connectorType === 'pair') return 'equal';
+  return 'equal';
 }
 
 function connectorContent(connectorType, inputType, opts) {
@@ -114,7 +117,7 @@ function newPair(inputType, options = {}) {
   return EmberObject.create(params);
 }
 
-function newInput(options = {}) {
+function newInput(options = {}, count) {
   const inputType = defaultInputType(options);
       
   let params = {
@@ -132,12 +135,16 @@ function newInput(options = {}) {
     )
   }
     
+  if (count > 0) {
+    params.connector = options.inputConnector;
+  }
+  
   if (['conditional', 'assignment'].indexOf(inputType) > -1 ||
       options.outputDefaultSelection ||
       options.outputConnector) {
     
     params['output_type'] = defaultSelectionType('output', options);
-    params['connector'] = defaultConnector('output', inputType, options);
+    params['output_connector'] = defaultConnector('output', inputType, options);
   }
     
   return EmberObject.create(params);

assets/javascripts/discourse/templates/admin-wizard.hbs

@@ -111,14 +111,14 @@
         {{wizard-mapper
           inputs=model.permitted
           options=(hash
-            singular=true
             context='wizard'
-            inputTypes='assignment'
+            inputTypes='assignment,validation'
             groupSelection='output'
-            textSelection='key,value'
+            userFieldSelection='key'
+            textSelection='value'
+            inputConnector='and'
           )}}
       </div>
-      
     </div>
     
     {{wizard-advanced-toggle showAdvanced=model.showAdvanced}}

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

@@ -248,7 +248,7 @@
             {{wizard-mapper
               inputs=action.custom_fields
               options=(hash
-                pairConnector='set'
+                inputTypes='association'
                 wizardFieldSelection='value'
                 userFieldSelection='value'
                 keyPlaceholder='admin.wizard.action.custom_fields.key'

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

@@ -48,6 +48,7 @@
           {{wizard-mapper
             inputs=step.required_data
             options=(hash
+              inputTypes='validation'
               wizardFieldSelection='value'
               userFieldSelection='value'
               keyPlaceholder="admin.wizard.submission_key"

assets/javascripts/discourse/templates/components/wizard-mapper-connector.hbs

@@ -1,10 +1,10 @@
-{{#if single}}
+{{#if hasMultiple}}
-  <span class="connector-single">
-    {{connectorLabel}}
-  </span>
-{{else}}
   {{combo-box
     value=connector
     content=connectors
     onChange=(action (mut connector))}}
+{{else}}
+  <span class="connector-single">
+    {{connectorLabel}}
+  </span>
 {{/if}}

assets/javascripts/discourse/templates/components/wizard-mapper-input.hbs

@@ -25,7 +25,7 @@
 {{#if hasOutput}}
   {{#if hasPairs}}
     {{wizard-mapper-connector
-      connector=input.connector
+      connector=input.output_connector
       connectors=connectors}}
   {{/if}}
 

assets/javascripts/discourse/templates/components/wizard-mapper.hbs

@@ -1,4 +1,8 @@
 {{#each inputs as |input|}}
+  {{#if input.connector}}
+    {{wizard-mapper-connector connector=input.connector}}
+  {{/if}}
+  
   {{wizard-mapper-input 
     input=input
     options=inputOptions

assets/javascripts/wizard/templates/components/wizard-field-dropdown.hbs

@@ -2,6 +2,5 @@
             class=fieldClass
             value=field.value
             content=field.content
-            none=(hash id="__none__" label=field.dropdown_none)
             nameProperty="label"
             tabindex="9"}}

assets/stylesheets/common/wizard-mapper.scss

@@ -17,6 +17,12 @@
   div.mapper-block:not(:last-of-type) {
     margin-right: 10px;
   }
+  
+  > .mapper-connector.single {
+    width: min-content;
+    margin-bottom: 10px;
+    height: 20px
+  }
 }
 
 [class~='mapper-input'] {
@@ -156,6 +162,10 @@
     left: 50%;
     transform: translateX(-50%);
   }
+  
+  .mapper-connector {
+    min-width: 50px;
+  }
 }
 
 .mapper-pair {

config/locales/client.en.yml

@@ -67,8 +67,10 @@ en:
             output: 'then'
           assignment:
             name: 'set'
-          pair:
+          association:
-            name: 'pair'
+            name: 'map'
+          validation:
+            name: 'ensure'
   
         selector:
           label:
@@ -119,8 +121,6 @@ en:
           description: "Description"
           image: "Image"
           image_placeholder: "Image url"
-          dropdown_none: "None"
-          dropdown_none_placeholder: "Label"
           required: "Required"
           required_label: "Field is Required"
           min_length: "Min Length"
@@ -134,6 +134,8 @@ en:
           content: "Content"
         
         connector:
+          and: "and"
+          or: "or"
           then: "then"
           set: "set"
           equal: '='
@@ -141,6 +143,7 @@ en:
           less: '<'
           greater_or_equal: '>='
           less_or_equal: '<='
+          regex: '=~'
           association: '→'
         
         action:

lib/custom_wizard/actions.rb

@@ -45,7 +45,7 @@ class CustomWizard::Action
       inputs: action['recipient'],
       data: data,
       user: user
-    ).output
+    ).perform
         
     if params[:title] && params[:raw]
       params[:archetype] = Archetype.private_message
@@ -144,7 +144,7 @@ class CustomWizard::Action
       opts: {
         multiple: true
       }
-    ).output
+    ).perform
         
     groups = groups.flatten.reduce([]) do |result, g|
       begin
@@ -183,7 +183,7 @@ class CustomWizard::Action
       inputs: action['category'],
       data: data,
       user: user
-    ).output
+    ).perform
     
     if output.is_a?(Array)
       output.first
@@ -199,7 +199,7 @@ class CustomWizard::Action
       inputs: action['tags'],
       data: data,
       user: user,
-    ).output
+    ).perform
     
     if output.is_a?(Array)
       output.flatten
@@ -212,16 +212,17 @@ class CustomWizard::Action
   
   def add_custom_fields(params = {})
     if (custom_fields = action['custom_fields']).present?
-      custom_fields.each do |field|
+      field_map = CustomWizard::Mapper.new(
-        pair = field['pairs'].first
+        inputs: custom_fields,
-        value = mapper.map_field(pair['key'], pair['key_type'])
+        data: data,
-        key = mapper.map_field(pair['value'], pair['value_type'])
+        user: user
+      ).perform
       
-        if key && 
+      field_map.each do |field|
-          value.present? &&
+        keyArr = field[:key].split('.')
-          (keyArr = key.split('.')).length === 2
+        value = field[:value]
         
-          if keyArr.first === 'topic'
+        if keyArr.length != 2 || keyArr.first === 'topic'
           params[:topic_opts] ||= {}
           params[:topic_opts][:custom_fields] ||= {}
           params[:topic_opts][:custom_fields][keyArr.last] = value
@@ -231,7 +232,6 @@ class CustomWizard::Action
         end
       end
     end
-    end
     
     params
   end
@@ -245,7 +245,7 @@ class CustomWizard::Action
       inputs: action['title'],
       data: data,
       user: user
-    ).output
+    ).perform
 
     params[:raw] = action['post_builder'] ?
       mapper.interpolate(action['post_template']) :

lib/custom_wizard/builder.rb

@@ -235,12 +235,26 @@ class CustomWizard::Builder
       @wizard.needs_groups = true
     end
     
-    if (content = field_template['content']).present?
+    if (content_inputs = field_template['content']).present?
-      params[:content] = CustomWizard::Mapper.new(
+      content = CustomWizard::Mapper.new(
-        inputs: content,
+        inputs: content_inputs,
         user: @wizard.user,
-        data: @submissions.last
+        data: @submissions.last,
-      ).output
+        opts: {
+          with_type: true
+        }
+      ).perform
+            
+      if content[:type] == 'association'
+        content[:result] = content[:result].map do |item|
+          { 
+            id: item[:key],
+            name: item[:value] 
+          }
+        end
+      end
+      
+      params[:content] = content[:result]
     end
     
     field = step.add_field(params)
@@ -252,7 +266,7 @@ class CustomWizard::Builder
         inputs: prefill,
         user: @wizard.user,
         data: @submissions.last
-      ).output
+      ).perform
     end
   end
 

lib/custom_wizard/mapper.rb

@@ -8,7 +8,8 @@ class CustomWizard::Mapper
     greater: '>',
     less: '<',
     greater_or_equal: '>=',
-    less_or_equal: '<='
+    less_or_equal: '<=',
+    regex: '=~'
   }
  
   def initialize(params)
@@ -18,33 +19,63 @@ class CustomWizard::Mapper
     @opts = params[:opts] || {}
   end
   
-  def output
+  def perform
     multiple = @opts[:multiple]
-    output = multiple ? [] : nil
+    perform_result = multiple ? [] : nil
     
     inputs.each do |input|
-      if input['type'] === 'conditional' && validate_pairs(input['pairs'])
+      input_type = input['type']
+      pairs = input['pairs']
+
+      if (input_type === 'conditional' && validate_pairs(pairs)) || input_type === 'assignment'
+        output = input['output']
+        output_type = input['output_type']
+        
+        result = build_result(map_field(output, output_type), input_type)
+        
         if multiple
-          output.push(map_field(input['output'], input['output_type']))
+          perform_result.push(result)
         else
-          output = map_field(input['output'], input['output_type'])
+          perform_result = result
           break
         end
       end
       
-      if input['type'] === 'assignment'
+      if input_type === 'validation'
-        value = map_field(input['output'], input['output_type'])
+        result = build_result(validate_pairs(pairs), input_type)
         
-        if @opts[:multiple]
+        if multiple
-          output.push(value)
+          perform_result.push(result)
         else
-          output = value
+          perform_result = result
+          break
+        end
+      end
+      
+      if input_type === 'association'
+        result = build_result(map_pairs(pairs), input_type)
+        
+        if multiple
+          perform_result.push(result)
+        else
+          perform_result = result
           break
         end
       end
     end
           
-    output
+    perform_result
+  end
+  
+  def build_result(result, type)
+    if opts[:with_type]
+      {
+        type: type,
+        result: result
+      }
+    else
+      result
+    end
   end
   
   def validate_pairs(pairs)
@@ -52,10 +83,12 @@ class CustomWizard::Mapper
     
     pairs.each do |pair|
       key = map_field(pair['key'], pair['key_type'])
-      value = map_field(pair['value'], pair['value_type'])
+      operator = map_operator(pair['connector'])
+      value = interpolate(map_field(pair['value'], pair['value_type']))
+      value = "/#{value}/" if pair['connector'] == 'regex'
             
       begin
-        failed = true unless key.public_send(operator(pair['connector']), value)
+        failed = true unless key.public_send(operator, value)
       rescue NoMethodError
         #
       end
@@ -64,7 +97,25 @@ class CustomWizard::Mapper
     !failed
   end
   
-  def operator(connector)
+  def map_pairs(pairs)
+    result = []
+    
+    pairs.each do |pair|
+      key = map_field(pair['key'], pair['key_type'])
+      value = map_field(pair['value'], pair['value_type'])
+      
+      if key && value
+        result.push(
+          key: key,
+          value: value
+        )
+      end
+    end
+    
+    result
+  end
+  
+  def map_operator(connector)
     OPERATORS[connector.to_sym] || '=='
   end
   

lib/custom_wizard/wizard.rb

@@ -129,8 +129,27 @@ class CustomWizard::Wizard
   def permitted?
     return false unless user
     return true if user.admin? || permitted.blank?
-    group_ids = permitted.first['output']
+    
-    return GroupUser.exists?(group_id: group_ids, user_id: user.id)
+    mapper = CustomWizard::Mapper.new(
+      inputs: permitted,
+      user: user,
+      opts: {
+        with_type: true,
+        multiple: true
+      }
+    ).perform
+        
+    return true if mapper.blank?
+    
+    mapper.all? do |m|
+      if m.type === 'assignment'
+        GroupUser.exists?(group_id: m.result, user_id: user.id)
+      elsif m.type === 'validation'
+        mapper.result
+      else
+        true
+      end
+    end
   end
 
   def reset

serializers/custom_wizard/wizard_field_serializer.rb

@@ -2,8 +2,7 @@
 
 class CustomWizardFieldSerializer < ::WizardFieldSerializer
   
-  attributes :dropdown_none,
+  attributes :image,
-             :image,
              :file_types,
              :limit,
              :property,
@@ -31,10 +30,6 @@ class CustomWizardFieldSerializer < ::WizardFieldSerializer
     I18n.t("#{object.key || i18n_key}.placeholder", default: '')
   end
 
-  def dropdown_none
-    object.dropdown_none
-  end
-
   def file_types
     object.file_types
   end