Cake5

.github/workflows/ci.yml

@@ -0,0 +1,22 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - master
+      - cake5
+  pull_request:
+    branches:
+      - '*'
+
+permissions:
+  contents: read
+
+jobs:
+  testsuite:
+    uses: cakephp/.github/.github/workflows/[email protected]
+    secrets: inherit
+
+  cs-stan:
+    uses: cakephp/.github/.github/workflows/[email protected]
+    secrets: inherit

.phive/phars.xml

@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phive xmlns="https://phar.io/phive">
+  <phar name="phpstan" version="1.10.32" installed="1.10.32" location="./tools/phpstan" copy="false"/>
+  <phar name="psalm" version="5.15.0" installed="5.15.0" location="./tools/psalm" copy="false"/>
+</phive>

README.md

@@ -24,7 +24,7 @@ bin/cake plugin load Muffin/Tags
 or by manually adding the following line to `src/Application.php`:
 
 ```php
-$this->addPlugin('Muffin/Obfuscate');
+$this->addPlugin('Muffin/Tags');
 ```
 
 ## Usage

composer.json

@@ -30,13 +30,15 @@
         "source": "https://github.com/usemuffin/tags"
     },
     "require": {
-        "cakephp/orm": "^4.0",
-        "cakephp/utility": "^4.0"
+        "cakephp/orm": "^5.0.0",
+        "cakephp/utility": "^5.0.0"
     },
     "require-dev": {
-        "cakephp/cakephp": "^4.0",
-        "cakephp/cakephp-codesniffer": "^4.0",
-        "phpunit/phpunit": "~8.5.0"
+        "cakephp/cakephp": "^5.0.0",
+        "cakephp/cakephp-codesniffer": "^5.0",
+        "cakephp/migrations": "^4.0.0",
+        "phpunit/phpunit": "^10.1.0",
+        "vimeo/psalm": "^5.15"
     },
     "suggest": {
         "muffin/slug": "For adding slugs to tags"
@@ -53,6 +55,23 @@
         }
     },
     "config": {
-        "sort-packages": true
+        "sort-packages": true,
+        "allow-plugins": {
+            "dealerdirect/phpcodesniffer-composer-installer": true
+        }
+    },
+    "scripts": {
+        "cs-check": "phpcs --colors --parallel=16 -p src/ tests/",
+        "cs-fix": "phpcbf --colors --parallel=16 -p src/ tests/",
+        "phpstan": "tools/phpstan analyse",
+        "psalm": "tools/psalm --show-info=false",
+        "stan": [
+            "@phpstan",
+            "@psalm"
+        ],
+        "stan-baseline": "tools/phpstan --generate-baseline",
+        "psalm-baseline": "tools/psalm  --set-baseline=psalm-baseline.xml",
+        "stan-setup": "phive install",
+        "test": "phpunit"
     }
 }

config/Migrations/20150412195500_create_tags_tags.php

