From e2bd001a8a4d5048723318ab3d10dfeb4e37116e Mon Sep 17 00:00:00 2001
From: asus <asus@example.com>
Date: Sun, 11 Feb 2024 19:46:40 +0100
Subject: [PATCH] dont add ajax script multiple times

---
 plugins/wp_model_plugin/php/menu/menu.php        |  4 ++++
 .../wp_model_plugin/php/utils/add_to_front.php   | 16 ++++++++++++----
 plugins/wp_model_plugin/plugin_hooks.php         |  4 ----
 3 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/plugins/wp_model_plugin/php/menu/menu.php b/plugins/wp_model_plugin/php/menu/menu.php
index fa806b1..3b428f3 100644
--- a/plugins/wp_model_plugin/php/menu/menu.php
+++ b/plugins/wp_model_plugin/php/menu/menu.php
@@ -1,6 +1,10 @@
 <?php
 
 function wp_model_plugin_content() {
+	global $first_script;
+	console_log("in menu : first_script:");
+	console_log($first_script);
+
 	add_files_to_front( array(
 		"menu/menu.js",
 	));
diff --git a/plugins/wp_model_plugin/php/utils/add_to_front.php b/plugins/wp_model_plugin/php/utils/add_to_front.php
index 28f86dd..05e892e 100644
--- a/plugins/wp_model_plugin/php/utils/add_to_front.php
+++ b/plugins/wp_model_plugin/php/utils/add_to_front.php
@@ -5,12 +5,21 @@
 /*
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 function that add the ajax script to front
+no need to check if already included, because $handle is uniq
+so it will not be recincluded, the first enqueued version will be kept
 */
 function add_ajax_post() {
 	global $first_script;
 	global $ajax_file;
 
 	$file = init_file($ajax_file);
+
+	// check if ajax script was already enqueued
+	global $wp_scripts;
+	$already_enqueued = array_search($file->handle, $wp_scripts->queue);
+	if ($already_enqueued !== false)
+		return ;
+
 	$first_script = $file->handle;
 	wp_enqueue_script( $file->handle, $file->url, $previous_js_basename, $file->version, true);
 	$ajax_url = admin_url( 'admin-ajax.php' );
@@ -25,6 +34,7 @@ function add_ajax_post() {
 
 
 
+
 /*
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 @param string : name of the file, with its path from its extension directory
@@ -121,12 +131,10 @@ function add_var_to_front($vars, $handle = null) {
 	extract($vars);
 	foreach ($vars as $key => $var)
 	{
-		$js_var  = 'const ' . $key . ' = ';
-		$js_var .= json_encode($var);
-		$js_var .= ';';
+		$js_var = 'let '.$key.' = '.json_encode($var).';';
 		wp_add_inline_script($handle, $js_var, 'before');
 	}
-//
+
 //	the other way with localize has multiple incidences :
 //  - it creates an object from wich you can access the variables
 //  - so if you call it again wiht the same name, it will overwrite the previous
diff --git a/plugins/wp_model_plugin/plugin_hooks.php b/plugins/wp_model_plugin/plugin_hooks.php
index b787f1b..020aec1 100644
--- a/plugins/wp_model_plugin/plugin_hooks.php
+++ b/plugins/wp_model_plugin/plugin_hooks.php
@@ -70,10 +70,6 @@ function main_shortcode() {
 		)
 	);
 
-	global $wp_scripts;
-	console_log("wp_scripts->queue: ");
-	console_log($wp_scripts->queue);
-
 	return $html_front;
 }
 add_shortcode('wp_model_plugin', 'main_shortcode');