CHANGELOG.md

@@ -1,10 +1,3 @@
-alias 1.11 - 2023.10.23
-===========================================================
-* Require Dotclear 2.28
-* Require PHP 8.1
-* Use class for alias row
-* Code review
-
 alias 1.10 - 2023.10.15
 ===========================================================
 * require Dotclear 2.28

README.md

@@ -1,7 +1,7 @@
 # README
 
-[![Release](https://img.shields.io/badge/release-1.11-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/alias/releases)
+[![Release](https://img.shields.io/badge/release-1.10-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/alias/releases)
-![Date](https://img.shields.io/badge/date-2023.10.23-c44d58.svg)
+![Date](https://img.shields.io/badge/date-2023.10.15-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/alias)
 [![License](https://img.shields.io/badge/license-GPL--2.0-ececec.svg)](https://git.dotclear.watch/JcDenis/alias/src/branch/master/LICENSE)

_define.php

@@ -18,7 +18,7 @@ $this->registerModule(
     'alias',
     "Create aliases of your blog's URLs",
     'Olivier Meunier and contributors',
-    '1.11',
+    '1.10',
     [
         'requires'    => [['core', '2.28']],
         'permissions' => 'My',

dcstore.xml

@@ -2,10 +2,10 @@
 <modules xmlns:da="http://dotaddict.org/da/">
   <module id="alias">
     <name>alias</name>
-    <version>1.11</version>
+    <version>1.10</version>
     <author>Olivier Meunier and contributors</author>
     <desc>Create aliases of your blog's URLs</desc>
-    <file>https://git.dotclear.watch/JcDenis/alias/releases/download/v1.11/plugin-alias.zip</file>
+    <file>https://git.dotclear.watch/JcDenis/alias/releases/download/v1.10/plugin-alias.zip</file>
     <da:dcmin>2.28</da:dcmin>
     <da:details>https://git.dotclear.watch/JcDenis/alias/src/branch/master/README.md</da:details>
     <da:support>https://git.dotclear.watch/JcDenis/alias/issues</da:support>

src/Alias.php

@@ -31,14 +31,14 @@ class Alias
     /**
      * Stack of aliases.
      *
-     * @var     array<int, AliasRow>    $aliases
+     * @var     array<int, array<string, string>>   $aliases
      */
     protected array $aliases = [];
 
     /**
      * Get aliases.
      *
-     * @return  array<int, AliasRow>    Stack of aliases
+     * @return  array<int, array<string, string>>   Stack of aliases
      */
     public function getAliases(): array
     {
@@ -46,6 +46,10 @@ class Alias
             return $this->aliases;
         }
 
+        if (!App::blog()->isDefined()) {
+            return [];
+        }
+
         $sql = new SelectStatement();
         $rs  = $sql->from(App::con()->prefix() . Alias::ALIAS_TABLE_NAME)
             ->columns([
@@ -58,11 +62,7 @@ class Alias
             ->order('alias_position ASC')
             ->select();
 
-        if (!is_null($rs)) {
+        $this->aliases = is_null($rs) ? [] : $rs->rows();
-            while ($rs->fetch()) {
-                $this->aliases[] = AliasRow::newFromRecord($rs);
-            }
-        }
 
         return $this->aliases;
     }
@@ -70,24 +70,24 @@ class Alias
     /**
      * Update aliases stack.
      *
-     * @param   array<int, AliasRow>    $aliases    The alias stack
+     * @param   array<int, array<string, string>>   $aliases    The alias stack
      */
     public function updateAliases(array $aliases): void
     {
-        foreach ($aliases as $row) {
+        usort($aliases, fn ($a, $b) => (int) $a['alias_position'] <=> (int) $b['alias_position']);
-            if (!is_a($row, AliasRow::class)) {
+        foreach ($aliases as $v) {
+            if (!isset($v['alias_url']) || !isset($v['alias_destination'])) {
                 throw new Exception(__('Invalid aliases definitions'));
             }
         }
-        usort($aliases, fn ($a, $b) => $a->position <=> $b->position);
 
         App::con()->begin();
 
         try {
             $this->deleteAliases();
-            foreach ($aliases as $k => $alias) {
+            foreach ($aliases as $k => $v) {
-                if (!empty($alias->url) && !empty($alias->destination)) {
+                if (!empty($v['alias_url']) && !empty($v['alias_destination'])) {
-                    $this->createAlias(new AliasRow($alias->url, $alias->destination, $k + 1, $alias->redirect));
+                    $this->createAlias($v['alias_url'], $v['alias_destination'], $k + 1, !empty($v['alias_redirect']));
                 }
             }
 
@@ -102,16 +102,19 @@ class Alias
     /**
      * Create an alias.
      *
-     * @param   AliasRow    $alias  The new Alias descriptor
+     * @param   string  $url            The URL
+     * @param   string  $destination    The destination
+     * @param   int     $position       The position
+     * @param   bool    $redirect       Do redirection
      */
-    public function createAlias(AliasRow $alias): void
+    public function createAlias(string $url, string $destination, int $position, bool $redirect): void
     {
         if (!App::blog()->isDefined()) {
             return;
         }
 
-        $url         = self::removeBlogUrl($alias->url);
+        $url         = self::removeBlogUrl($url);
-        $destination = self::removeBlogUrl($alias->destination);
+        $destination = self::removeBlogUrl($destination);
 
         if (empty($url)) {
             throw new Exception(__('Alias URL is empty.'));
@@ -122,10 +125,10 @@ class Alias
 
         $cur = App::con()->openCursor(App::con()->prefix() . Alias::ALIAS_TABLE_NAME);
         $cur->setField('blog_id', App::blog()->id());
-        $cur->setField('alias_url', $url);
+        $cur->setField('alias_url', (string) $url);
-        $cur->setField('alias_destination', $destination);
+        $cur->setField('alias_destination', (string) $destination);
-        $cur->setField('alias_position', $alias->position);
+        $cur->setField('alias_position', abs((int) $position));
-        $cur->setField('alias_redirect', (int) $alias->redirect);
+        $cur->setField('alias_redirect', (int) $redirect);
         $cur->insert();
     }
 
@@ -136,6 +139,10 @@ class Alias
      */
     public function deleteAlias(string $url): void
     {
+        if (!App::blog()->isDefined()) {
+            return;
+        }
+
         $sql = new DeleteStatement();
         $sql->from(App::con()->prefix() . Alias::ALIAS_TABLE_NAME)
             ->where('blog_id = ' . $sql->quote(App::blog()->id()))
@@ -148,6 +155,10 @@ class Alias
      */
     public function deleteAliases(): void
     {
+        if (!App::blog()->isDefined()) {
+            return;
+        }
+
         $sql = new DeleteStatement();
         $sql->from(App::con()->prefix() . Alias::ALIAS_TABLE_NAME)
             ->where('blog_id = ' . $sql->quote(App::blog()->id()))
@@ -163,6 +174,6 @@ class Alias
      */
     public static function removeBlogUrl(string $url): string
     {
-        return str_replace(App::blog()->url(), '', trim($url));
+        return App::blog()->isDefined() ? str_replace(App::blog()->url(), '', trim($url)) : trim($url);
     }
 }

src/AliasRow.php

@@ -1,41 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Dotclear\Plugin\alias;
-
-use Dotclear\Database\MetaRecord;
-
-/**
- * @brief       alias decriptor class.
- * @ingroup     alias
- *
- * @author      Olivier Meunier (author)
- * @author      Jean-Christian Denis (latest)
- * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
-class AliasRow
-{
-    public readonly string $url;
-    public readonly string $destination;
-    public readonly int $position;
-    public readonly bool $redirect;
-
-    public function __construct(?string $url, ?string $destination, null|string|int $position, null|int|string|bool $redirect)
-    {
-        $this->url         = (string) $url;
-        $this->destination = (string) $destination;
-        $this->position    = (int) $position;
-        $this->redirect    = !empty($redirect);
-    }
-
-    public static function newFromRecord(MetaRecord $rs): AliasRow
-    {
-        return new self(
-            (string) $rs->field('alias_url'),
-            (string) $rs->field('alias_destination'),
-            (int) $rs->field('alias_position'),
-            !empty($rs->field('alias_redirect'))
-        );
-    }
-}

src/Backend.php

@@ -6,6 +6,11 @@ namespace Dotclear\Plugin\alias;
 
 use Dotclear\App;
 use Dotclear\Core\Process;
+use Dotclear\Plugin\importExport\{
+    FlatBackupItem,
+    FlatExport,
+    FlatImportV2
+};
 
 /**
  * @brief       alias backend class.
@@ -31,10 +36,46 @@ class Backend extends Process
         My::addBackendMenuItem();
 
         App::behavior()->addBehaviors([
-            'exportFullV2'   => PluginImportExportBehaviors::exportFullV2(...),
+            'exportFullV2' => function (FlatExport $exp): void {
-            'importInitV2'   => PluginImportExportBehaviors::importInitV2(...),
+                $exp->exportTable(Alias::ALIAS_TABLE_NAME);
-            'importFullV2'   => PluginImportExportBehaviors::importFullV2(...),
+            },
-            'importSingleV2' => PluginImportExportBehaviors::importSingleV2(...),
+            'exportSingleV2' => function (FlatExport $exp, ?string $blog_id): void {
+                $exp->export(
+                    'alias',
+                    'SELECT alias_url, alias_destination, alias_position ' .
+                    'FROM ' . App::con()->prefix() . Alias::ALIAS_TABLE_NAME . ' A ' .
+                    "WHERE A.blog_id = '" . $blog_id . "'"
+                );
+            },
+            'importInitV2' => function (FlatImportV2 $bk): void {
+                $bk->cur_alias = App::con()->openCursor(App::con()->prefix() . Alias::ALIAS_TABLE_NAME);
+                $bk->alias     = new Alias();
+                $bk->aliases   = $bk->alias->getAliases();
+            },
+            'importFullV2' => function (/*bool|FlatBackupItem */$line, FlatImportV2 $bk): void {
+                if ($line->__name == Alias::ALIAS_TABLE_NAME) {
+                    $bk->cur_alias->clean();
+                    $bk->cur_alias->setField('blog_id', (string) $line->blog_id);
+                    $bk->cur_alias->setField('alias_url', (string) $line->alias_url);
+                    $bk->cur_alias->setField('alias_destination', (string) $line->alias_destination);
+                    $bk->cur_alias->setField('alias_position', (int) $line->alias_position);
+                    $bk->cur_alias->insert();
+                }
+            },
+            'importSingleV2' => function (/*bool|FlatBackupItem */$line, FlatImportV2 $bk): void {
+                if ($line->__name == Alias::ALIAS_TABLE_NAME) {
+                    $found = false;
+                    foreach ($bk->aliases as $v) {
+                        if ($v['alias_url'] == $line->alias_url) {
+                            $found = true;
+                        }
+                    }
+                    if ($found) {
+                        $bk->alias->deleteAlias($line->alias_url);
+                    }
+                    $bk->alias->createAlias($line->alias_url, $line->alias_destination, $line->alias_position);
+                }
+            },
         ]);
 
         return true;

src/Manage.php

@@ -51,22 +51,13 @@ class Manage extends Process
             return true;
         }
 
-        $utils   = new Alias();
+        $alias   = new Alias();
-        $aliases = $utils->getAliases();
+        $aliases = $alias->getAliases();
 
         # Update aliases
         if (isset($_POST['a']) && is_array($_POST['a'])) {
             try {
-                $stack = [];
+                $alias->updateAliases($_POST['a']);
-                foreach ($_POST['a'] as $alias) {
-                    $stack[] = new AliasRow(
-                        $alias['alias_url']         ?? '',
-                        $alias['alias_destination'] ?? '',
-                        (int) ($alias['alias_position'] ?? 0),
-                        !empty($alias['alias_redirect']),
-                    );
-                }
-                $utils->updateAliases($stack);
                 Notices::addSuccessNotice(__('Aliases successfully updated.'));
                 My::redirect();
             } catch (Exception $e) {
@@ -77,7 +68,7 @@ class Manage extends Process
         # New alias
         if (isset($_POST['alias_url'])) {
             try {
-                $utils->createAlias(new AliasRow($_POST['alias_url'], $_POST['alias_destination'], count($aliases) + 1, !empty($_POST['alias_redirect'])));
+                $alias->createAlias($_POST['alias_url'], $_POST['alias_destination'], count($aliases) + 1, !empty($_POST['alias_redirect']));
                 Notices::addSuccessNotice(__('Alias successfully created.'));
                 My::redirect();
             } catch (Exception $e) {
@@ -94,7 +85,8 @@ class Manage extends Process
             return;
         }
 
-        $aliases = (new Alias())->getAliases();
+        $alias   = new Alias();
+        $aliases = $alias->getAliases();
 
         Page::openModule(My::name());
 
@@ -112,11 +104,11 @@ class Manage extends Process
                 (new Form(My::id() . '_form'))->method('post')->action(App::backend()->getPageURL())->fields([
                     (new Para())->class('field')->items([
                         (new Label(__('Alias URL:'), Label::OUTSIDE_LABEL_BEFORE))->for('alias_url'),
-                        (new Input('alias_url'))->size(50)->maxlength(255),
+                        (new Input('alias_url'))->size(50)->maxlenght(255),
                     ]),
                     (new Para())->class('field')->items([
                         (new Label(__('Alias destination:'), Label::OUTSIDE_LABEL_BEFORE))->for('alias_destination'),
-                        (new Input('alias_destination'))->size(50)->maxlength(255),
+                        (new Input('alias_destination'))->size(50)->maxlenght(255),
                     ]),
                     (new Note())->class('form-note')->text(sprintf(__('Do not put blog URL "%s" in fields.'), App::blog()->url())),
                     (new Para())->items([
@@ -160,17 +152,17 @@ class Manage extends Process
                 '</tr>' .
                 '</thead><tbody>';
 
-                foreach ($aliases as $k => $alias) {
+                foreach ($aliases as $k => $v) {
                     echo
                     '<tr class="line" id="l_' . $k . '">' .
                     '<td class="minimal">' .
-                    (new Input(['a[' . $k . '][alias_url]']))->size(50)->maxlength(255)->value(Html::escapeHTML($alias->url))->render() . '</td>' .
+                    (new Input(['a[' . $k . '][alias_url]']))->size(50)->maxlenght(255)->value(Html::escapeHTML($v['alias_url']))->render() . '</td>' .
                     '<td class="minimal">' .
-                    (new Input(['a[' . $k . '][alias_destination]']))->size(50)->maxlength(255)->value(Html::escapeHTML($alias->destination))->render() . '</td>' .
+                    (new Input(['a[' . $k . '][alias_destination]']))->size(50)->maxlenght(255)->value(Html::escapeHTML($v['alias_destination']))->render() . '</td>' .
                     '<td class="minimal">' .
-                    (new Number(['a[' . $k . '][alias_position]']))->min(1)->max(count($aliases))->default($alias->position)->class('position')->title(sprintf(__('position of %s'), Html::escapeHTML($alias->url)))->render() . '</td>' .
+                    (new Number(['a[' . $k . '][alias_position]']))->min(1)->max(count($aliases))->default((int) $v['alias_position'])->class('position')->title(sprintf(__('position of %s'), Html::escapeHTML($v['alias_url'])))->render() . '</td>' .
                     '<td class="maximal">' .
-                    (new Checkbox(['a[' . $k . '][alias_redirect]'], $alias->redirect))->title(sprintf(__('visible redirection to %s'), Html::escapeHTML(App::blog()->url() . $alias->destination)))->render() . '</td>' .
+                    (new Checkbox(['a[' . $k . '][alias_redirect]'], (bool) $v['alias_redirect']))->title(sprintf(__('visible redirection to %s'), Html::escapeHTML(App::blog()->url() . $v['alias_destination'])))->render() . '</td>' .
                     '</tr>';
                 }
 

src/My.php

@@ -4,6 +4,7 @@ declare(strict_types=1);
 
 namespace Dotclear\Plugin\alias;
 
+use Dotclear\App;
 use Dotclear\Module\MyPlugin;
 
 /**
@@ -16,5 +17,15 @@ use Dotclear\Module\MyPlugin;
  */
 class My extends MyPlugin
 {
-    // Use default permissions
+    public static function checkCustomContext(int $context): ?bool
+    {
+        return match ($context) {
+            My::BACKEND, My::MANAGE, My::MENU => App::task()->checkContext('BACKEND')
+                && App::auth()->check(App::auth()->makePermissions([
+                    App::auth()::PERMISSION_ADMIN,
+                ]), App::blog()->id()),
+
+            default => null,
+        };
+    }
 }

src/PluginImportExportBehaviors.php

@@ -1,75 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace Dotclear\Plugin\alias;
-
-use Dotclear\App;
-use Dotclear\Database\Statement\SelectStatement;
-use Dotclear\Plugin\importExport\{
-    FlatBackupItem,
-    FlatExport,
-    FlatImportV2
-};
-
-/**
- * @brief       alias plugin importExport features class.
- * @ingroup     alias
- *
- * @author      Olivier Meunier (author)
- * @author      Jean-Christian Denis (latest)
- * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
- */
-class PluginImportExportBehaviors
-{
-    public static function exportFullV2(FlatExport $exp): void
-    {
-        $exp->exportTable(Alias::ALIAS_TABLE_NAME);
-    }
-
-    public static function exportSingleV2(FlatExport $exp, ?string $blog_id): void
-    {
-        $sql = new SelectStatement();
-        $sql->columns(['alias_url', 'alias_destination', 'alias_position', 'alias_redirect'])
-            ->from($sql->as(App::con()->prefix() . Alias::ALIAS_TABLE_NAME, 'A'))
-            ->where('blog_id = ' . $sql->quote((string) $blog_id));
-
-        $exp->export('alias', $sql->statement());
-    }
-
-    public static function importInitV2(FlatImportV2 $bk): void
-    {
-        $bk->__set('cur_alias', App::con()->openCursor(App::con()->prefix() . Alias::ALIAS_TABLE_NAME));
-        $bk->__set('alias', new Alias());
-        $bk->__set('aliases', $bk->__get('alias')->getAliases());
-    }
-
-    public static function importFullV2(bool|FlatBackupItem $line, FlatImportV2 $bk): void
-    {
-        if (!is_bool($line) && $line->__name == Alias::ALIAS_TABLE_NAME) {
-            $bk->__get('cur_alias')->clean();
-            $bk->__get('cur_alias')->setField('blog_id', (string) $line->f('blog_id'));
-            $bk->__get('cur_alias')->setField('alias_url', (string) $line->f('alias_url'));
-            $bk->__get('cur_alias')->setField('alias_destination', (string) $line->f('alias_destination'));
-            $bk->__get('cur_alias')->setField('alias_position', (int) $line->f('alias_position'));
-            $bk->__get('cur_alias')->setField('alias_redirect', (int) $line->f('alias_redirect'));
-            $bk->__get('cur_alias')->insert();
-        }
-    }
-
-    public static function importSingleV2(bool|FlatBackupItem $line, FlatImportV2 $bk): void
-    {
-        if (!is_bool($line) && $line->__name == Alias::ALIAS_TABLE_NAME) {
-            $found = false;
-            foreach ($bk->__get('aliases') as $alias) {
-                if ($alias->url == $line->f('alias_url')) {
-                    $found = true;
-                }
-            }
-            if ($found) {
-                $bk->__get('alias')->deleteAlias($line->f('alias_url'));
-            }
-            $bk->__get('alias')->createAlias(new AliasRow($line->f('alias_url'), $line->f('alias_destination'), $line->f('alias_position'), $line->f('alias_redirect')));
-        }
-    }
-}

src/Prepend.php

@@ -10,18 +10,18 @@ use Dotclear\Core\Process;
 use Dotclear\Helper\Network\Http;
 
 /**
- * @brief       alias frontend class.
+ * @brief       alias prepend class.
  * @ingroup     alias
  *
  * @author      Olivier Meunier (author)
  * @author      Jean-Christian Denis (latest)
  * @copyright   GPL-2.0 https://www.gnu.org/licenses/gpl-2.0.html
  */
-class Frontend extends Process
+class Prepend extends Process
 {
     public static function init(): bool
     {
-        return self::status(My::checkContext(My::FRONTEND));
+        return self::status(My::checkContext(My::PREPEND));
     }
 
     public static function process(): bool
@@ -36,19 +36,19 @@ class Frontend extends Process
             $part  = $args = $_SERVER['URL_REQUEST_PART'];
 
             // load all Aliases
-            foreach ((new Alias())->getAliases() as $alias) {
+            foreach ((new Alias())->getAliases() as $v) {
                 // multi alias using "/url/" to "destination"
-                if (@preg_match('#^/.*/$#', $alias->url) && @preg_match($alias->url, $args)) {
+                if (@preg_match('#^/.*/$#', $v['alias_url']) && @preg_match($v['alias_url'], $args)) {
-                    $part  = preg_replace($alias->url, $alias->destination, $args);
+                    $part  = preg_replace($v['alias_url'], $v['alias_destination'], $args);
                     $found = true;
-                    $redir = !empty($alias->redirect);
+                    $redir = !empty($v['alias_redirect']);
 
                     break;
                     // single alias using "url" to "destination"
-                } elseif ($alias->url == $args) {
+                } elseif ($v['alias_url'] == $args) {
-                    $part  = $alias->destination;
+                    $part  = $v['alias_destination'];
                     $found = true;
-                    $redir = !empty($alias->redirect);
+                    $redir = !empty($v['alias_redirect']);
 
                     break;
                 }