Merge branch 'improve-action-builder' into doryphore-dev

Author: seballot

.eslintrc.js

@@ -33,7 +33,9 @@ module.exports = {
     'func-names': ['error', 'never'],
     'space-before-function-paren': ['error', 'never'],
     'lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: true }],
-    'no-new': 'off'
+    'no-new': 'off',
+    'no-restricted-syntax': 'off',
+    'guard-for-in': 'off'
   },
   ignorePatterns: [
     'vendor/',

docs/actions/README.md

@@ -60,6 +60,8 @@ actions:
     description: Une description courte
     hint: Le champ XX doit être présent... # Information importante à savoir si on utilise cette action
     isWrapper: true # rajouter cette ligne pour les actions qui doivent se fermer avec un {{end elem="action"}}
+    onlyAdd: true # rajouter cette ligne pour les actions qui n'ont pas vocation à être éditées avec l'action-builder
+    onlyEdit: true # rajouter cette ligne pour les actions qui n'ont pas vocation à être ajouter via l'action-builder
     wrappedContentExample: 'Teeest' # si l'action est un wrapper, le texte à inclure dans l'action à titre d'exemple
     properties:
       # La liste des paramètres de l'action.
@@ -77,6 +79,7 @@ advanced: true # sera masqué tant que l'utilisateur ne coche pas la case "param
 hint: Mon Texte # Indications
 icon: leaf # nom d'une icone fantwesome
 doclink: https://... # Lien vers une documentation en ligne
+mapped: false # Utilisé uniquement pour générer d'autres options, ne pas reporter ce param dans le code généré
 showif: colorfield # Ce paramètre sera visible uniquement lorsque le paramètre colorfield n'est pas vide
 showif:
   format: portrait # Uniquement visible quand le paramètre "format" est égal à "portait"

docs/actions/bazarliste.yaml

@@ -43,36 +43,6 @@ actions:
           - bazartableau
         type: checkbox
         advanced: true
-      champ:
-        label: _t(AB_bazar_commons2_champ_label)
-        type: form-field
-        extraFields:
-          - id_typeannonce
-          - date_creation_fiche
-          - date_maj_fiche
-        default:
-        advanced: true
-        showExceptFor:
-          - bazarcarto
-          - bazargogocarto
-          - bazartimeline
-          - bazarannuaire
-          - bazarcalendar
-          - bazarcarousel
-      ordre:
-        label: _t(AB_bazar_commons2_ordre_label)
-        type: list
-        default: asc
-        advanced: true
-        options:
-          asc: _t(AB_bazar_commons2_ordre_option_asc)
-          desc: _t(AB_bazar_commons2_ordre_option_desc)
-        showExceptFor:
-          - bazarcarto
-          - bazargogocarto
-          - bazartimeline
-          - bazarannuaire
-          - bazarcalendar
       pagination:
         label: _t(AB_bazar_commons_pagination_label)
         hint: _t(AB_bazar_commons_pagination_hint)
@@ -246,12 +216,6 @@ actions:
         type: checkbox
         advanced: true
         default: 'false'
-      
-      
-      #      query:
-      #        advanced: true
-      #        label: _t(AB_bazar_commons2_query_label)
-      #        type: text
       datefilter:
         label: _t(AB_bazar_commons2_filter_on_date)
         type: list
@@ -282,6 +246,59 @@ actions:
         showif: facettes
         showExceptFor:
           - bazargogocarto
+      champ:
+        intro: <h3>_t(AB_bazar_sort)</h3>
+        label: _t(AB_bazar_facettes_field_label)
+        type: form-field
+        extraFields:
+          - id_typeannonce
+          - date_creation_fiche
+          - date_maj_fiche
+        default:
+        advanced: true
+        showExceptFor:
+          - bazarcarto
+          - bazargogocarto
+          - bazartimeline
+          - bazarannuaire
+          - bazarcalendar
+          - bazarcarousel
+      ordre:
+        label: _t(AB_bazar_commons2_ordre_label)
+        type: list
+        default: asc
+        advanced: true
+        options:
+          asc: _t(AB_bazar_commons2_ordre_option_asc)
+          desc: _t(AB_bazar_commons2_ordre_option_desc)
+        showExceptFor:
+          - bazarcarto
+          - bazargogocarto
+          - bazartimeline
+          - bazarannuaire
+          - bazarcalendar
+      sortfields:
+        advanced: true
+        intro: <center><h4>_t(AB_bazar_sort_dynamique)</h4></center>
+        type: sort-fields
+        btn-label-add: _t(AB_bazar_sort_add_field)
+        subproperties:
+          field:
+            label: _t(AB_bazar_facettes_field_label)
+            type: form-field
+            extraFields:
+              - date_creation_fiche
+              - date_maj_fiche
+          title:
+            label: _t(AB_bazar_facettes_title_label)
+            type: text
+        showExceptFor:
+          - bazarcarto
+          - bazargogocarto
+          - bazartimeline
+          - bazarannuaire
+          - bazarcalendar
+          - bazarcarousel
 
   # -----------------
   # Liste des Actions

docs/actions/col.yaml

@@ -23,7 +23,7 @@
     'AB_bazar_commons_search_label_dynamic' => 'Recherche dynamique',
     'AB_bazar_commons_subproperty_id_label' => 'Valeur',
     'AB_bazar_commons_title' => 'Affichage',
-    'AB_bazar_commons2_champ_label' => 'Champ pour le tri',
+    'AB_bazar_commons2_champ_label' => 'Champ pour le tri par défaut',
     'AB_bazar_commons2_champ_hint' => "Il est possible d'effectuer des requètes avancées, pour en savoir plus",
     'AB_bazar_commons2_filter_index' => "'bf_date_debut_evenement' doit être défini.",
     'AB_bazar_commons2_filter_on_date' => 'Filtrer sur les dates',
@@ -40,7 +40,7 @@
     'AB_bazar_commons2_groupsexpanded_label' => 'Affichage initial des facettes',
     'AB_bazar_commons2_groupsexpanded_false' => 'Uniquement la première dépliée',
     'AB_bazar_commons2_groupsexpanded_true' => 'Toutes Dépliées',
-    'AB_bazar_commons2_ordre_label' => 'Ordre de classement',
+    'AB_bazar_commons2_ordre_label' => 'Ordre',
     'AB_bazar_commons2_ordre_option_asc' => 'Croissant',
     'AB_bazar_commons2_ordre_option_desc' => 'Décroissant',
     'AB_bazar_commons2_nb_label' => 'Limitation',
@@ -57,6 +57,9 @@
     'AB_bazar_facettes_title_label' => 'Titre',
     'AB_bazar_bf_date_debut_evenement_label' => 'Champ pour la date de début',
     'AB_bazar_bf_date_fin_evenement_label' => 'Champ pour la date de fin',
+    'AB_bazar_sort' => 'Trier les données',
+    'AB_bazar_sort_dynamique' => "Tri dynamique",
+    'AB_bazar_sort_add_field' => 'Ajouter un tri dynamique',
     // Bazarcarto
     'AB_bazarcarto_label' => 'Cartographie',
     'AB_bazarcarto_description' => 'Les fiches seront sous forme de marqueur.',
@@ -290,7 +293,7 @@
     'AB_management_despam_label' => 'Nettoyer les pages récemment vandalisées',
     'AB_management_despam_hint' => 'Liste les dernières versions des pages récemment créées pour pouvoir choisir celles à supprimer',
     'AB_management_despam_hint_details' => 'Ceci permet de rétablir les pages vandalisées ou de supprimer les nouvelles pages créées'
-            . ' par des spams. ATTENTION : la suppression des pages choisies sera DÉFINITITVE.',
+        . ' par des spams. ATTENTION : la suppression des pages choisies sera DÉFINITITVE.',
     'AB_management_editconfig_label' => 'Modifier des paramètres du fichier de configuration',
     'AB_management_adminbackups_label' => 'Gestion des sauvegardes',
     'AB_management_commentstable_label' => 'Table des commentaires',

docs/actions/lang/actionsbuilder_fr.inc.php

@@ -363,15 +363,19 @@ Mise en forme propose 6 possibilités :
     - `type="collapsible` : permet de laisser ouvert le panel lorque la page sera chargé.
 
 - **Afficher plusieurs colonnes** : permet de créer plusieurs colonnes dans une page ou une partie de page, très utile pour des mises en page avancées.
-  - cette action ne bénificie pas encore d'aide au codage. Pour ajouter ou supprimer des colonnes à votre tableau, il suffira d'ajouter ou supprimer le code suivant) avant {{end elem="grid"}}.
 
