alpha to beta

.github/workflows/auto-label.yml

@@ -0,0 +1,36 @@
+name: Auto label issues
+
+on:
+  issues:
+    types: [opened]
+
+permissions:
+  issues: write
+
+jobs:
+  auto-label:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Add label based on title
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const title = context.payload.issue.title;
+
+            if (title.includes('[BUG]')) {
+              await github.rest.issues.addLabels({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                issue_number: context.payload.issue.number,
+                labels: ['bug'],
+              });
+            }
+
+            if (title.includes('[FEAT]')) {
+              await github.rest.issues.addLabels({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                issue_number: context.payload.issue.number,
+                labels: ['enhancement'],
+              });
+            }

.github/workflows/phpstan.yaml

@@ -0,0 +1,103 @@
+name: PHPStan
+
+on:
+  push:
+    branches: [ alpha ]
+  pull_request:
+    branches: [ alpha ]
+
+jobs:
+  phpstan:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup PHP 7.4 for dependencies
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: 7.4
+
+      - name: Install Dependencies
+        run: composer update --ignore-platform-reqs
+
+      - name: Setup PHP 8.2 for PHPStan
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: 8.2
+
+      - name: Restore PHPStan cache
+        id: cache-phpstan
+        uses: actions/cache/restore@v3
+        with:
+          path: phpstan.phar
+          key: phpstan-1
+
+      - name: Download PHPStan
+        if: steps.cache-phpstan.outputs.cache-hit != 'true'
+        run: wget https://github.com/phpstan/phpstan/releases/latest/download/phpstan.phar
+
+      - name: Run PHPStan
+        run: php phpstan.phar analyse --configuration phpstan.neon
+
+  update-baseline:
+    needs: phpstan
+    if: github.event_name == 'push' && github.ref == 'refs/heads/alpha'
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Delete existing branch if exists
+        run: |
+          if git ls-remote --heads origin update-phpstan-baseline | grep update-phpstan-baseline; then
+            git push origin --delete update-phpstan-baseline
+          fi
+
+      - name: Setup PHP 7.4 for dependencies
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: 7.4
+
+      - name: Install Dependencies
+        run: composer update --ignore-platform-reqs
+
+      - name: Setup PHP 8.2 for PHPStan
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: 8.2
+
+      - name: Download PHPStan
+        run: wget https://github.com/phpstan/phpstan/releases/latest/download/phpstan.phar
+
+      - name: Generate new baseline
+        id: generate-baseline
+        run: |
+          cp phpstan-baseline.neon phpstan-baseline.neon.old
+          php phpstan.phar analyse --configuration phpstan.neon --generate-baseline
+          if ! diff -q phpstan-baseline.neon phpstan-baseline.neon.old > /dev/null; then
+            echo "baseline_changed=true" >> $GITHUB_OUTPUT
+          fi
+
+      - name: Create Pull Request
+        if: steps.generate-baseline.outputs.baseline_changed == 'true'
+        uses: peter-evans/create-pull-request@v5
+        with:
+          commit-message: Update PHPStan baseline
+          title: '[CI] Update PHPStan baseline'
+          body: |
+            Mise à jour automatique du baseline PHPStan suite à la correction d'erreurs.
+
+            Cette PR a été générée automatiquement par le workflow CI/CD.
+          branch: update-phpstan-baseline
+          base: alpha
+          delete-branch: true
+          add-paths: |
+            phpstan-baseline.neon
+          reviewers: |
+            zoic21
+            Hotfirenet
+            tmartinez69009
+            Salvialf
+            Sekiro-kost
+            reineabs

.gitignore

