Les scripts js ne sont plus systématiquement inclus dans <head> (trop de...

Les scripts js ne sont plus systématiquement inclus dans <head> (trop de requêtes intempestives sur toutes les pages) mais en "inline" par l'aide de vue IdrefPopupTriggerViewHelper si besoin.

Les scripts js ne sont plus systématiquement inclus dans <head> (trop de...
48cde2db · Bertrand Gauthier · ce77ba4e · 48cde2db · 48cde2db · 48cde2db
Commit 48cde2db authored 2 months ago by Bertrand Gauthier
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
 Journal des modifications
 =========================

+2.2.0
+-----
+- Les scripts js ne sont plus systématiquement inclus dans <head> (trop de requêtes intempestives sur toutes les pages) 
+  mais en "inline" par l'aide de vue IdrefPopupTriggerViewHelper si besoin.
+
 2.1.0
 -----
 - Suppression de la dépendance avec laminas/laminas-dependency-plugin.

--- a/config/module.config.php
+++ b/config/module.config.php
@@ -89,11 +89,11 @@ return [
     * Les chemins ci-dessous sont donc relatifs au dossier racine de l'appli.
     */
    'public_files' => [
-        'head_scripts' => [
-            '099_unicaen-idref_js_1' => '/unicaen/idref/js/formulaire.js',
-            '099_unicaen-idref_js_2' => '/unicaen/idref/js/subModal.js',
-            '099_unicaen-idref_js_3' => '/unicaen/idref/js/trigger.js',
-        ],
+//        'head_scripts' => [
+//            '099_unicaen-idref_js_1' => '/unicaen/idref/js/formulaire.js',
+//            '099_unicaen-idref_js_2' => '/unicaen/idref/js/subModal.js',
+//            '099_unicaen-idref_js_3' => '/unicaen/idref/js/trigger.js',
+//        ],
        'stylesheets' => [
            '099_unicaen-idref_css_1' => '/unicaen/idref/css/subModal.css',
            '099_unicaen-idref_css_2' => '/unicaen/idref/css/trigger.css',

--- a/src/UnicaenIdref/View/Helper/IdrefPopupTriggerViewHelper.php
+++ b/src/UnicaenIdref/View/Helper/IdrefPopupTriggerViewHelper.php
@@ -62,6 +62,8 @@ class IdrefPopupTriggerViewHelper extends AbstractHelper
        $this->sourceElements = $sourceElements;
        $this->destinationElement = $destinationElement;

+        $this->attachInlineScripts();
+
        return $this;
    }

@@ -81,4 +83,26 @@ class IdrefPopupTriggerViewHelper extends AbstractHelper
            'params' => $this->params,
        ]);
    }
+
+    /**
+     * Inscrit les scripts js "inline" nécessaires si ce n'est pas déjà fait.
+     */
+    protected function attachInlineScripts(): void
+    {
+        /** @var \Laminas\View\Helper\InlineScript $helper */
+        $helper = $this->view->plugin('inlineScript');
+
+        $prefix = __NAMESPACE__ . '\\';
+
+        $scripts = [
+            1 => 'unicaen/idref/js/formulaire.js',
+            2 => 'unicaen/idref/js/subModal.js',
+            3 => 'unicaen/idref/js/trigger.js',
+        ];
+        foreach ($scripts as $i => $script) {
+            if (!$helper->offsetExists($offset = $prefix . $i)) {
+                $helper->offsetSetFile($offset, $this->view->basePath($script));
+            }
+        }
+    }
 }
\ No newline at end of file