release 2023.11.04

CHANGELOG.md

@@ -1,3 +1,28 @@
+pacKman 2023.11.04
+===========================================================
+* Require Dotclear 2.28
+* Require PHP 8.1+
+* Fix typo
+* Code review
+
+pacKman 2023.10.19
+===========================================================
+* Require Dotclear 2.28
+* Require PHP 8.1+
+* cosmetic code review
+
+pacKman 2023.10.13
+===========================================================
+* Require Dotclear 2.28
+* Require PHP 8.1+
+* Upgrade to last minute change to Dotclear 2.28
+
+pacKman 2023.10.09
+===========================================================
+* Require Dotclear 2.28
+* Require PHP 8.1+
+* Code review
+
 pacKman 2023.10.07
 ===========================================================
 * Require Dotclear 2.28

README.md

@@ -1,28 +1,24 @@
 # README
 
-[![Release](https://img.shields.io/badge/release-2023.10.07-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/pacKman/releases)
+[![Release](https://img.shields.io/badge/release-2023.11.04-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/pacKman/releases)
-[![Date](https://img.shields.io/badge/date-2023.10.07-c44d58.svg)](https://git.dotclear.watch/JcDenis/pacKman/releases)
+![Date](https://img.shields.io/badge/date-2023.11.04-c44d58.svg)
 [![Dotclear](https://img.shields.io/badge/dotclear-v2.28-137bbb.svg)](https://fr.dotclear.org/download)
 [![Dotaddict](https://img.shields.io/badge/dotaddict-official-9ac123.svg)](https://plugins.dotaddict.org/dc2/details/pacKman)
-[![License](https://img.shields.io/github/license/JcDenis/pacKman)](https://git.dotclear.watch/JcDenis/pacKman/blob/master/LICENSE)
+[![License](https://img.shields.io/badge/license-GPL--2.0-ececec.svg)](https://git.dotclear.watch/JcDenis/pacKman/src/branch/master/LICENSE)
 
-## WHAT IS PACKMAN ?
+## ABOUT
 
-_pacKman_ is a plugin for the open-source 
+_pacKman_ is a plugin for the open-source web publishing software called [Dotclear](https://www.dotclear.org).
-web publishing software called Dotclear.
 
-It help super admin to create and manage packages of
+> Create and manage packages of themes and plugins from Dotclear administration pages.
-themes and plugins from Dotclear administration pages.
 
 ## REQUIREMENTS
 
-_pacKman_ requires: 
-
-* Super administrator permissions
 * Dotclear 2.28
 * PHP 8.1+
-* A writable cache directory 
+* Dotclear super administrator permissions
-* A writable directory to put packages. (can be VAR dir)
+* System writable cache directory 
+* System writable directory to put packages. (can be VAR dir)
 
 ## USAGE
 
@@ -37,14 +33,15 @@ Once it's done you can manage your packages from menu
 
 ## LINKS
 
-* License : [GNU GPL v2](https://www.gnu.org/licenses/old-licenses/lgpl-2.0.html)
+* [License](https://git.dotclear.watch/JcDenis/pacKman/src/branch/master/LICENSE)
-* Source & contribution : [Gitea Page](https://git.dotclear.watch/JcDenis/pacKman) or [GitHub Page](https://github.com/JcDenis/pacKman)
+* [Packages & details](https://git.dotclear.watch/JcDenis/pacKman/releases) (or on [Dotaddict](https://plugins.dotaddict.org/dc2/details/pacKman))
-* Packages & details: [Gitea Page](https://git.dotclear.watch/JcDenis/pacKman/releases) or [Dotaddict Page](https://plugins.dotaddict.org/dc2/details/pacKman)
+* [Sources & contributions](https://git.dotclear.watch/JcDenis/pacKman) (or on [GitHub](https://github.com/JcDenis/pacKman))
-* Discuss and help : [Dotclear Forum](https://forum.dotclear.org/viewtopic.php?id=40066)
+* [Issues & security](https://git.dotclear.watch/JcDenis/pacKman/issues) (or on [GitHub](https://github.com/JcDenis/pacKman/issues))
+* [Discuss & help](https://forum.dotclear.org/viewtopic.php?id=40066)
 
 ## CONTRIBUTORS
 
-* Jean-Christian Denis
+* Jean-Christian Denis (author)
 * Philippe aka Dissitou
 * franck-paul
 

_define.php

@@ -1,32 +1,26 @@
 <?php
 /**
- * @brief pacKman, a plugin for Dotclear 2
+ * @file
+ * @brief       The plugin pacKman definition
+ * @ingroup     pacKman
  *
- * @package Dotclear
+ * @defgroup    pacKman Plugin pacKman.
- * @subpackage Plugin
  *
- * @author Jean-Christian Denis
+ * Manage your Dotclear packages.
  *
- * @copyright Jean-Christian Denis
+ * @author      Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
  */
 declare(strict_types=1);
 
-if (!defined('DC_RC_PATH')) {
-    return null;
-}
-
 $this->registerModule(
     'Packages repository',
     'Manage your Dotclear packages',
     'Jean-Christian Denis',
-    '2023.10.07',
+    '2023.11.04',
     [
-        'requires' => [
+        'requires'    => [['core', '2.28']],
-            ['php', '8.1'],
+        'permissions' => 'My',
-            ['core', '2.28'],
-        ],
-        'permissions' => null,
         'type'        => 'plugin',
         'support'     => 'https://git.dotclear.watch/JcDenis/' . basename(__DIR__) . '/issues',
         'details'     => 'https://git.dotclear.watch/JcDenis/' . basename(__DIR__) . '/src/branch/master/README.md',

dcstore.xml

@@ -2,10 +2,10 @@
 <modules xmlns:da="http://dotaddict.org/da/">
   <module id="pacKman">
     <name>Packages repository</name>
-    <version>2023.10.07</version>
+    <version>2023.11.04</version>
     <author>Jean-Christian Denis</author>
     <desc>Manage your Dotclear packages</desc>
-    <file>git.dotclear.watch/JcDenis/pacKman/releases/download/v2023.10.07/plugin-pacKman.zip</file>
+    <file>https://git.dotclear.watch/JcDenis/pacKman/releases/download/v2023.11.04/plugin-pacKman.zip</file>
     <da:dcmin>2.28</da:dcmin>
     <da:details>https://git.dotclear.watch/JcDenis/pacKman/src/branch/master/README.md</da:details>
     <da:support>https://git.dotclear.watch/JcDenis/pacKman/issues</da:support>

locales/fr/resources.php

@@ -1,19 +1,10 @@
 <?php
 /**
- * @brief pacKman, a plugin for Dotclear 2
+ * @file
+ * @brief       The plugin pacKman locales resources
+ * @ingroup     pacKman
  *
- * @package Dotclear
+ * @author      Jean-Christian Denis
- * @subpackage Plugin
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
  */
-if (!defined('DC_RC_PATH')) {
+\Dotclear\App::backend()->resources()->set('help', 'pacKman', __DIR__ . '/help/help.html');
-    return null;
-}
-
-if (!isset(dcCore::app()->resources['help']['pacKman'])) {
-    dcCore::app()->resources['help']['pacKman'] = __DIR__ . '/help/help.html';
-}

src/Backend.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -18,6 +8,13 @@ use Dotclear\App;
 use Dotclear\Core\Process;
 use Dotclear\Core\Backend\Favorites;
 
+/**
+ * @brief       pacKman backend class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Backend extends Process
 {
     public static function init(): bool

src/Config.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -30,6 +20,13 @@ use Dotclear\Helper\Html\Form\{
 };
 use Exception;
 
+/**
+ * @brief       pacKman configuration class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Config extends Process
 {
     public static function init(): bool
@@ -62,7 +59,7 @@ class Config extends Process
             Notices::addSuccessNotice(
                 __('Configuration has been successfully updated.')
             );
-            App::backend()->url->redirect('admin.plugins', [
+            App::backend()->url()->redirect('admin.plugins', [
                 'module' => My::id(),
                 'conf'   => '1',
                 'redir'  => App::backend()->__get('list')->getRedir(),
@@ -114,7 +111,7 @@ class Config extends Process
                 // pack_repository
                 (new Para())->items([
                     (new Label($check_repo . __('Path to repository:')))->for('pack_repository'),
-                    (new Input('pack_repository'))->class('maximal')->size(65)->maxlenght(255)->value($s->pack_repository),
+                    (new Input('pack_repository'))->class('maximal')->size(65)->maxlength(255)->value($s->pack_repository),
                 ]),
                 (new Note())->class('form-note')->text(
                     sprintf(
@@ -134,13 +131,13 @@ class Config extends Process
                 // pack_filename
                 (new Para())->items([
                     (new Label($check_first . __('Name of exported package:')))->for('pack_filename'),
-                    (new Input('pack_filename'))->class('maximal')->size(65)->maxlenght(255)->value($s->pack_filename),
+                    (new Input('pack_filename'))->class('maximal')->size(65)->maxlength(255)->value($s->pack_filename),
                 ]),
                 (new Note())->text(sprintf(__('Preconization: %s'), '%type%-%id%'))->class('form-note'),
                 // secondpack_filename
                 (new Para())->items([
                     (new Label($check_second . __('Name of second exported package:')))->for('secondpack_filename'),
-                    (new Input('secondpack_filename'))->class('maximal')->size(65)->maxlenght(255)->value($s->secondpack_filename),
+                    (new Input('secondpack_filename'))->class('maximal')->size(65)->maxlength(255)->value($s->secondpack_filename),
                 ]),
                 (new Note())->text(sprintf(__('Preconization: %s'), '%type%-%id%-%version%'))->class('form-note'),
                 // pack_overwrite
@@ -153,7 +150,7 @@ class Config extends Process
                 // pack_excludefiles
                 (new Para())->items([
                     (new Label(__('Extra files to exclude from package:')))->for('pack_excludefiles'),
-                    (new Input('pack_excludefiles'))->class('maximal')->size(65)->maxlenght(255)->value($s->pack_excludefiles),
+                    (new Input('pack_excludefiles'))->class('maximal')->size(65)->maxlength(255)->value($s->pack_excludefiles),
                 ]),
                 (new Note())->text(sprintf(__('Preconization: %s'), '*.zip,*.tar,*.tar.gz'))->class('form-note'),
                 // pack_nocomment

src/Core.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -21,8 +11,22 @@ use Dotclear\Helper\File\Zip\Unzip;
 use Dotclear\Module\ModuleDefine;
 use Exception;
 
+/**
+ * @brief       pacKman main class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Core
 {
+    /**
+     * Quote excluded list.
+     *
+     * @param   array<int,string>   $exclude    The list
+     *
+     * @return  array<int,string>   $exclude    The list
+     */
     public static function quote_exclude(array $exclude): array
     {
         foreach ($exclude as $k => $v) {
@@ -36,6 +40,13 @@ class Core
         return $exclude;
     }
 
+    /**
+     * Get packages list.
+     *
+     * @param   string  $root   The path to scan
+     *
+     * @return  array<int,ModuleDefine>     The list
+     */
     public static function getPackages(string $root): array
     {
         $res = [];
@@ -134,6 +145,19 @@ class Core
         return $res;
     }
 
+    /**
+     * Pack a module.
+     *
+     * @param   ModuleDefine        $define         The module
+     * @param   string              $root           The package path
+     * @param   array<int,string>   $files          The files to pack
+     * @param   bool                $overwrite      Overwrite package
+     * @param   array<int,string>   $exclude        The files to exclude (regexp)
+     * @param   bool                $nocomment      Remove comment from files
+     * @param   bool                $fixnewline     Fix new lines in files
+     *
+     * @return  bool    True on success
+     */
     public static function pack(ModuleDefine $define, string $root, array $files, bool $overwrite = false, array $exclude = [], bool $nocomment = false, bool $fixnewline = false): bool
     {
         // check define

src/Install.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -18,6 +8,13 @@ use Dotclear\App;
 use Dotclear\Core\Process;
 use Exception;
 
+/**
+ * @brief       pacKman install class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Install extends Process
 {
     public static function init(): bool

src/Manage.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -28,6 +18,13 @@ use Dotclear\Helper\Html\Form\{
 use Dotclear\Helper\Network\Http;
 use Exception;
 
+/**
+ * @brief       pacKman manage page class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Manage extends Process
 {
     public static function init(): bool
@@ -184,11 +181,8 @@ class Manage extends Process
                     # --BEHAVIOR-- packmanBeforeInstallPackage
                     App::behavior()->callBehavior('packmanBeforeInstallPackage', $type, $id, $root);
 
-                    if ($type == 'themes') {
+                    $mods = $type == 'themes' ? App::themes() : App::plugins();
-                        App::themes()->installPackage($root, App::themes());
+                    $mods->installPackage($root, $mods);
-                    } else {
-                        App::plugins()->installPackage($root, App::plugins());
-                    }
 
                     # --BEHAVIOR-- packmanAfterInstallPackage
                     App::behavior()->callBehavior('packmanAfterInstallPackage', $type, $id, $root);

src/My.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -18,11 +8,19 @@ use Dotclear\App;
 use Dotclear\Module\MyPlugin;
 
 /**
- * This module definitions.
+ * @brief       pacKman My plugin helper.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
  */
 class My extends MyPlugin
 {
-    /** @var    array   Excluded files */
+    /**
+     * Excluded files.
+     *
+     * @var    array<int,string>    EXCLUDED_FILES
+     */
     public const EXCLUDED_FILES = [
         '.',
         '..',
@@ -38,6 +36,10 @@ class My extends MyPlugin
 
     public static function checkCustomContext(int $context): ?bool
     {
-        return in_array($context, [My::BACKEND, My::MANAGE, My::MENU]) ? App::auth()->isSuperAdmin() : null;
+        return match ($context) {
+            // Limit to super admin
+            self::MODULE => App::auth()->isSuperAdmin(),
+            default      => null,
+        };
     }
 }

src/Settings.php

@@ -1,73 +1,113 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
 
+/**
+ * @brief       pacKman settings class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Settings
 {
-    // Remove comments from files
+    /**
+     * Remove comments from files.
+     *
+     * @var     bool   $pack_nocomment
+     */
     public readonly bool $pack_nocomment;
 
-    // Remove comments from files
+    /**
+     * Remove comments from files.
+     *
+     * @var     bool    $pack_fixnewline
+     */
     public readonly bool $pack_fixnewline;
 
-    // Overwrite existing package
+    /**
+     * Overwrite existing package.
+     *
+     * @var     bool    $pack_overwrite
+     */
     public readonly bool $pack_overwrite;
 
-    // Name of package
+    /**
+     * Name of package.
+     *
+     * @var     string  $pack_filename
+     */
     public readonly string $pack_filename;
 
-    // Name of second package
+    /**
+     * Name of second package.
+     *
+     * @var     string  $secondpack_filename
+     */
     public readonly string $secondpack_filename;
 
-    // Path to package repository
+    /**
+     * Path to package repository.
+     *
+     * @var     string  $pack_repository
+     */
     public readonly string $pack_repository;
 
-    // Seperate themes and plugins repository
+    /**
+     * Seperate themes and plugins repository.
+     *
+     * @var     bool    $pack_typedrepo
+     */
     public readonly bool $pack_typedrepo;
 
-    // Extra files to exclude from package
+    /**
+     * Extra files to exclude from package.
+     *
+     * @var     string  $pack_excludefiles
+     */
     public readonly string $pack_excludefiles;
 
-    // Hide distributed modules from lists
+    /**
+     * Hide distributed modules from lists.
+     *
+     * @var     bool    $hide_distrib
+     */
     public readonly bool $hide_distrib;
 
     /**
-     * Constructor set up plugin settings
+     * Constructor set up plugin settings.
      */
     public function __construct()
     {
         $s = My::settings();
 
-        $this->pack_nocomment      = (bool) ($s?->get('pack_nocomment') ?? false);
+        $this->pack_nocomment      = (bool) ($s->get('pack_nocomment') ?? false);
-        $this->pack_fixnewline     = (bool) ($s?->get('pack_fixnewline') ?? false);
+        $this->pack_fixnewline     = (bool) ($s->get('pack_fixnewline') ?? false);
-        $this->pack_overwrite      = (bool) ($s?->get('pack_overwrite') ?? false);
+        $this->pack_overwrite      = (bool) ($s->get('pack_overwrite') ?? false);
-        $this->pack_filename       = (string) ($s?->get('pack_filename') ?? '%type%-%id%');
+        $this->pack_filename       = (string) ($s->get('pack_filename') ?? '%type%-%id%');
-        $this->secondpack_filename = (string) ($s?->get('secondpack_filename') ?? '%type%-%id%-%version%');
+        $this->secondpack_filename = (string) ($s->get('secondpack_filename') ?? '%type%-%id%-%version%');
-        $this->pack_repository     = (string) ($s?->get('pack_repository') ?? '');
+        $this->pack_repository     = (string) ($s->get('pack_repository') ?? '');
-        $this->pack_typedrepo      = (bool) ($s?->get('pack_typedrepo') ?? false);
+        $this->pack_typedrepo      = (bool) ($s->get('pack_typedrepo') ?? false);
-        $this->pack_excludefiles   = (string) ($s?->get('pack_excludefiles') ?? '*.zip,*.tar,*.tar.gz,.directory,.hg');
+        $this->pack_excludefiles   = (string) ($s->get('pack_excludefiles') ?? '*.zip,*.tar,*.tar.gz,.directory,.hg');
-        $this->hide_distrib        = (bool) ($s?->get('hide_distrib') ?? false);
+        $this->hide_distrib        = (bool) ($s->get('hide_distrib') ?? false);
     }
 
-    public function getSetting(string $key): mixed
+    /**
+     * Get a setting.
+     *
+     * @param   string  $key    The key
+     *
+     * @return  null|bool|string    The value
+     */
+    public function getSetting(string $key): null|bool|string
     {
         return $this->{$key} ?? null;
     }
 
     /**
-     * Overwrite a plugin settings (in db)
+     * Overwrite a plugin settings (in db).
      *
      * @param   string  $key    The setting ID
      * @param   mixed   $value  The setting value
@@ -87,9 +127,9 @@ class Settings
     }
 
     /**
-     * List defined settings keys
+     * List defined settings keys.
      *
-     * @return  array   The settings keys
+     * @return  array<string,bool|string>   The settings keys
      */
     public function listSettings(): array
     {

src/Uninstall.php

@@ -1,23 +1,19 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
 
-use Dotclear\App;
 use Dotclear\Core\Process;
 use Dotclear\Plugin\Uninstaller\Uninstaller;
 
+/**
+ * @brief       pacKman Uninstaller class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Uninstall extends Process
 {
     public static function init(): bool
@@ -27,7 +23,7 @@ class Uninstall extends Process
 
     public static function process(): bool
     {
-        if (!self::status() || !App::plugins()->moduleExists('Uninstaller')) {
+        if (!self::status()) {
             return false;
         }
 

src/Utils.php

@@ -1,15 +1,5 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
@@ -31,13 +21,21 @@ use Dotclear\Helper\Html\Form\{
     Text
 };
 use Dotclear\Helper\Html\Html;
+use Dotclear\Module\ModuleDefine;
 use Exception;
 
+/**
+ * @brief       pacKman utils class.
+ * @ingroup     pacKman
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Utils
 {
     public static function getPluginsPath(): string
     {
-        $e = explode(PATH_SEPARATOR, DC_PLUGINS_ROOT);
+        $e = explode(PATH_SEPARATOR, App::config()->pluginsRoot());
         $p = array_pop($e);
 
         return (string) Path::real($p);
@@ -86,7 +84,7 @@ class Utils
     public static function getRepositoryDir(?string $dir, ?string $typed = null): string
     {
         $typed = empty($typed) ? '' : DIRECTORY_SEPARATOR . ($typed == 'themes' ? 'themes' : 'plugins');
-        $dir   = empty($dir) ? DC_VAR . DIRECTORY_SEPARATOR . 'packman' . $typed : $dir . $typed;
+        $dir   = empty($dir) ? App::config()->varRoot() . DIRECTORY_SEPARATOR . 'packman' . $typed : $dir . $typed;
 
         try {
             @Files::makeDir($dir, true);
@@ -97,6 +95,15 @@ class Utils
         return $dir;
     }
 
+    /**
+     * Get modules list form.
+     *
+     * @param   array<int|string, mixed>    $modules    The modules
+     * @param   string                      $type       The modules type
+     * @param   string                      $title      The list title
+     *
+     * @return  null|bool   True on render
+     */
     public static function modules(array $modules, string $type, string $title): ?bool
     {
         if (empty($modules)) {
@@ -109,6 +116,9 @@ class Utils
         $tbody = [];
         self::sort($modules);
         foreach ($modules as $module) {
+            if (!is_a($module, ModuleDefine::class)) {
+                continue;
+            }
             $tbody[] = (new Para(null, 'tr'))
                 ->class('line')
                 ->items([
@@ -178,6 +188,15 @@ class Utils
         return true;
     }
 
+    /**
+     * Get modules repository list form.
+     *
+     * @param   array<int,ModuleDefine>     $modules    The modules
+     * @param   string                      $type       The modules type
+     * @param   string                      $title      The list title
+     *
+     * @return  null|bool   True on render
+     */
     public static function repository(array $modules, string $type, string $title): ?bool
     {
         if (empty($modules)) {
@@ -209,7 +228,7 @@ class Utils
         if (str_contains($type, 'repository')) {
             $helpers_addon[] = (new Link())
                 ->class('button')
-                ->href(App::backend()->url->get('admin.plugin.' . My::id(), ['purge' => 1]) . '#packman-repository-' . $type)
+                ->href(App::backend()->url()->get('admin.plugin.' . My::id(), ['purge' => 1]) . '#packman-repository-' . $type)
                 ->text(__('Select non lastest versions'))
             ;
         }
@@ -257,7 +276,7 @@ class Utils
                             (new Text('a', Html::escapeHTML(basename($module->get('root')))))
                                 ->class('packman-download')
                                 ->extra(
-                                    'href="' . App::backend()->url->get('admin.plugin.' . My::id(), [
+                                    'href="' . App::backend()->url()->get('admin.plugin.' . My::id(), [
                                         'package' => basename($module->get('root')),
                                         'repo'    => $type,
                                     ]) . '"'
@@ -323,6 +342,11 @@ class Utils
         return true;
     }
 
+    /**
+     * Sort modules by id.
+     *
+     * @param   array<int,ModuleDefine>     $modules    The modules
+     */
     protected static function sort(array &$modules): void
     {
         uasort($modules, fn ($a, $b) => $a->get('version') <=> $b->get('version'));

src/Zip.php

@@ -1,25 +1,33 @@
 <?php
-/**
+
- * @brief pacKman, a plugin for Dotclear 2
- *
- * @package Dotclear
- * @subpackage Plugin
- *
- * @author Jean-Christian Denis
- *
- * @copyright Jean-Christian Denis
- * @copyright GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
 declare(strict_types=1);
 
 namespace Dotclear\Plugin\pacKman;
 
+/**
+ * @brief       pacKman zip class.
+ * @ingroup     pacKman
+ *
+ * This class extends dotclear zip class
+ * to tweak writeFile method.
+ *
+ * @author      Jean-Christian Denis
+ * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
+ */
 class Zip extends \Dotclear\Helper\File\Zip\Zip
 {
-    /** @var boolean Remove comments from files content */
+    /**
+     * Remove comments from files content.
+     *
+     * @var     bool    $remove_comment
+     */
     public static $remove_comment = false;
 
-    /** @var boolean Fix newline from files content */
+    /**
+     * Fix newline from files content.
+     *
+     * @var     bool    $fix_newline
+     */
     public static $fix_newline = false;
 
     /**
@@ -59,8 +67,8 @@ class Zip extends \Dotclear\Helper\File\Zip\Zip
 
         unset($content);
 
-        $mdate = $this->makeDate($mtime);
+        $mdate = $this->makeDate((int) $mtime);
-        $mtime = $this->makeTime($mtime);
+        $mtime = $this->makeTime((int) $mtime);
 
         # Data descriptor
         $data_desc = "\x50\x4b\x03\x04" .