@@ -45,3 +45,5 @@ tmp/*
 
 !.htaccess
 .env
+.phpstan.cache
+phpstan.phar

core/ajax/recovery.ajax.php

@@ -0,0 +1,51 @@
+<?php
+
+/* This file is part of Jeedom.
+*
+* Jeedom is free software: you can redistribute it and/or modify
+* it under the terms of the GNU General Public License as published by
+* the Free Software Foundation, either version 3 of the License, or
+* (at your option) any later version.
+*
+* Jeedom is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+* GNU General Public License for more details.
+*
+* You should have received a copy of the GNU General Public License
+* along with Jeedom. If not, see <http://www.gnu.org/licenses/>.
+*/
+
+try {
+
+	require_once __DIR__ . '/../php/core.inc.php';
+	include_file('core', 'authentification', 'php');
+
+	if (!isConnect('admin')) {
+		throw new Exception(__('401 - Accès non autorisé', __FILE__), -1234);
+	}
+
+	ajax::init();
+
+	if (init('action') == 'start') {
+		ajax::success(recovery::start(init('hardware'), init('mode')));
+	}
+
+	if (init('action') == 'cancel') {
+		recovery::cancel();
+		ajax::success();
+	}
+
+	if (init('action') == 'getProgress') {
+		ajax::success(recovery::getProgress());
+	}
+
+	if (init('action') == 'usbConnected') {
+		ajax::success(recovery::usbConnected(init('hardware')));
+	}
+
+	throw new Exception(__('Aucune méthode correspondante à :', __FILE__) . ' ' . init('action'));
+	/*     * *********Catch exeption*************** */
+} catch (Exception $e) {
+	ajax::error(displayException($e), $e->getCode());
+}

core/ajax/user.ajax.php

@@ -169,6 +169,14 @@
 			}
 			utils::a2o($user, $user_json);
 			$user->save();
+			if (isset($user_json['enable']) && $user_json['enable'] == 0) {
+        			$sessions = listSession();
+                		foreach ($sessions as $sessionId => $sessionData) {
+					if (isset($sessionData['user_id']) && $sessionData['user_id'] == $user->getId()) {
+                        			deleteSession($sessionId);
+                    			}
+                		}
+    			}
 		}
 		@session_start();
 		$_SESSION['user']->refresh();

core/ajax/view.ajax.php

@@ -171,13 +171,15 @@
 			}
 			$sql .= 'UPDATE viewData SET `order`= ' . $component['viewOrder'] . '  WHERE link_id=' . $component['id'] . ' AND type="' . $component['type'] . '" AND  viewZone_id=' . $component['viewZone_id'] . ';';
 			if ($component['type'] == 'eqLogic') {
+				unset($component['type']);
 				$eqLogic = eqLogic::byId($component['id']);
 				if (!is_object($eqLogic)) {
 					continue;
 				}
 				utils::a2o($eqLogic, $component);
 				$eqLogic->save(true);
 			} elseif ($component['type'] == 'scenario') {
+				unset($component['type']);
 				$scenario = scenario::byId($component['id']);
 				if (!is_object($scenario)) {
 					continue;

core/class/DB.class.php

@@ -129,6 +129,10 @@ public static function &Prepare($_query, $_params, $_fetchType = self::FETCH_TYP
 							$obj->setChanged(false);
 						}
 					}
+
+                    if (is_object($obj) && method_exists($obj, 'initialize')) {
+                        $obj->initialize();
+                    }
 				}
 			} else {
 				if (is_object($res) && method_exists($res, 'decrypt')) {
@@ -137,6 +141,10 @@ public static function &Prepare($_query, $_params, $_fetchType = self::FETCH_TYP
 						$res->setChanged(false);
 					}
 				}
+
+                if (is_object($res) && method_exists($res, 'initialize')) {
+                    $res->initialize();
+                }
 			}
 		}
 		return $res;

core/class/ajax.class.php

