From 30a6a36b653e3fcc5478da702dfefe1a15323b5e Mon Sep 17 00:00:00 2001 From: Benjamin Gaussorgues Date: Fri, 19 Jul 2024 15:36:46 +0200 Subject: [PATCH] Update developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.rst Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com> Signed-off-by: Benjamin Gaussorgues --- .../app_upgrade_guide/upgrade_to_30.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.rst b/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.rst index 8305d26b5..b9b660db1 100644 --- a/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.rst +++ b/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.rst @@ -124,7 +124,7 @@ Changed APIs - Calling ``OCP\Notification\INotification::setLink()`` with a relative URL is deprecated and will throw ``OCP\Notification\InvalidValueException`` in a future version. - Calling ``OCP\Notification\INotification::setIcon()`` with a relative URL is deprecated and will throw ``OCP\Notification\InvalidValueException`` in a future version. - ``OCP\Notification\INotifier::prepare()`` should no longer throw ``\InvalidArgumentException``. ``OCP\Notification\UnknownNotificationException`` should be thrown when the notifier does not want to handle the notification. ``\InvalidArgumentException`` are logged as debug for now and will be logged as error in the future to help developers find issues from code that unintentionally threw ``\InvalidArgumentException`` -- ``OCP\IGroupManager`` should be used instead of checking is current user is part of admin group manually. +- ``OCP\IGroupManager::isAdmin()`` should be used instead of checking is current user is part of admin group manually. Deprecated APIs ^^^^^^^^^^^^^^^