cosmetic code review

This commit is contained in:
Jean-Christian Denis 2023-11-04 21:58:41 +01:00
parent 25588c2d1b
commit 47d532dbb3
Signed by: JcDenis
GPG key ID: 1B5B8C5B90B6C951
6 changed files with 16 additions and 10 deletions

View file

@ -1,3 +1,9 @@
tinyPacker 1.4.3 - 2023.11.04
===========================================================
* Require Dotclear 2.28
* Require PHP 8.1+
* Code review
tinyPacker 1.4.2 - 2023.10.20 tinyPacker 1.4.2 - 2023.10.20
=========================================================== ===========================================================
* Require Dotclear 2.28 * Require Dotclear 2.28

View file

@ -1,7 +1,7 @@
# README # README
[![Release](https://img.shields.io/badge/release-1.4.2-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/tinyPacker/releases) [![Release](https://img.shields.io/badge/release-1.4.3-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/tinyPacker/releases)
![Date](https://img.shields.io/badge/date-2023.10.20-c44d58.svg) ![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) [![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/tinyPacker) [![Dotaddict](https://img.shields.io/badge/dotaddict-official-9ac123.svg)](https://plugins.dotaddict.org/dc2/details/tinyPacker)
[![License](https://img.shields.io/badge/license-GPL--2.0-ececec.svg)](https://git.dotclear.watch/JcDenis/tinyPacker/src/branch/master/LICENSE) [![License](https://img.shields.io/badge/license-GPL--2.0-ececec.svg)](https://git.dotclear.watch/JcDenis/tinyPacker/src/branch/master/LICENSE)

View file

@ -17,7 +17,7 @@ $this->registerModule(
'Tiny packer', 'Tiny packer',
'Quick pack theme or plugin into public dir', 'Quick pack theme or plugin into public dir',
'Jean-Christian Denis', 'Jean-Christian Denis',
'1.4.2', '1.4.3',
[ [
'requires' => [['core', '2.28']], 'requires' => [['core', '2.28']],
'permissions' => 'My', 'permissions' => 'My',

View file

@ -2,10 +2,10 @@
<modules xmlns:da="http://dotaddict.org/da/"> <modules xmlns:da="http://dotaddict.org/da/">
<module id="tinyPacker"> <module id="tinyPacker">
<name>Tiny packer</name> <name>Tiny packer</name>
<version>1.4.2</version> <version>1.4.3</version>
<author>Jean-Christian Denis</author> <author>Jean-Christian Denis</author>
<desc>Quick pack theme or plugin into public dir</desc> <desc>Quick pack theme or plugin into public dir</desc>
<file>https://git.dotclear.watch/JcDenis/tinyPacker/releases/download/v1.4.2/plugin-tinyPacker.zip</file> <file>https://git.dotclear.watch/JcDenis/tinyPacker/releases/download/v1.4.3/plugin-tinyPacker.zip</file>
<da:dcmin>2.28</da:dcmin> <da:dcmin>2.28</da:dcmin>
<da:details>https://git.dotclear.watch/JcDenis/tinyPacker/src/branch/master/README.md</da:details> <da:details>https://git.dotclear.watch/JcDenis/tinyPacker/src/branch/master/README.md</da:details>
<da:support>https://git.dotclear.watch/JcDenis/tinyPacker/issues</da:support> <da:support>https://git.dotclear.watch/JcDenis/tinyPacker/issues</da:support>

View file

@ -17,8 +17,8 @@ use Dotclear\Helper\Network\Http;
use Exception; use Exception;
/** /**
* @brief tinyPacker admin class. * @brief tinyPacker admin class.
* @ingroup tinyPacker * @ingroup tinyPacker
* *
* Add action and button to modules lists. * Add action and button to modules lists.
* *

View file

@ -8,8 +8,8 @@ use Dotclear\App;
use Dotclear\Module\MyPlugin; use Dotclear\Module\MyPlugin;
/** /**
* @brief tinyPacker My helper. * @brief tinyPacker My helper.
* @ingroup tinyPacker * @ingroup tinyPacker
* *
* @author 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
@ -44,8 +44,8 @@ class My extends MyPlugin
public static function checkCustomContext(int $context): ?bool public static function checkCustomContext(int $context): ?bool
{ {
// Limit to super admin
return match ($context) { return match ($context) {
// Limit to super admin
self::MODULE => App::auth()->isSuperAdmin(), self::MODULE => App::auth()->isSuperAdmin(),
default => null, default => null,
}; };