X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FGroup.php;h=6542cc74a8b02abc984fbdc128a74cc368487bd3;hb=800694e9b36bc261a671a80b7ca787f27564a7ac;hp=4c7672c01a7285d855547028cfb8072f7fd9c0be;hpb=8c03bdada92845974ecadeecb9e673c7ffac22b4;p=friendica.git

diff --git a/src/Module/Group.php b/src/Module/Group.php
index 4c7672c01a..6542cc74a8 100644
--- a/src/Module/Group.php
+++ b/src/Module/Group.php
@@ -8,10 +8,10 @@ namespace Friendica\Module;
 use Friendica\BaseModule;
 use Friendica\Core\Config;
 use Friendica\Core\L10n;
-use Friendica\Core\PConfig;
 use Friendica\Core\Renderer;
 use Friendica\Core\System;
 use Friendica\Database\DBA;
+use Friendica\DI;
 use Friendica\Model;
 use Friendica\Util\Strings;
 
@@ -21,15 +21,15 @@ class Group extends BaseModule
 {
 	public static function post(array $parameters = [])
 	{
-		$a = self::getApp();
+		$a = DI::app();
 
-		if ($a->isAjax()) {
+		if (DI::mode()->isAjax()) {
 			self::ajaxPost();
 		}
 
 		if (!local_user()) {
 			notice(L10n::t('Permission denied.'));
-			$a->internalRedirect();
+			DI::baseUrl()->redirect();
 		}
 
 		// @TODO: Replace with parameter from router
@@ -42,12 +42,12 @@ class Group extends BaseModule
 				info(L10n::t('Group created.'));
 				$r = Model\Group::getIdByName(local_user(), $name);
 				if ($r) {
-					$a->internalRedirect('group/' . $r);
+					DI::baseUrl()->redirect('group/' . $r);
 				}
 			} else {
 				notice(L10n::t('Could not create group.'));
 			}
-			$a->internalRedirect('group');
+			DI::baseUrl()->redirect('group');
 		}
 
 		// @TODO: Replace with parameter from router
@@ -57,7 +57,7 @@ class Group extends BaseModule
 			$group = DBA::selectFirst('group', ['id', 'name'], ['id' => $a->argv[1], 'uid' => local_user()]);
 			if (!DBA::isResult($group)) {
 				notice(L10n::t('Group not found.'));
-				$a->internalRedirect('contact');
+				DI::baseUrl()->redirect('contact');
 			}
 			$groupname = Strings::escapeTags(trim($_POST['groupname']));
 			if (strlen($groupname) && ($groupname != $group['name'])) {
@@ -71,7 +71,7 @@ class Group extends BaseModule
 	public static function ajaxPost()
 	{
 		try {
-			$a = self::getApp();
+			$a = DI::app();
 
 			if (!local_user()) {
 				throw new \Exception(L10n::t('Permission denied.'), 403);
@@ -87,34 +87,28 @@ class Group extends BaseModule
 					throw new \Exception(L10n::t('Unknown group.'), 404);
 				}
 
-				$contact = DBA::selectFirst('contact', ['pending', 'blocked', 'deleted'], ['id' => $contact_id, 'uid' => local_user()]);
+				$contact = DBA::selectFirst('contact', ['deleted'], ['id' => $contact_id, 'uid' => local_user()]);
 				if (!DBA::isResult($contact)) {
 					throw new \Exception(L10n::t('Contact not found.'), 404);
 				}
 
-				if ($contact['pending']) {
-					throw new \Exception(L10n::t('Contact is unavailable.'), 400);
-				}
-
 				if ($contact['deleted']) {
 					throw new \Exception(L10n::t('Contact is deleted.'), 410);
 				}
 
 				switch($command) {
 					case 'add':
-						if ($contact['blocked']) {
-							throw new \Exception(L10n::t('Contact is blocked, unable to add it to a group.'), 400);
-						}
-
 						if (!Model\Group::addMember($group_id, $contact_id)) {
 							throw new \Exception(L10n::t('Unable to add the contact to the group.'), 500);
 						}
+
 						$message = L10n::t('Contact successfully added to group.');
 						break;
 					case 'remove':
 						if (!Model\Group::removeMember($group_id, $contact_id)) {
 							throw new \Exception(L10n::t('Unable to remove the contact from the group.'), 500);
 						}
+
 						$message = L10n::t('Contact successfully removed from group.');
 						break;
 					default:
@@ -140,18 +134,18 @@ class Group extends BaseModule
 			throw new \Friendica\Network\HTTPException\ForbiddenException();
 		}
 
-		$a = self::getApp();
+		$a = DI::app();
 
-		$a->page['aside'] = Model\Group::sidebarWidget('contact', 'group', 'extended', (($a->argc > 1) ? $a->argv[1] : 'everyone'));
+		DI::page()['aside'] = Model\Group::sidebarWidget('contact', 'group', 'extended', (($a->argc > 1) ? $a->argv[1] : 'everyone'));
 
 		// With no group number provided we jump to the unassigned contacts as a starting point
 		// @TODO: Replace with parameter from router
 		if ($a->argc == 1) {
-			$a->internalRedirect('group/none');
+			DI::baseUrl()->redirect('group/none');
 		}
 
 		// Switch to text mode interface if we have more than 'n' contacts or group members
-		$switchtotext = PConfig::get(local_user(), 'system', 'groupedit_image_limit');
+		$switchtotext = DI::pConfig()->get(local_user(), 'system', 'groupedit_image_limit');
 		if (is_null($switchtotext)) {
 			$switchtotext = Config::get('system', 'groupedit_image_limit', 200);
 		}
@@ -205,7 +199,7 @@ class Group extends BaseModule
 			if (intval($a->argv[2])) {
 				if (!Model\Group::exists($a->argv[2], local_user())) {
 					notice(L10n::t('Group not found.'));
-					$a->internalRedirect('contact');
+					DI::baseUrl()->redirect('contact');
 				}
 
 				if (Model\Group::remove($a->argv[2])) {
@@ -214,7 +208,7 @@ class Group extends BaseModule
 					notice(L10n::t('Unable to remove group.'));
 				}
 			}
-			$a->internalRedirect('group');
+			DI::baseUrl()->redirect('group');
 		}
 
 		// @TODO: Replace with parameter from router
@@ -231,7 +225,7 @@ class Group extends BaseModule
 			$group = DBA::selectFirst('group', ['id', 'name'], ['id' => $a->argv[1], 'uid' => local_user(), 'deleted' => false]);
 			if (!DBA::isResult($group)) {
 				notice(L10n::t('Group not found.'));
-				$a->internalRedirect('contact');
+				DI::baseUrl()->redirect('contact');
 			}
 
 			$members = Model\Contact::getByGroupId($group['id']);