Remove DI dependency from Module\Contact\Profile

[friendica.git] / src / Module / Delegation.php

diff --git a/src/Module/Delegation.php b/src/Module/Delegation.php
index ba1f0298d70841f7a4258ed7a016354be642b79d..8a2b313000bce2db24c0d200ac99d924b44b7822 100644 (file)
--- a/src/Module/Delegation.php
+++ b/src/Module/Delegation.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2022, the Friendica project
+ * @copyright Copyright (C) 2010-2023, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -38,22 +38,22 @@ class Delegation extends BaseModule
 {
        protected function post(array $request = [])
        {
-               if (!Session::getLocalUser()) {
+               if (!DI::userSession()->getLocalUserId()) {
                        return;
                }
 
-               $uid = Session::getLocalUser();
+               $uid = DI::userSession()->getLocalUserId();
                $orig_record = User::getById(DI::app()->getLoggedInUserId());
 
-               if (DI::session()->get('submanage')) {
-                       $user = User::getById(DI::session()->get('submanage'));
+               if (DI::userSession()->getSubManagedUserId()) {
+                       $user = User::getById(DI::userSession()->getSubManagedUserId());
                        if (DBA::isResult($user)) {
                                $uid = intval($user['uid']);
                                $orig_record = $user;
                        }
                }
 
-               $identity = intval($_POST['identity'] ?? 0);
+               $identity = intval($request['identity'] ?? 0);
                if (!$identity) {
                        return;
                }
@@ -76,16 +76,16 @@ class Delegation extends BaseModule
                        $user = DBA::selectFirst('user', [], ['uid' => $identity, 'parent-uid' => $orig_record['uid']]);
 
                        // Check if the target user is one of our siblings
-                       if (!DBA::isResult($user) && ($orig_record['parent-uid'] != 0)) {
+                       if (!DBA::isResult($user) && $orig_record['parent-uid']) {
                                $user = DBA::selectFirst('user', [], ['uid' => $identity, 'parent-uid' => $orig_record['parent-uid']]);
                        }
 
                        // Check if it's our parent or our own user
                        if (!DBA::isResult($user)
                                && (
-                                       $orig_record['parent-uid'] != 0 && $orig_record['parent-uid'] == $identity
+                                       $orig_record['parent-uid'] && $orig_record['parent-uid'] === $identity
                                        ||
-                                       $orig_record['uid'] != 0 && $orig_record['uid'] == $identity
+                                       $orig_record['uid'] && $orig_record['uid'] === $identity
                                )
                        ) {
                                $user = User::getById($identity);
@@ -101,7 +101,7 @@ class Delegation extends BaseModule
                DI::auth()->setForUser(DI::app(), $user, true, true);
 
                if ($limited_id) {
-                       DI::session()->set('submanage', $original_id);
+                       DI::userSession()->setSubManagedUserId($original_id);
                }
 
                $ret = [];
@@ -114,13 +114,13 @@ class Delegation extends BaseModule
 
        protected function content(array $request = []): string
        {
-               if (!Session::getLocalUser()) {
+               if (!DI::userSession()->getLocalUserId()) {
                        throw new ForbiddenException(DI::l10n()->t('Permission denied.'));
                }
 
-               $identities = User::identities(DI::session()->get('submanage', Session::getLocalUser()));
+               $identities = User::identities(DI::userSession()->getSubManagedUserId() ?: DI::userSession()->getLocalUserId());
 
-               //getting additinal information for each identity
+               //getting additional information for each identity
                foreach ($identities as $key => $identity) {
                        $identities[$key]['thumb'] = User::getAvatarUrl($identity, Proxy::SIZE_THUMB);