From fee301fbf450274b81c0c946234674a1ef4cd2fb Mon Sep 17 00:00:00 2001 From: Nathaniel Hammond Date: Mon, 3 Feb 2025 17:23:50 +0000 Subject: [PATCH] #3686 change error -> info for delete handler on orders --- CHANGELOG.md | 4 ++++ src/services/Orders.php | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 150a80e4c3..1852aacac5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,9 @@ # Release Notes for Craft Commerce +## Unreleased + +- Improved logging when a user deletion is prevented due to the user having Commerce orders. ([#3686](https://github.com/craftcms/commerce/issues/3686)) + ## 4.8.0.1 - 2025-02-03 - Fixed a bug where the deprecated `TaxRate::$isVat` property was still being set. ([#3874](https://github.com/craftcms/commerce/issues/3874)) diff --git a/src/services/Orders.php b/src/services/Orders.php index 2ae2d59833..34de3713f4 100644 --- a/src/services/Orders.php +++ b/src/services/Orders.php @@ -193,7 +193,7 @@ public function beforeDeleteUserHandler(ModelEvent $event): void if (Order::find()->customerId($user->id)->status(null)->exists()) { $event->isValid = false; - Craft::error(Craft::t('commerce', 'Unable to delete user {user}: the user has a Craft Commerce order.', [ + Craft::info(Craft::t('commerce', 'Unable to delete user {user}: the user has a Craft Commerce order.', [ 'user' => $user->id, ])); }