@@ -4,7 +4,7 @@
 
 class CreateTagsTags extends AbstractMigration
 {
-    public function change()
+    public function change(): void
     {
         $table = $this->table('tags_tags');
 

config/Migrations/20150412195501_create_tags_tagged.php

@@ -4,7 +4,7 @@
 
 class CreateTagsTagged extends AbstractMigration
 {
-    public function change()
+    public function change(): void
     {
         $table = $this->table('tags_tagged');
 

config/Migrations/20151013144821_unique_tags.php

@@ -25,7 +25,7 @@ class UniqueTags extends AbstractMigration
      * Remember to call "create()" or "update()" and NOT "save()" when working
      * with the Table class.
      */
-    public function change()
+    public function change(): void
     {
         $table = $this->table('tags_tags');
 

phpcs.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0"?>
+<ruleset name="Muffin/Obfuscate">
+    <config name="installed_paths" value="../../cakephp/cakephp-codesniffer" />
+    <rule ref="CakePHP" />
+    <arg value="s"/>
+</ruleset>

phpstan.neon

@@ -2,3 +2,5 @@ parameters:
     level: 6
     checkGenericClassInNonGenericObjectType: false
     checkMissingIterableValueType: false
+    paths:
+        - src/

phpunit.xml.dist

@@ -1,30 +1,22 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit
-    bootstrap="./tests/bootstrap.php"
-    colors="true"
-    stopOnFailure="false"
-    >
-
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" colors="true" processIsolation="false" stopOnFailure="false" bootstrap="tests/bootstrap.php" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.2/phpunit.xsd" cacheDirectory=".phpunit.cache">
+    <coverage/>
+    <php>
+        <ini name="memory_limit" value="-1"/>
+        <ini name="apc.enable_cli" value="1"/>
+        <env name="FIXTURE_SCHEMA_METADATA" value="./tests/schema.php"/>
+    </php>
     <testsuites>
         <testsuite name="Muffin/Tags Test Cases">
-            <directory>./tests/</directory>
+            <directory>tests/TestCase/</directory>
         </testsuite>
     </testsuites>
-
-    <!-- Setup a listener for fixtures -->
-    <listeners>
-        <listener
-        class="\Cake\TestSuite\Fixture\FixtureInjector"
-        file="./vendor/cakephp/cakephp/src/TestSuite/Fixture/FixtureInjector.php">
-            <arguments>
-                <object class="\Cake\TestSuite\Fixture\FixtureManager" />
-            </arguments>
-        </listener>
-    </listeners>
-
-    <filter>
-        <whitelist>
-            <directory suffix=".php">./src/</directory>
-        </whitelist>
-    </filter>
+    <extensions>
+        <bootstrap class="Cake\TestSuite\Fixture\Extension\PHPUnitExtension"/>
+    </extensions>
+    <source>
+        <include>
+            <directory suffix=".php">src/</directory>
+        </include>
+    </source>
 </phpunit>

src/Model/Behavior/TagBehavior.php

@@ -19,9 +19,9 @@ class TagBehavior extends Behavior
      * - delimiter: The delimiter used to explode() the tags. Default is comma.
      * - separator: Namespace separator, by default semicolon.
      *
-     * @var array
+     * @var array<string, mixed>
      */
-    protected $_defaultConfig = [
+    protected array $_defaultConfig = [
         'delimiter' => ',',
         'separator' => ':',
         'namespace' => null,
@@ -65,7 +65,7 @@ public function initialize(array $config): void
     /**
      * Return lists of event's this behavior is interested in.
      *
-     * @return array Events list.
+     * @return array<string, mixed> Events list.
      */
     public function implementedEvents(): array
     {
@@ -80,7 +80,7 @@ public function implementedEvents(): array
      * @param \ArrayObject $options Options.
      * @return void
      */
-    public function beforeMarshal(EventInterface $event, ArrayObject $data, ArrayObject $options)
+    public function beforeMarshal(EventInterface $event, ArrayObject $data, ArrayObject $options): void
     {
         $field = $this->getConfig('tagsAssoc.propertyName');
         if (!empty($data[$field]) && (!is_array($data[$field]) || !array_key_exists('_ids', $data[$field]))) {
@@ -192,7 +192,6 @@ public function attachCounters(): void
         }
 
         if (!$counterCache->getConfig($taggedAlias)) {
-            $field = key($config['taggedCounter']);
             $config['taggedCounter']['tag_count']['conditions'] = [
                 $taggedTable->aliasField($this->getConfig('fkTableField')) => $this->_table->getTable(),
             ];
@@ -206,7 +205,7 @@ public function attachCounters(): void
      * @param array|string $tags List of tags as an array or a delimited string (comma by default).
      * @return array Normalized tags valid to be marshaled.
      */
-    public function normalizeTags($tags): array
+    public function normalizeTags(array|string $tags): array
     {
         if (is_string($tags)) {
             $tags = explode($this->getConfig('delimiter'), $tags);
@@ -237,6 +236,7 @@ public function normalizeTags($tags): array
             }
             [$id, $label] = $this->_normalizeTag($tag);
             $result[] = $common + compact(empty($id) ? $df : $pk) + [
+                'label' => $label,
                 'tag_key' => $tagKey,
             ];
         }
@@ -259,7 +259,7 @@ protected function _getTagKey(string $tag): string
      * Checks if a tag already exists and returns the id if yes.
      *
      * @param string $tag Tag key.
-     * @return null|int
+     * @return int|null
      */
     protected function _tagExists(string $tag): ?int
     {
@@ -291,7 +291,8 @@ protected function _normalizeTag(string $tag): array
         $namespace = '';
         $label = $tag;
         $separator = $this->getConfig('separator');
-        if (strpos($tag, $separator) !== false) {
+        if (str_contains($tag, $separator)) {
+            /** @psalm-suppress ArgumentTypeCoercion */
             [$namespace, $label] = explode($separator, $tag);
         }
 

src/Model/Entity/Tag.php

@@ -12,7 +12,7 @@ class Tag extends Entity
      *
      * @var array
      */
-    public $accessible = [
+    public array $accessible = [
         '*' => false,
         'label' => true,
     ];

src/Model/Entity/TagAwareTrait.php

@@ -3,30 +3,31 @@
 
 namespace Muffin\Tags\Model\Entity;
 
+use Cake\Datasource\EntityInterface;
 use Cake\ORM\TableRegistry;
 
 trait TagAwareTrait
 {
     /**
      * Tag entity with given tags.
      *
-     * @param string|array $tags List of tags as an array or a delimited string (comma by default).
+     * @param array|string $tags List of tags as an array or a delimited string (comma by default).
      * @param bool $merge Whether to merge or replace tags. Default true.
-     * @return bool|\Cake\ORM\Entity False on failure, entity on success.
+     * @return \Cake\Datasource\EntityInterface|bool False on failure, entity on success.
      */
-    public function tag($tags, bool $merge = true)
+    public function tag(array|string $tags, bool $merge = true): EntityInterface|bool
     {
         return $this->_updateTags($tags, $merge ? 'append' : 'replace');
     }
 
     /**
      * Untag entity from given tags.
      *
-     * @param string|array|null $tags List of tags as an array or a delimited string (comma by default).
+     * @param array|string|null $tags List of tags as an array or a delimited string (comma by default).
      *   If no value is passed all tags will be removed.
-     * @return bool|\Cake\ORM\Entity False on failure, entity on success.
+     * @return \Cake\Datasource\EntityInterface|bool False on failure, entity on success.
      */
-    public function untag($tags = null)
+    public function untag(array|string|null $tags = null): EntityInterface|bool
     {
         if (empty($tags)) {
             return $this->_updateTags([], 'replace');
@@ -41,8 +42,9 @@ public function untag($tags = null)
 
         $tags = $this->get($property);
         if (!$tags) {
-            $contain = [$behavior->getConfig('tagsAlias')];
-            $tags = $table->get($id, compact('contain'))->get($property);
+            $tags = $table
+                ->get(primaryKey: $id, contain: [$behavior->getConfig('tagsAlias')])
+                ->get($property);
         }
 
         $tagsTable = $table->{$behavior->getConfig('tagsAlias')};
@@ -81,12 +83,12 @@ function ($i) {
     /**
      * Tag entity with given tags.
      *
-     * @param string|array $tags List of tags as an array or a delimited string (comma by default).
+     * @param array|string $tags List of tags as an array or a delimited string (comma by default).
      * @param string $saveStrategy Whether to merge or replace tags.
      *   Valid values 'append', 'replace'.
-     * @return bool|\Cake\ORM\Entity False on failure, entity on success.
+     * @return \Cake\Datasource\EntityInterface|bool False on failure, entity on success.
      */
-    protected function _updateTags($tags, string $saveStrategy)
+    protected function _updateTags(array|string $tags, string $saveStrategy): EntityInterface|bool
     {
         $table = TableRegistry::getTableLocator()->get($this->source());
         $behavior = $table->behaviors()->Tag;

src/Model/Entity/Tagged.php

@@ -15,7 +15,7 @@ class Tagged extends Entity
      *
      * @var array
      */
-    public $accessible = [
+    public array $accessible = [
         '*' => false,
     ];
 }