@@ -20,59 +20,26 @@
 require_once __DIR__ . '/../../core/php/core.inc.php';
 
 /**
- * Gère les réponses AJAX de Jeedom
+ * Handles AJAX responses in Jeedom
  *
- * @note Évolutions possibles et compatibles
- *   Cette classe pourrait être enrichie progressivement avec :
- *   - Une interface ResponseFormatterInterface pour supporter différents formats (json, xml...)
- *   - Un système de middleware pour la validation des entrées
- *   - Des codes d'erreur HTTP standards via une énumération
- *   Ces changements peuvent être implémentés graduellement sans casser l'existant
+ * @example
+ * ajax::init(['getInfos']); // returns void
+ * ajax::success(['result' => 'ok']); // sends JSON response
  *
- * @example Utilisation actuelle
- * ```php
- * ajax::init(['getInfos']);
- * ajax::success($data);
- * ```
- *
- * @example Utilisation future possible
- * ```php
- * // Même API, plus de fonctionnalités
- * ajax::init(['getInfos'])
- *    ->withValidator(new InputValidator())
- *    ->withFormat(new JsonFormatter());
- * ajax::success($data);
- * ```
- *
- * @see config::class Pour la gestion des configurations
- * @see log::class Pour la gestion des logs
- *
- * @todo Version 4.6 ou 5.0
- *   - [OPTIONNEL] Ajouter un système de middleware pour valider les entrées
- *   - [OPTIONNEL] Support de différents formats via interfaces
- *   - [COMPATIBLE] Utiliser des codes HTTP standards
+ * @see config::class For configuration management
+ * @see log::class For logging management
  */
 class ajax {
 	/*     * *************************Attributs****************************** */
 
 	/*     * *********************Methode static ************************* */
 
     /**
-     * Initialise la réponse AJAX
-     * Configure les en-têtes HTTP et vérifie les actions autorisées en GET
+     * Initializes AJAX response with HTTP headers and GET action validation
      *
-     * @param array $_allowGetAction Liste des actions autorisées en GET
+     * @param string[] $_allowGetAction List of allowed GET actions
      * @return void
-     * @throws \Exception Si l'action demandée en GET n'est pas autorisée
-     *
-     * @note Évolution possible
-     * Cette méthode pourrait retourner $this pour permettre le chaînage :
-     * ```php
-     * ajax::init(['action'])
-     *     ->withValidator()
-     *     ->withFormat();
-     * ```
-     * Ce changement serait rétrocompatible
+     * @throws \Exception When requested GET action is not allowed
      */
 	public static function init($_allowGetAction = array()) {
 		if (!headers_sent()) {
@@ -84,38 +51,31 @@ public static function init($_allowGetAction = array()) {
 	}
 
     /**
-     * Retourne un token (méthode non utilisée ?)
+     * Returns authentication token
      *
-     * @return string Token vide
+     * @deprecated Since version 4.4, authentication is handled differently
+     * @return string Empty token
      */
 	public static function getToken(){
 		return '';
 	}
 
     /**
-     * Envoie une réponse de succès et termine l'exécution
+     * Sends a success response and ends execution
      *
-     * @param mixed $_data Données à renvoyer dans la réponse
+     * @param mixed $_data Data to send in response
      * @return never
-     *
-     * @note Compatibilité et évolution
-     * Pour maintenir la compatibilité tout en permettant l'évolution :
-     * - Garder le comportement actuel par défaut
-     * - Permettre l'injection d'un formatter optionnel
-     * ```php
-     * ajax::success($data, new JsonFormatter()); // Optionnel
-     * ```
      */
 	public static function success($_data = '') {
 		echo self::getResponse($_data);
 		die();
 	}
 
     /**
-     * Envoie une réponse d'erreur et termine l'exécution
+     * Sends an error response and ends execution
      *
-     * @param mixed $_data Message d'erreur ou données à renvoyer
-     * @param int $_errorCode Code d'erreur
+     * @param mixed $_data Error message or data to send
+     * @param int $_errorCode Custom error code for client-side handling (default: 0)
      * @return never
      */
 	public static function error($_data = '', $_errorCode = 0) {
@@ -124,19 +84,11 @@ public static function error($_data = '', $_errorCode = 0) {
 	}
 
     /**
-     * Génère la réponse JSON formatée
-     *
-     * @param mixed $_data Données à inclure dans la réponse
-     * @param ?int $_errorCode Code d'erreur (null pour une réponse de succès)
-     * @return string Réponse JSON encodée
+     * Generates formatted JSON response
      *
-     * @note Architecture future
-     * Cette méthode pourrait déléguer le formatage à des classes dédiées :
-     * - JsonFormatter (comportement actuel)
-     * - XmlFormatter
-     * - CsvFormatter
-     * etc.
-     * La transition peut se faire graduellement en gardant le comportement par défaut
+     * @param mixed $_data Data to include in response
+     * @param ?int $_errorCode Error code (null for success response)
+     * @return string|false Encoded JSON response
      */
 	public static function getResponse($_data = '', $_errorCode = null) {
 		$isError = !(null === $_errorCode);