Fix for CakePHP 5.2

.gitignore

@@ -4,3 +4,4 @@
 .idea
 /.phpunit.result.cache
 /tools
+/nbproject/*

README.md

@@ -1,9 +1,5 @@
 # Trash
 
-[![Build Status](https://img.shields.io/github/actions/workflow/status/UseMuffin/Trash/ci.yml?style=flat-square
-)](https://github.com/UseMuffin/Trash/actions?query=workflow%3ACI+branch%3Amaster)
-[![Coverage](https://img.shields.io/codecov/c/github/UseMuffin/Trash/master.svg?style=flat-square)](https://codecov.io/github/UseMuffin/Trash)
-[![Total Downloads](https://img.shields.io/packagist/dt/muffin/trash.svg?style=flat-square)](https://packagist.org/packages/muffin/trash)
 [![License](https://img.shields.io/badge/license-MIT-blue.svg?style=flat-square)](LICENSE)
 
 Adds "soft"-delete support to CakePHP tables.
@@ -13,7 +9,7 @@ Adds "soft"-delete support to CakePHP tables.
 Using [Composer][composer]:
 
 ```
-composer require muffin/trash
+composer require vandalorumrex/trash
 ```
 
 You then need to load the plugin. You can use the shell command:
@@ -115,13 +111,12 @@ To ensure your PRs are considered for upstream, you MUST follow the CakePHP codi
 
 ## Bugs & Feedback
 
-http://github.com/usemuffin/trash/issues
+http://github.com/vandalorumrex/trash/issues
 
 ## License
 
-Copyright (c) 2015-present, [Use Muffin][muffin] and licensed under [The MIT License][mit].
+This library is license under the MIT License (MIT). Please see License File for more information.
 
-[cakephp]:http://cakephp.org
-[composer]:http://getcomposer.org
-[mit]:http://www.opensource.org/licenses/mit-license.php
-[muffin]:http://usemuffin.com
+## Credits
+
+This library was forked from [UseMuffin Trash's](https://github.com/UseMuffin) on Github https://github.com/UseMuffin/Trash.

composer.json

@@ -1,14 +1,14 @@
 {
-    "name": "muffin/trash",
+    "name": "vandalorumrex/trash",
     "description": "Adds soft delete support to CakePHP ORM tables.",
     "type": "cakephp-plugin",
     "keywords": [
         "cakephp",
-        "muffin",
+        "vandalorumrex",
         "trash",
         "orm"
     ],
-    "homepage": "https://github.com/usemuffin/trash",
+    "homepage": "https://github.com/vandalorumrex/trash",
     "license": "MIT",
     "authors": [
         {
@@ -21,14 +21,19 @@
             "homepage": "https://github.com/ADmad",
             "role": "Author"
         },
+        {
+            "name": "VandalorumRex",
+            "homepage": "https://github.com/VandalorumRex",
+            "role": "Fixer"
+        },
         {
             "name": "Others",
             "homepage": "https://github.com/usemuffin/trash/graphs/contributors"
         }
     ],
     "support": {
-        "issues": "https://github.com/usemuffin/trash/issues",
-        "source": "https://github.com/usemuffin/trash"
+        "issues": "https://github.com/vandalorumrex/trash/issues",
+        "source": "https://github.com/vandalorumrex/trash"
     },
     "require": {
         "php": ">=8.1",

src/Model/Behavior/TrashBehavior.php

@@ -1,5 +1,6 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Muffin\Trash\Model\Behavior;
 
@@ -16,12 +17,13 @@
 use Cake\ORM\Behavior;
 use Cake\ORM\Table;
 use InvalidArgumentException;
+
 use function Cake\Core\pluginSplit;
 
 /**
  * Trash Behavior.
  */
 class TrashBehavior extends Behavior
 {
     public const AFTER_DELETE_EVENT_OPTION = 'trash';
 
@@ -50,7 +52,7 @@
      * @param array $config The config for this behavior.
      * @return void
      */
     public function initialize(array $config): void
     {
         if (isset($config['events']) && $config['events'] !== []) {
             $this->setConfig('events', $config['events'], false);
@@ -63,7 +65,7 @@
      * @return array<string, mixed>
      * @throws \InvalidArgumentException When events are configured in an invalid format.
      */
     public function implementedEvents(): array
     {
         $events = [];
         $config = $this->getConfig('events');
@@ -71,7 +73,7 @@
             return $events;
         }
 
-        foreach ((array)$config as $eventKey => $event) {
+        foreach ((array) $config as $eventKey => $event) {
             if (is_numeric($eventKey)) {
                 $eventKey = $event;
                 $event = null;
@@ -143,7 +145,7 @@
      */
     public function trash(EntityInterface $entity, array $options = []): bool
     {
-        $primaryKey = (array)$this->_table->getPrimaryKey();
+        $primaryKey = (array) $this->_table->getPrimaryKey();
 
         foreach ($primaryKey as $field) {
             if (!$entity->has($field)) {
@@ -160,9 +162,13 @@
             }
         }
 
-        $entity->set($this->getTrashField(false), new DateTime());
+        if (method_exists($entity, 'patch')) {
+            $entity = $entity->patch([$this->getTrashField(false) => new DateTime()]);
+        } else {
+            $entity->set($this->getTrashField(false), new DateTime());
+        }
 
-        return (bool)$this->_table->save($entity, $options);
+        return (bool) $this->_table->save($entity, $options);
     }
 
     /**
@@ -202,17 +208,15 @@
             }
 
             if (
-                $expression instanceof IdentifierExpression
-                && in_array($expression->getIdentifier(), $fieldIdentifiers, true)
+                $expression instanceof IdentifierExpression && in_array($expression->getIdentifier(), $fieldIdentifiers, true)
             ) {
                 $addCondition = false;
 
                 return;
             }
 
             if (
-                $expression instanceof FieldInterface
-                && in_array($expression->getField(), $fieldIdentifiers, true)
+                $expression instanceof FieldInterface && in_array($expression->getField(), $fieldIdentifiers, true)
             ) {
                 $addCondition = false;
             }
@@ -231,8 +235,8 @@
     public function findOnlyTrashed(SelectQuery $query, array $options): SelectQuery
     {
         return $query
-            ->applyOptions(['skipAddTrashCondition' => true])
-            ->andWhere([$this->getTrashField() . ' IS NOT' => null]);
+                ->applyOptions(['skipAddTrashCondition' => true])
+                ->andWhere([$this->getTrashField() . ' IS NOT' => null]);
     }
 
     /**
@@ -258,7 +262,7 @@
     {
         return $this->_table->updateAll(
             [$this->getTrashField(false) => new DateTime()],
             $conditions
         );
     }
 
@@ -287,7 +291,11 @@
             if ($entity->isDirty()) {
                 throw new CakeException('Can not restore from a dirty entity.');
             }
-            $entity->set($data, ['guard' => false]);
+            if (method_exists($entity, 'patch')) {
+                $entity = $entity->patch($data, ['guard' => false]);
+            } else {
+                $entity->set($data, ['guard' => false]);
+            }
 
             return $this->_table->save($entity, $options);
         }
@@ -304,7 +312,7 @@
      */
     public function cascadingRestoreTrash(
         ?EntityInterface $entity = null,
         array $options = []
     ): bool|int|EntityInterface {
         $result = $this->restoreTrash($entity, $options);
 
@@ -313,20 +321,20 @@
             if ($this->_isRecursable($association, $this->_table)) {
                 if ($entity === null) {
                     if ($result > 1) {
                         $result += $association->getTarget()->cascadingRestoreTrash(null, $options);
                     }
                 } else {
                     /** @var list<string> $foreignKey */
-                    $foreignKey = (array)$association->getForeignKey();
+                    $foreignKey = (array) $association->getForeignKey();
                     /** @var list<string> $bindingKey */
-                    $bindingKey = (array)$association->getBindingKey();
+                    $bindingKey = (array) $association->getBindingKey();
                     $conditions = array_combine($foreignKey, $entity->extract($bindingKey));
 
                     foreach ($association->find('withTrashed')->where($conditions) as $related) {
                         if (
                             !$association
                                 ->getTarget()
                                 ->cascadingRestoreTrash($related, ['_primary' => false] + $options)
                         ) {
                             $result = false;
                         }
@@ -383,11 +391,7 @@
     protected function _isRecursable(Association $association, Table $table): bool
     {
         return (
-                $association->getTarget()->hasBehavior('Trash')
-                || $association->getTarget()->hasBehavior(static::class)
-            )
-            && $association->isOwningSide($table)
-            && $association->getDependent()
-            && $association->getCascadeCallbacks();
+            $association->getTarget()->hasBehavior('Trash') || $association->getTarget()->hasBehavior(static::class)
+            ) && $association->isOwningSide($table) && $association->getDependent() && $association->getCascadeCallbacks();
     }
 }