Adaptations suite à la remarque de Loïc

Author: MrWaloo

core/class/script.class.php

@@ -228,16 +228,28 @@ public function execute($_options = null) {
 				if ($this->getType() == 'info' && isset(script::$_requet_cache[$request])) {
 					return script::$_requet_cache[$request];
 				}
-				$cmd = 'sudo chmod +x ' . explode(' ', $request)[0] . ' 2>/dev/null;';
-				$use_shebang = $this->getConfiguration('useShebang', '0') == '1';
-				if (!$use_shebang && strpos($request, '.php') !== false) {
+				$first_element = explode(' ', $request)[0];
+				$use_shebang = false;
+				$from_path = false;
+				if (file_exists($first_element)) {
+					$env_path = explode(PATH_SEPARATOR, getenv('PATH'));
+					$from_path = in_array(dirname($first_element), $env_path);
+					if (!$from_path) {
+						$shebang = trim(file_get_contents($first_element, false, null, 0, 3));
+						$use_shebang = $shebang == '#!/';
+					}
+					if (!is_executable($first_element)) {
+						$cmd = 'sudo chmod +x ' . $first_element . ' 2>/dev/null;';
+					}
+				}
+				if (!$from_path && !$use_shebang && substr($request, -4) === '.php') {
 					$cmd .= 'php ' . $request;
-				} elseif (!$use_shebang && strpos($request, '.rb') !== false) {
-					$cmd .= 'ruby ' . $request;
-				} elseif (!$use_shebang && strpos($request, '.py') !== false) {
-					$cmd .= 'python ' . $request;
-				} elseif (!$use_shebang && strpos($request, '.pl') !== false) {
+				} elseif (!$from_path && !$use_shebang && substr($request, -3) === '.pl') {
 					$cmd .= 'perl ' . $request;
+				} elseif (!$from_path && !$use_shebang && substr($request, -3) === '.py') {
+					$cmd .= 'python ' . $request;
+				} elseif (!$from_path && !$use_shebang && substr($request, -3) === '.rb') {
+					$cmd .= 'ruby ' . $request;
 				} else {
 					$cmd .= $request;
 				}

desktop/js/script.js

@@ -33,9 +33,15 @@ $("#table_cmd tbody").delegate(".cmdAttr[data-l1key=configuration][data-l2key=re
   if ($(this).value() == 'script') {
     $(this).closest('tr').find('.browseScriptFile').show()
     $(this).closest('tr').find('.editScriptFile').show()
+
+    $(this).closest('tr').find('.tdRequest').attr('colspan', '2')
+    $(this).closest('tr').find('.tdOptions').hide()
   } else {
     $(this).closest('tr').find('.browseScriptFile').hide()
     $(this).closest('tr').find('.editScriptFile').hide()
+
+    $(this).closest('tr').find('.tdRequest').attr('colspan', '1')
+    $(this).closest('tr').find('.tdOptions').show()
   }
 })
 
@@ -100,7 +106,7 @@ function addCmdToTable(_cmd) {
   tr += '<span class="subType" subType="' + init(_cmd.subType) + '"></span>'
   tr += '</td>'
 
-  tr += '<td class="tdRequest">'
+  tr += '<td class="tdRequest" colspan="2">'
   tr += '<div class="btn-group" role="group">'
   tr += '<span class="input-group" style="margin-top : 5px;">'
   tr += '<a style="width:30px" class="btn btn-default browseScriptFile btn-xs roundedLeft" title="{{Parcourir}}"><i class="far fa-folder-open"></i></a>'
@@ -110,14 +116,10 @@ function addCmdToTable(_cmd) {
   tr += '<textarea style="height : 95px;margin-top:5px;" class="cmdAttr form-control input-sm" data-l1key="configuration" data-l2key="request"></textarea>'
   tr += '</td>'
 
-  tr += '<td class="tdOptions">'
-  tr += '<div class="requestTypeConfig" data-type="script">'
-  tr += '<center>'
-  tr += '<span><label class="checkbox-inline"><input type="checkbox" class="cmdAttr" data-l1key="configuration" data-l2key="useShebang"/>{{Interpréteur shebang}}'
-  tr += '<sup><i class="fas fa-question-circle tooltips" title="{{Si cette option est cochée, l\'interpréteur à utiliser est celui de la ligne shebang}}"></i></sup>'
-  tr += '</label></span> '
-  tr += '</center>'
-  tr += '</div>'
+  tr += '<td class="tdOptions" style="display : none;">'
+
+  //tr += '<div class="requestTypeConfig" data-type="script">'
+  //tr += '</div>'
 
   tr += '<div class="requestTypeConfig" data-type="http" style="display : none;">'
   tr += '<center>'
@@ -186,7 +188,8 @@ function addCmdToTable(_cmd) {
   tr += '</div>'
   tr += '</div>'
   tr += '</div>'
-  tr += '</td>'
+
+  tr += '</td>' // tdOptions
 
   tr += '<td>'
   tr += '<label class="checkbox-inline"><input type="checkbox" class="cmdAttr" data-l1key="isVisible" checked/>{{Afficher}}</label> '

docs/fr_FR/changelog.md

@@ -8,8 +8,8 @@
 
 # xx/07/2025
 
-- Ajout de la possibilité de ne pas forcer l'interpréteur selon une règle propre à Jeedom et qui est incompatible avec certains systèmes (python3 sous Debian 11+).
 - Correction de l'erreur d'affichage des options pour un type "HTTP" lors de la création d'une commande alors que le type "script" est sélectionné à ce moment.
+- Traitement différent pour les commandes de type Script : le forçage de l'interpréteur selon une règle propre au plugin et qui est incompatible avec certains systèmes (python3 sous Debian 11+) est conditionné afin que le script soit lancé comme s'il l'était dans un shell sauf si le forçage de l'interpréteur est nécessaire.
 - Mise à jour de la documentation et petites corrections de celle-ci
 
 # 01/07/2024

docs/fr_FR/index.md

@@ -51,7 +51,7 @@ Vous retrouvez ici la liste des commandes :
   - Le bouton **Editer** : permet d’éditer à l’aide d’un éditeur de code interne un des fichiers contenus dans le répertoire permettant l’accès au code du fichier.
   - Le bouton **Nouveau** : permet de créer un fichier de commande.
 
-        > Ne pas oublier de saisir le nom du fichier ainsi que son extension complète sous peine de voir votre superbe script ne pas fonctionner. Sans extension Jeedom ne saura pas reconnaître le langage associé à votre fichier. Il est possible de contourner ce fonctionnement en cochant la case "Interpréteur shebang", ce point est détaillé plus loin.
+        > Ne pas oublier de saisir le nom du fichier ainsi que son extension complète sous peine de voir votre superbe script ne pas fonctionner. Sans extension Jeedom ne saura pas reconnaître le langage associé à votre fichier. Il est possible de contourner ce fonctionnement en précisant l'interpréteur dans la ligne shebang ou en plaçant l'interpréteur en première position de la ligne de commande, ce point est détaillé plus loin.
 
   - Le bouton **Supprimer** : permet de supprimer un fichier de commande.
 
@@ -194,25 +194,24 @@ Action : Lancez le script, via un équipement virtuel, lié à votre script !
 
 Le plus sympa mais pas le plus simple à expliquer.
 
-**Prérequis : savoir développer un script en php, python, perl ou ruby.**
+**Prérequis : savoir développer un script**
 
 >**IMPORTANT**
 >
-> L'extension de votre script doit absolument correspondre à son type. En effet Jeedom se base sur l'extension du script pour l'exécutable à lancer
+> - Si le script en première position de la ligne de commande contient une ligne shebang, le plugin script lancera un shell qui l’exécutera en se basant sur la directive de la 1ère ligne (shebang).
+> - Si le premier élément de la ligne de commande est un exécutable reconnu par le système, par exemple `/usr/bin/python3`, le plugin script lancera un shell qui exécutera cette ligne de commande.
+> - Si le script en première position de la ligne de commande ne contient pas de ligne shebang ET que le premier élément de laligne de commande n'est pas un exécutable reconnu par le système, l'extension de votre script doit absolument correspondre à son type. En effet le plugin script se base alors sur l'extension du script pour l'exécutable à lancer.
 
-Si le nom de votre fichier ne contient pas :
-- .php .py .pl .rb
+Dans ce dernier cas, si le nom de votre fichier ne termine pas par .php .py .pl ou .rb, le plugin script lancera un shell qui l’exécutera comme en ligne de commande.
 
-ou si la case "Interpréteur shebang" est cochée, le plugin script lancera un shell qui l’exécutera en se basant sur la directive de la 1ère ligne (shebang).
-Exemple :
+Exemples de ligne shebang :
 
-```bash
+```
 #!/bin/csh -f
 #!/bin/ksh
 #!/usr/bin/env python3
 #!/usr/bin/env php
 #!/usr/bin/env node
-etc ...
 ```
 
 Le script de monitoring température du Raspberry va servir d’exemple pour l’utilisation du type de script : Script