From 057a33e8bfbe766fbe617fcb503f95ed689260f1 Mon Sep 17 00:00:00 2001
From: axolotle <nchesnais@mailoo.org>
Date: Fri, 24 Mar 2023 13:49:50 +0100
Subject: [PATCH 1/3] AppInfo: send config panel `args` only if not empty

---
 app/src/views/app/AppInfo.vue | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/src/views/app/AppInfo.vue b/app/src/views/app/AppInfo.vue
index 23c9618a..66a5d66c 100644
--- a/app/src/views/app/AppInfo.vue
+++ b/app/src/views/app/AppInfo.vue
@@ -265,6 +265,7 @@ import api, { objectToParams } from '@/api'
 import { readableDate } from '@/helpers/filters/date'
 import { humanPermissionName } from '@/helpers/filters/human'
 import { required } from '@/helpers/validators'
+import { isEmptyValue } from '@/helpers/commons'
 import {
   formatFormData,
   formatI18nField,
@@ -438,7 +439,7 @@ export default {
         action
           ? `apps/${this.id}/actions/${action}`
           : `apps/${this.id}/config/${id}`,
-        { args: objectToParams(args) },
+        isEmptyValue(args) ? {} : { args: objectToParams(args) },
         { key: `apps.${action ? 'action' : 'update'}_config`, id, name: this.id }
       ).then(() => {
         this.$refs.view.fetchQueries({ triggerLoading: true })

From 3fbf4cfdeba7fe89aeb35f71771c435a513e6932 Mon Sep 17 00:00:00 2001
From: axolotle <nchesnais@mailoo.org>
Date: Fri, 24 Mar 2023 14:45:59 +0100
Subject: [PATCH 2/3] AppInfo: properly display config panel init error without
 blocking the app info page

---
 app/src/components/globals/YunoAlert.vue |  1 +
 app/src/i18n/locales/en.json             |  3 +-
 app/src/views/app/AppInfo.vue            | 41 +++++++++++++++++-------
 3 files changed, 33 insertions(+), 12 deletions(-)

diff --git a/app/src/components/globals/YunoAlert.vue b/app/src/components/globals/YunoAlert.vue
index 421835f5..c3a6d42c 100644
--- a/app/src/components/globals/YunoAlert.vue
+++ b/app/src/components/globals/YunoAlert.vue
@@ -28,6 +28,7 @@ export default {
 
   computed: {
     _icon () {
+      if (this.icon) return this.icon
       return DEFAULT_STATUS_ICON[this.variant]
     }
   }
diff --git a/app/src/i18n/locales/en.json b/app/src/i18n/locales/en.json
index eaf563df..cb1652a3 100644
--- a/app/src/i18n/locales/en.json
+++ b/app/src/i18n/locales/en.json
@@ -75,7 +75,8 @@
       },
       "info": {
         "forum": "Search or ask the forum!",
-        "problem": "A problem with this app?"
+        "problem": "A problem with this app?",
+        "config_panel_error": "An error prevents the configuration panel from being displayed:"
       },
       "install": {
         "license": "License: {license}",
diff --git a/app/src/views/app/AppInfo.vue b/app/src/views/app/AppInfo.vue
index 66a5d66c..b1c314d2 100644
--- a/app/src/views/app/AppInfo.vue
+++ b/app/src/views/app/AppInfo.vue
@@ -1,5 +1,8 @@
 <template>
-  <view-base :queries="queries" @queries-response="onQueriesResponse" ref="view">
+  <view-base
+    :queries="queries" @queries-response="onQueriesResponse" :loading="loading"
+    ref="view"
+  >
     <yuno-alert v-if="app && app.doc && app.doc.notifications && app.doc.notifications.postInstall.length" variant="info" class="my-4">
       <div class="d-md-flex align-items-center mb-3">
         <h2 v-t="'app.doc.notifications.post_install'" class="md-m-0" />
@@ -89,6 +92,14 @@
       <vue-showdown :markdown="app.description" flavor="github" />
     </section>
 
+    <yuno-alert
+      v-if="config_panel_err"
+      class="mb-4" variant="danger" icon="bug"
+    >
+      <p>{{ $t('app.info.config_panel_error') }}</p>
+      <p>{{ config_panel_err }}</p>
+    </yuno-alert>
+
     <!-- BASIC INFOS -->
     <config-panels v-bind="config" @submit="onConfigSubmit">
       <!-- OPERATIONS TAB -->
@@ -289,11 +300,12 @@ export default {
       queries: [
         ['GET', `apps/${this.id}?full`],
         ['GET', { uri: 'users/permissions?full', storeKey: 'permissions' }],
-        ['GET', { uri: 'domains' }],
-        ['GET', `apps/${this.id}/config?full`]
+        ['GET', { uri: 'domains' }]
       ],
+      loading: true,
       app: undefined,
       form: undefined,
+      config_panel_err: null,
       config: {
         panels: [
           // Fake integration of operations in config panels
@@ -346,14 +358,8 @@ export default {
         }
         return linksIcons[linkType]
     },
-    onQueriesResponse (app, _, __, config) {
-      if (app.supports_config_panel) {
-        const config_ = formatYunoHostConfigPanels(config)
-        // reinject 'operations' fake config tab
-        config_.panels.unshift(this.config.panels[0])
-        this.config = config_
-      }
 
+    async onQueriesResponse (app) {
       const form = { labels: [] }
 
       const mainPermission = app.permissions[this.id + '.main']
@@ -430,6 +436,18 @@ export default {
       if (!Object.values(this.app.doc.notifications).some((notif) => notif.length)) {
         this.app.doc.notifications = null
       }
+
+      if (app.supports_config_panel) {
+        await api.get(`apps/${this.id}/config?full`).then((config) => {
+          const config_ = formatYunoHostConfigPanels(config)
+          // reinject 'operations' fake config tab
+          config_.panels.unshift(this.config.panels[0])
+          this.config = config_
+        }).catch((err) => {
+          this.config_panel_err = err.message
+        })
+      }
+      this.loading = false
     },
 
     async onConfigSubmit ({ id, form, action, name }) {
@@ -442,7 +460,8 @@ export default {
         isEmptyValue(args) ? {} : { args: objectToParams(args) },
         { key: `apps.${action ? 'action' : 'update'}_config`, id, name: this.id }
       ).then(() => {
-        this.$refs.view.fetchQueries({ triggerLoading: true })
+        this.loading = true
+        this.$refs.view.fetchQueries()
       }).catch(err => {
         if (err.name !== 'APIBadRequestError') throw err
         const panel = this.config.panels.find(panel => panel.id === id)

From a6d8acf706078daedac4499134f9179dddf85069 Mon Sep 17 00:00:00 2001
From: Axolotle <nchesnais@mailoo.org>
Date: Fri, 24 Mar 2023 14:09:47 +0100
Subject: [PATCH 3/3] AppInfo: ask for error report in config panel

Co-authored-by: Alexandre Aubin <alex.aubin@mailoo.org>
---
 app/src/i18n/locales/en.json  | 3 ++-
 app/src/views/app/AppInfo.vue | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/src/i18n/locales/en.json b/app/src/i18n/locales/en.json
index cb1652a3..1cfe4502 100644
--- a/app/src/i18n/locales/en.json
+++ b/app/src/i18n/locales/en.json
@@ -76,7 +76,8 @@
       "info": {
         "forum": "Search or ask the forum!",
         "problem": "A problem with this app?",
-        "config_panel_error": "An error prevents the configuration panel from being displayed:"
+        "config_panel_error": "An error prevents the configuration panel from being displayed:",
+        "config_panel_error_please_report": "Please report this error to the YunoHost team to get it fixed!"
       },
       "install": {
         "license": "License: {license}",
diff --git a/app/src/views/app/AppInfo.vue b/app/src/views/app/AppInfo.vue
index b1c314d2..8f22236b 100644
--- a/app/src/views/app/AppInfo.vue
+++ b/app/src/views/app/AppInfo.vue
@@ -98,6 +98,7 @@
     >
       <p>{{ $t('app.info.config_panel_error') }}</p>
       <p>{{ config_panel_err }}</p>
+      <p>{{ $t('app.info.config_panel_error_please_report') }}</p>
     </yuno-alert>
 
     <!-- BASIC INFOS -->