-```yeswiki
-{{col size="3"}}
-Texte de la colonne à changer par la suite
-{{end elem="col"}}
-```
+  ```yeswiki
+  {{grid}}
+  {{col size="3"}}
+  Texte de la colonne à changer par la suite
+  {{end elem="col"}}
+  {{col size="9"}}
+  Texte de la colonne à changer par la suite
+  {{end elem="col"}}
+  {{end elem="grid"}}
+  ```
 
-!> important : la somme des size (largeur) de vos colonnes doit être égal à 12, cela veut dire que vous pouvez faire 12 colonnes de size 1 ou 3 colonnes de size 4 et bien sur mixer les possibilités (une de 6 et deux de 3)
+  !> important : la somme des size (largeur) de vos colonnes doit être égal à 12, cela veut dire que vous pouvez faire 12 colonnes de size 1 ou 3 colonnes de size 4 et bien sur mixer les possibilités (une de 6 et deux de 3)
 
 ### Syndication / Flux RSS
 

docs/fr/prise-en-main.md

@@ -53,7 +53,6 @@ class Aceditor {
     setupAceditorKeyBindings(this.$aceContainer, this.$toolbar)
     this.initToolbar()
     this.initEditionHelpers()
-    console.log(this)
     this.editor.ace.setOptions({ placeholder: this.$textarea.attr('placeholder') })
     this.editor.on('blur', () => {
       this.flyingButton.hide()
@@ -117,11 +116,8 @@ class Aceditor {
       switch (cursor.groupType) {
         case 'yw-action': {
           const actionName = cursor.groupData['action-name']
-          if (
-            this.actionsBuilder.allAvailableActionsWithBackward.includes(
-              actionName
-            )
-          ) {
+          if (this.actionsBuilder.allAvailableActionsWithBackward.includes(actionName)) {
+            if (this.actionsBuilder.getActionConfiguration(actionName).onlyAdd) return
             this.$toolbar.find('.component-action-list').addClass('only-edit')
             this.flyingButton.show().onClick(() => {
               this.actionsBuilder.open(this.editor, { action: cursor.groupTextWithoutMarkup })

tools/aceditor/presentation/javascripts/aceditor.js

@@ -2,13 +2,15 @@ import InputHelper from './components/InputHelper.js'
 import InputHidden from './components/InputHidden.js'
 import InputText from './components/InputText.js'
 import InputPageList from './components/InputPageList.js'
+import InputNavLinks from './components/InputNavLinks.js'
 import InputCheckbox from './components/InputCheckbox.js'
 import InputList from './components/InputList.js'
 import InputDivider from './components/InputDivider.js'
 import InputIcon from './components/InputIcon.js'
 import InputColor from './components/InputColor.js'
 import InputFormField from './components/InputFormField.js'
 import InputFacette from './components/InputFacette.js'
+import InputSortFields from './components/InputSortFields.js'
 import InputReaction from './components/InputReaction.js'
 import InputIconMapping from './components/InputIconMapping.js'
 import InputColorMapping from './components/InputColorMapping.js'
@@ -32,9 +34,11 @@ const components = {
   InputHidden,
   InputDivider,
   InputFacette,
+  InputSortFields,
   InputReaction,
   InputIconMapping,
   InputColorMapping,
+  InputNavLinks,
   InputGeo,
   InputClass,
   InputCorrespondance,
@@ -116,7 +120,7 @@ export const app = {
       this.configPanels.forEach((panel) => { result = { ...result, ...panel.params.properties } })
       return result
     },
-    wikiCodeBase() {
+    wikiCodeStart() {
       let actionId = this.selectedActionId
       if (this.isBazarListeAction) actionId = 'bazarliste'
       let result = `{{${actionId}`
@@ -126,18 +130,45 @@ export const app = {
       result += ' }}'
       return result
     },
+    wikiCodeDefaultContent() {
+      let content = this.selectedAction.wrappedContentExample
+      if (this.selectedActionId === 'tabs' && this.actionParams.titles) {
+        content = this.actionParams.titles.split(',')
+          .map((tabName) => this.selectedAction.wrappedContentExample.replace('{tabName}', tabName))
+          .join('\n')
+      }
+      if (this.selectedActionId === 'accordion') {
+        content = '\n'
+        for (let i = 0; i < this.values.nb; i++) {
+          content += `${this.selectedAction.wrappedContentExample}\n`
+        }
+      }
+      if (this.selectedActionId === 'grid') {
+        content = '\n'
+        const size = 12 / this.values.nb
+        for (let i = 0; i < this.values.nb; i++) {
+          content += `${this.selectedAction.wrappedContentExample.replace('{size}', size)}\n`
+        }
+      }
+      if (!['label', 'accordion', 'grid'].includes(this.selectedActionId)) content = `\n${content}\n`
+      return content
+    },
+    wikiCodeEnd() {
+      return `{{end elem="${this.selectedActionId}"}}`
+    },
     wikiCode() {
-      let result = this.wikiCodeBase
+      let result = this.wikiCodeStart
       if (this.selectedAction.isWrapper && !this.isEditingExistingAction) {
-        result += `\n${this.selectedAction.wrappedContentExample}{{end elem="${this.selectedActionId}"}}`
+        result += this.wikiCodeDefaultContent
+        result += this.wikiCodeEnd
       }
       return result
     },
     wikiCodeForIframe() {
-      let result = this.wikiCodeBase
+      let result = this.wikiCodeStart
       if (this.selectedAction.isWrapper && result) {
-        result += `${this.selectedAction.wrappedContentExample}\n`
-        result += `{{end elem="${this.selectedActionId}"}}`
+        result += this.wikiCodeDefaultContent
+        result += this.wikiCodeEnd
       }
       return result
     }
@@ -159,7 +190,9 @@ export const app = {
         // use a fake dom to parse wiki code attributes
         const fakeDom = $(`<${this.currentSelectedAction}/>`)[0]
 
-        for (const attribute of fakeDom.attributes) Vue.set(this.values, attribute.name, attribute.value)
+        for (const attribute of fakeDom.attributes) {
+          Vue.set(this.values, attribute.name, attribute.value)
+        }
 
         const newActionId = fakeDom.tagName.toLowerCase()
         this.selectedActionId = newActionId
@@ -319,7 +352,8 @@ export const app = {
         const config = this.selectedActionAllConfigs[key]
         const value = this.values[key]
         if (result.hasOwnProperty(key) || value === undefined || config && config.default && `${value}` == `${config.default}`
-            || typeof value == 'object' || config && !this.checkConfigDisplay(config)) { continue }
+            || typeof value == 'object' || config && config.mapped === false
+            || config && !this.checkConfigDisplay(config)) { continue }
         result[key] = value
       }
       // Adds values from special components

tools/aceditor/presentation/javascripts/actions-builder-app.js

@@ -28,6 +28,15 @@ export default class {
     return this.allAvailableActions.concat(Object.keys(ACTIONS_BACKWARD_COMPATIBILITY))
   }
 
+  getActionConfiguration(actionName) {
+    for (const group of Object.values(actionsBuilderData.action_groups)) {
+      if (group.actions[actionName]) {
+        return group.actions[actionName]
+      }
+    }
+    return {}
+  }
+
   open(editor, options) {
     // Handle backward compat
     if (options.action) {

tools/aceditor/presentation/javascripts/actions-builder.js

@@ -0,0 +1,26 @@
+import InputMultiInput from './InputMultiInput.js'
+import InputPageList from './InputPageList.js'
+
+export default {
+  mixins: [InputMultiInput],
+  components: { InputPageList },
+  methods: {
+    parseNewValues(newValues) {
+      this.elements = []
+      const links = newValues.links ? newValues.links.split(',') : [window.wiki.pageTag, '']
+      const titles = newValues.titles ? newValues.titles.split(',') : [window.wiki.pageTag, '']
+      for (let i = 0; i < links.length; i++) {
+        this.elements.push({
+          link: links[i],
+          title: titles[i]
+        })
+      }
+    },
+    getValues() {
+      return {
+        links: this.elements.map((g) => g.link).filter((e) => e != '').join(','),
+        titles: this.elements.map((g) => g.title).filter((e) => e != '').join(',')
+      }
+    }
+  }
+}