functions.php ➔ query_the_posts() - Code Metrics - Inspection of "Merge remote-tracking branch 'refs/remotes/origin/..." - gctools-outilsgc/gcconnex - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — Api_opt ( 47a261...24829b )

unknown

created 2018-03-13 13:43 UTC

functions.php ➔ query_the_posts() D

↳ Parent: Project

Complexity

Conditions	15
Paths	228

Size

Total Lines	68
Code Lines	50

Duplication

Lines	5
Ratio	7.35 %

Importance

Changes

Metric	Value
cc	15
eloc	50
nc	228
nop	8
dl	5
loc	68
rs	4.7145
c	0
b	0
f	0

How to fix Long Method Complexity Many Parameters

<?php
/*
 * GC Mobile API functions.php
 */

function get_user_block($userid, $lang = "en")
{
	$user_entity = is_numeric($userid) ? get_user($userid) : (strpos($userid, '@') !== false ? get_user_by_email($userid)[0] : get_user_by_username($userid));

	if (!$user_entity) {
		return "";
	}

	if (!$user_entity instanceof ElggUser) {
		return "";
	}

	$user = array();
	$user['user_id'] = $user_entity->guid;
	$user['username'] = $user_entity->username;
	$user['displayName'] = $user_entity->name;
	$user['email'] = $user_entity->email;
	$user['profileURL'] = $user_entity->getURL();
	$user['iconURL'] = $user_entity->getIconURL();
	$user['dateJoined'] = date("Y-m-d H:i:s", $user_entity->time_created);

	$userType = $user_entity->user_type;
	$user['user_type'] = elgg_echo("gcRegister:occupation:{$userType}", [], $lang);
	$department = "";

	if ($userType == 'federal') {
		$deptObj = elgg_get_entities(array(
			'type' => 'object',
			'subtype' => 'federal_departments',
		));
		$depts = get_entity($deptObj[0]->guid);

		$federal_departments = array();
		if ($lang == 'en') {
			$federal_departments = json_decode($depts->federal_departments_en, true);
		} else {
			$federal_departments = json_decode($depts->federal_departments_fr, true);
		}

		$department = $federal_departments[$user_entity->federal];

		// otherwise if user is student or academic
	} elseif ($userType == 'student' || $userType == 'academic') {
		$institution = $user_entity->institution;
		$department = ($institution == 'university') ? $user_entity->university : ($institution == 'college' ? $user_entity->college : $user_entity->highschool);

		// otherwise if user is provincial employee
	} elseif ($userType == 'provincial') {
		$provObj = elgg_get_entities(array(
			'type' => 'object',
			'subtype' => 'provinces',
		));
		$provs = get_entity($provObj[0]->guid);

		$provinces = array();
		if ($lang == 'en') {
			$provinces = json_decode($provs->provinces_en, true);
		} else {
			$provinces = json_decode($provs->provinces_fr, true);
		}

		$minObj = elgg_get_entities(array(
			'type' => 'object',
			'subtype' => 'ministries',
		));
		$mins = get_entity($minObj[0]->guid);

		$ministries = array();
		if ($lang == 'en') {
			$ministries = json_decode($mins->ministries_en, true);
		} else {
			$ministries = json_decode($mins->ministries_fr, true);
		}

		$department = $provinces[$user_entity->provincial];
		if ($user_entity->ministry && $user_entity->ministry !== "default_invalid_value") {
			$department .= ' / ' . $ministries[$user_entity->provincial][$user_entity->ministry];
		}

		// otherwise show basic info
	} else {
		$department = $user_entity->$userType;
	}
	$user['organization'] = $department;
	$user['job'] = $user_entity->job;

	return $user;
}

function get_entity_comments($guid)
{
	$entity = get_entity($guid);

	$comments = array();
	$comments['count'] = $entity->countComments();
	$commentEntites = elgg_get_entities(array(
		'type' => 'object',
		'subtype' => 'comment',
		'container_guid' => $entity->guid,
		'order_by' => 'time_created asc'
	));

	$i = 0;
	foreach ($commentEntites as $comment) {
		$i++;
		$comments['comment_'.$i] = array('comment_user'=>get_userBlock($comment->getOwner()),'comment_text'=>$comment->description,'comment_date'=>date("Y-m-d H:i:s", $comment->time_created));
	}
	return $comments;
}

function wire_filter($text)
{
	$site_url = elgg_get_site_url();

	$text = ''.$text;

	// email addresses
	$text = preg_replace('/(^|[^\w])([\w\-\.]+)@(([0-9a-z-]+\.)+[0-9a-z]{2,})/i', '$1<a href="mailto:$2@$3">$2@$3</a>', $text);

	// links
	$text = parse_urls($text);

	// usernames
	$text = preg_replace('/(^|[^\w])@([\p{L}\p{Nd}._]+)/u', '$1<a href="' . $site_url . 'thewire/owner/$2">@$2</a>', $text);

	// hashtags
	$text = preg_replace('/(^|[^\w])#(\w*[^\s\d!-\/:-@]+\w*)/', '$1<a href="' . $site_url . 'thewire/tag/$2">#$2</a>', $text);

	$text = trim($text);

	return $text;
}

function clean_text($text)
{
	return trim(preg_replace('/ +/', ' ', preg_replace('/[^A-Za-z0-9 ]/', ' ', urldecode(html_entity_decode(strip_tags($text))))));
}

function replace_accents($str)
{
	$a = array('À', 'Á', 'Â', 'Ã', 'Ä', 'Å', 'Æ', 'Ç', 'È', 'É', 'Ê', 'Ë', 'Ì', 'Í', 'Î', 'Ï', 'Ð', 'Ñ', 'Ò', 'Ó', 'Ô', 'Õ', 'Ö', 'Ø', 'Ù', 'Ú', 'Û', 'Ü', 'Ý', 'ß', 'à', 'á', 'â', 'ã', 'ä', 'å', 'æ', 'ç', 'è', 'é', 'ê', 'ë', 'ì', 'í', 'î', 'ï', 'ñ', 'ò', 'ó', 'ô', 'õ', 'ö', 'ø', 'ù', 'ú', 'û', 'ü', 'ý', 'ÿ', 'Ā', 'ā', 'Ă', 'ă', 'Ą', 'ą', 'Ć', 'ć', 'Ĉ', 'ĉ', 'Ċ', 'ċ', 'Č', 'č', 'Ď', 'ď', 'Đ', 'đ', 'Ē', 'ē', 'Ĕ', 'ĕ', 'Ė', 'ė', 'Ę', 'ę', 'Ě', 'ě', 'Ĝ', 'ĝ', 'Ğ', 'ğ', 'Ġ', 'ġ', 'Ģ', 'ģ', 'Ĥ', 'ĥ', 'Ħ', 'ħ', 'Ĩ', 'ĩ', 'Ī', 'ī', 'Ĭ', 'ĭ', 'Į', 'į', 'İ', 'ı', 'Ĳ', 'ĳ', 'Ĵ', 'ĵ', 'Ķ', 'ķ', 'Ĺ', 'ĺ', 'Ļ', 'ļ', 'Ľ', 'ľ', 'Ŀ', 'ŀ', 'Ł', 'ł', 'Ń', 'ń', 'Ņ', 'ņ', 'Ň', 'ň', 'ŉ', 'Ō', 'ō', 'Ŏ', 'ŏ', 'Ő', 'ő', 'Œ', 'œ', 'Ŕ', 'ŕ', 'Ŗ', 'ŗ', 'Ř', 'ř', 'Ś', 'ś', 'Ŝ', 'ŝ', 'Ş', 'ş', 'Š', 'š', 'Ţ', 'ţ', 'Ť', 'ť', 'Ŧ', 'ŧ', 'Ũ', 'ũ', 'Ū', 'ū', 'Ŭ', 'ŭ', 'Ů', 'ů', 'Ű', 'ű', 'Ų', 'ų', 'Ŵ', 'ŵ', 'Ŷ', 'ŷ', 'Ÿ', 'Ź', 'ź', 'Ż', 'ż', 'Ž', 'ž', 'ſ', 'ƒ', 'Ơ', 'ơ', 'Ư', 'ư', 'Ǎ', 'ǎ', 'Ǐ', 'ǐ', 'Ǒ', 'ǒ', 'Ǔ', 'ǔ', 'Ǖ', 'ǖ', 'Ǘ', 'ǘ', 'Ǚ', 'ǚ', 'Ǜ', 'ǜ', 'Ǻ', 'ǻ', 'Ǽ', 'ǽ', 'Ǿ', 'ǿ');
	$b = array('A', 'A', 'A', 'A', 'A', 'A', 'AE', 'C', 'E', 'E', 'E', 'E', 'I', 'I', 'I', 'I', 'D', 'N', 'O', 'O', 'O', 'O', 'O', 'O', 'U', 'U', 'U', 'U', 'Y', 's', 'a', 'a', 'a', 'a', 'a', 'a', 'ae', 'c', 'e', 'e', 'e', 'e', 'i', 'i', 'i', 'i', 'n', 'o', 'o', 'o', 'o', 'o', 'o', 'u', 'u', 'u', 'u', 'y', 'y', 'A', 'a', 'A', 'a', 'A', 'a', 'C', 'c', 'C', 'c', 'C', 'c', 'C', 'c', 'D', 'd', 'D', 'd', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'G', 'g', 'G', 'g', 'G', 'g', 'G', 'g', 'H', 'h', 'H', 'h', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'IJ', 'ij', 'J', 'j', 'K', 'k', 'L', 'l', 'L', 'l', 'L', 'l', 'L', 'l', 'l', 'l', 'N', 'n', 'N', 'n', 'N', 'n', 'n', 'O', 'o', 'O', 'o', 'O', 'o', 'OE', 'oe', 'R', 'r', 'R', 'r', 'R', 'r', 'S', 's', 'S', 's', 'S', 's', 'S', 's', 'T', 't', 'T', 't', 'T', 't', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'W', 'w', 'Y', 'y', 'Y', 'Z', 'z', 'Z', 'z', 'Z', 'z', 's', 'f', 'O', 'o', 'U', 'u', 'A', 'a', 'I', 'i', 'O', 'o', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'A', 'a', 'AE', 'ae', 'O', 'o');
	return str_replace($a, $b, $str);
}

function create_username($str, $a_char = array("'", "-", "."))
{
	$string = replace_accents(mb_strtolower(strtok($str, '@')));
	foreach ($a_char as $temp) {
		$pos = strpos($string, $temp);
		if ($pos) {
			$mend = '';
			$a_split = explode($temp, $string);
			foreach ($a_split as $temp2) {
				$mend .= ucfirst($temp2).$temp;
			}
			$string = substr($mend, 0, -1);
		}
	}
	return ucfirst($string);
}

elgg_ws_expose_function(
	"query.posts",
	"query_the_posts",
	array(
		"user" => array('type' => 'string', 'required' => true),
		"password" => array('type' => 'string', 'required' => true),
		"object" => array('type' => 'string', 'required' => false, 'default' => ""),
		"query" => array('type' => 'string', 'required' => false, 'default' => ""),
		"group" => array('type' => 'string', 'required' => false, 'default' => ""),
		"limit" => array('type' => 'int', 'required' => false, 'default' => 10),
		"offset" => array('type' => 'int', 'required' => false, 'default' => 0),
		"lang" => array('type' => 'string', 'required' => false, 'default' => "en")
	),
	'Query GCcollab data based on user-given parameters',
	'POST',
	false,
	false
);

function query_the_posts($user, $password, $object, $query, $group, $limit, $offset, $lang)
{
	$user_entity = is_numeric($user) ? get_user($user) : (strpos($user, '@') !== false ? get_user_by_email($user)[0] : get_user_by_username($user));
	if (!$user_entity) {
		return "User was not found. Please try a different GUID, username, or email address";
	}
	if (!$user_entity instanceof ElggUser) {
		return "Invalid user. Please try a different GUID, username, or email address";
	}

    $valid = elgg_authenticate($user_entity->username, $password);

    $type = "object";
    $subtype = "";
    switch ($object) {
    	case "blog":
    		$subtype = "blog";
        	break;
        case "discussion":
    		$subtype = "groupforumtopic";
        	break;
        case "event":
    		$subtype = "event_calendar";
        	break;
        case "group":
    		$type = "group";
        	break;
        case "opportunity":
    		$subtype = "mission";
        	break;
        case "wire":
    		$subtype = "thewire";
        	break;
        default:
    		return "Please use one of the following object types: 'blog', 'discussion', 'event', 'group', 'opportunity', 'wire'";
        	break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
	}

    $data = "Username/password combination is not correct.";
	if ($valid === true) {
		if (!elgg_is_logged_in()) {
			login($user_entity);
		}

		$params = array(
			'type' => $type,
			'subtype' => $subtype,
			'limit' => $limit,
			'offset' => $offset
		);

		if ($query) {
			$db_prefix = elgg_get_config('dbprefix');
			$params['joins'] = array("JOIN {$db_prefix}objects_entity oe ON e.guid = oe.guid");
			$params['wheres'] = array("(oe.title LIKE '%" . $query . "%' OR oe.description LIKE '%" . $query . "%')");
		}

		if ($group) {
			$params['container_guid'] = $group;
		}

		$ia = elgg_set_ignore_access(true);
		$data = json_decode(elgg_list_entities_from_metadata($params));
		elgg_set_ignore_access($ia);
	}

	return $data;
}

elgg_ws_expose_function(
	"login.redirect",
	"login_and_redirect",
	array(
		"user" => array('type' => 'string', 'required' => true),
		"password" => array('type' => 'string', 'required' => true),
		"redirect_en" => array('type' => 'string', 'required' => true),
		"redirect_fr" => array('type' => 'string', 'required' => true),
		"lang" => array('type' => 'string', 'required' => false, 'default' => "en")
	),
	'Login user into GCcollab and redirect them',
	'POST',
	false,
	false
);

function login_and_redirect($user, $password, $redirect_en, $redirect_fr, $lang)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
{
	$user_entity = is_numeric($user) ? get_user($user) : (strpos($user, '@') !== false ? get_user_by_email($user)[0] : get_user_by_username($user));
	if (!$user_entity) {
		header("Location: " . $_SERVER['HTTP_REFERER']);
		exit();

	}
	if (!$user_entity instanceof ElggUser) {
		header("Location: " . $_SERVER['HTTP_REFERER']);
		exit();

	}

    $valid = elgg_authenticate($user_entity->username, $password);

	if ($valid === true) {
		login($user_entity);

		if($lang == "fr"){
			setcookie("gcconnex_lang", "fr");
			header("Location: $redirect_fr");
			exit();

		} else {
			setcookie("gcconnex_lang", "en");
			header("Location: $redirect_en");
			exit();

		}
	} else {
		header("Location: " . $_SERVER['HTTP_REFERER']);
		exit();

	}
}


1		<?php
2		/*
3		* GC Mobile API functions.php
4		*/
5
6		function get_user_block($userid, $lang = "en")
7		{
8		$user_entity = is_numeric($userid) ? get_user($userid) : (strpos($userid, '@') !== false ? get_user_by_email($userid)[0] : get_user_by_username($userid));
9
10		if (!$user_entity) {
11		return "";
12		}
13
14		if (!$user_entity instanceof ElggUser) {
15		return "";
16		}
17
18		$user = array();
19		$user['user_id'] = $user_entity->guid;
20		$user['username'] = $user_entity->username;
21		$user['displayName'] = $user_entity->name;
22		$user['email'] = $user_entity->email;
23		$user['profileURL'] = $user_entity->getURL();
24		$user['iconURL'] = $user_entity->getIconURL();
25		$user['dateJoined'] = date("Y-m-d H:i:s", $user_entity->time_created);
26
27		$userType = $user_entity->user_type;
28		$user['user_type'] = elgg_echo("gcRegister:occupation:{$userType}", [], $lang);
29		$department = "";
30
31		if ($userType == 'federal') {
32		$deptObj = elgg_get_entities(array(
33		'type' => 'object',
34		'subtype' => 'federal_departments',
35		));
36		$depts = get_entity($deptObj[0]->guid);
37
38		$federal_departments = array();
39		if ($lang == 'en') {
40		$federal_departments = json_decode($depts->federal_departments_en, true);
41		} else {
42		$federal_departments = json_decode($depts->federal_departments_fr, true);
43		}
44
45		$department = $federal_departments[$user_entity->federal];
46
47		// otherwise if user is student or academic
48	View Code Duplication	} elseif ($userType == 'student' \|\| $userType == 'academic') {
49		$institution = $user_entity->institution;
50		$department = ($institution == 'university') ? $user_entity->university : ($institution == 'college' ? $user_entity->college : $user_entity->highschool);
51
52		// otherwise if user is provincial employee
53		} elseif ($userType == 'provincial') {
54		$provObj = elgg_get_entities(array(
55		'type' => 'object',
56		'subtype' => 'provinces',
57		));
58		$provs = get_entity($provObj[0]->guid);
59
60		$provinces = array();
61		if ($lang == 'en') {
62		$provinces = json_decode($provs->provinces_en, true);
63		} else {
64		$provinces = json_decode($provs->provinces_fr, true);
65		}
66
67		$minObj = elgg_get_entities(array(
68		'type' => 'object',
69		'subtype' => 'ministries',
70		));
71		$mins = get_entity($minObj[0]->guid);
72
73		$ministries = array();
74		if ($lang == 'en') {
75		$ministries = json_decode($mins->ministries_en, true);
76		} else {
77		$ministries = json_decode($mins->ministries_fr, true);
78		}
79
80		$department = $provinces[$user_entity->provincial];
81		if ($user_entity->ministry && $user_entity->ministry !== "default_invalid_value") {
82		$department .= ' / ' . $ministries[$user_entity->provincial][$user_entity->ministry];
83		}
84
85		// otherwise show basic info
86		} else {
87		$department = $user_entity->$userType;
88		}
89		$user['organization'] = $department;
90		$user['job'] = $user_entity->job;
91
92		return $user;
93		}
94
95		function get_entity_comments($guid)
96		{
97		$entity = get_entity($guid);
98
99		$comments = array();
100		$comments['count'] = $entity->countComments();
101		$commentEntites = elgg_get_entities(array(
102		'type' => 'object',
103		'subtype' => 'comment',
104		'container_guid' => $entity->guid,
105		'order_by' => 'time_created asc'
106		));
107
108		$i = 0;
109	View Code Duplication	foreach ($commentEntites as $comment) {
110		$i++;
111		$comments['comment_'.$i] = array('comment_user'=>get_userBlock($comment->getOwner()),'comment_text'=>$comment->description,'comment_date'=>date("Y-m-d H:i:s", $comment->time_created));
112		}
113		return $comments;
114		}
115
116	View Code Duplication	function wire_filter($text)
117		{
118		$site_url = elgg_get_site_url();
119
120		$text = ''.$text;
121
122		// email addresses
123		$text = preg_replace('/(^\|[^\w])([\w\-\.]+)@(([0-9a-z-]+\.)+[0-9a-z]{2,})/i', '$1<a href="mailto:$2@$3">$2@$3</a>', $text);
124
125		// links
126		$text = parse_urls($text);
127
128		// usernames
129		$text = preg_replace('/(^\|[^\w])@([\p{L}\p{Nd}._]+)/u', '$1<a href="' . $site_url . 'thewire/owner/$2">@$2</a>', $text);
130
131		// hashtags
132		$text = preg_replace('/(^\|[^\w])#(\w[^\s\d!-\/:-@]+\w)/', '$1<a href="' . $site_url . 'thewire/tag/$2">#$2</a>', $text);
133
134		$text = trim($text);
135
136		return $text;
137		}
138
139		function clean_text($text)
140		{
141		return trim(preg_replace('/ +/', ' ', preg_replace('/[^A-Za-z0-9 ]/', ' ', urldecode(html_entity_decode(strip_tags($text))))));
142		}
143
144		function replace_accents($str)
145		{
146		$a = array('À', 'Á', 'Â', 'Ã', 'Ä', 'Å', 'Æ', 'Ç', 'È', 'É', 'Ê', 'Ë', 'Ì', 'Í', 'Î', 'Ï', 'Ð', 'Ñ', 'Ò', 'Ó', 'Ô', 'Õ', 'Ö', 'Ø', 'Ù', 'Ú', 'Û', 'Ü', 'Ý', 'ß', 'à', 'á', 'â', 'ã', 'ä', 'å', 'æ', 'ç', 'è', 'é', 'ê', 'ë', 'ì', 'í', 'î', 'ï', 'ñ', 'ò', 'ó', 'ô', 'õ', 'ö', 'ø', 'ù', 'ú', 'û', 'ü', 'ý', 'ÿ', 'Ā', 'ā', 'Ă', 'ă', 'Ą', 'ą', 'Ć', 'ć', 'Ĉ', 'ĉ', 'Ċ', 'ċ', 'Č', 'č', 'Ď', 'ď', 'Đ', 'đ', 'Ē', 'ē', 'Ĕ', 'ĕ', 'Ė', 'ė', 'Ę', 'ę', 'Ě', 'ě', 'Ĝ', 'ĝ', 'Ğ', 'ğ', 'Ġ', 'ġ', 'Ģ', 'ģ', 'Ĥ', 'ĥ', 'Ħ', 'ħ', 'Ĩ', 'ĩ', 'Ī', 'ī', 'Ĭ', 'ĭ', 'Į', 'į', 'İ', 'ı', 'Ĳ', 'ĳ', 'Ĵ', 'ĵ', 'Ķ', 'ķ', 'Ĺ', 'ĺ', 'Ļ', 'ļ', 'Ľ', 'ľ', 'Ŀ', 'ŀ', 'Ł', 'ł', 'Ń', 'ń', 'Ņ', 'ņ', 'Ň', 'ň', 'ŉ', 'Ō', 'ō', 'Ŏ', 'ŏ', 'Ő', 'ő', 'Œ', 'œ', 'Ŕ', 'ŕ', 'Ŗ', 'ŗ', 'Ř', 'ř', 'Ś', 'ś', 'Ŝ', 'ŝ', 'Ş', 'ş', 'Š', 'š', 'Ţ', 'ţ', 'Ť', 'ť', 'Ŧ', 'ŧ', 'Ũ', 'ũ', 'Ū', 'ū', 'Ŭ', 'ŭ', 'Ů', 'ů', 'Ű', 'ű', 'Ų', 'ų', 'Ŵ', 'ŵ', 'Ŷ', 'ŷ', 'Ÿ', 'Ź', 'ź', 'Ż', 'ż', 'Ž', 'ž', 'ſ', 'ƒ', 'Ơ', 'ơ', 'Ư', 'ư', 'Ǎ', 'ǎ', 'Ǐ', 'ǐ', 'Ǒ', 'ǒ', 'Ǔ', 'ǔ', 'Ǖ', 'ǖ', 'Ǘ', 'ǘ', 'Ǚ', 'ǚ', 'Ǜ', 'ǜ', 'Ǻ', 'ǻ', 'Ǽ', 'ǽ', 'Ǿ', 'ǿ');
147		$b = array('A', 'A', 'A', 'A', 'A', 'A', 'AE', 'C', 'E', 'E', 'E', 'E', 'I', 'I', 'I', 'I', 'D', 'N', 'O', 'O', 'O', 'O', 'O', 'O', 'U', 'U', 'U', 'U', 'Y', 's', 'a', 'a', 'a', 'a', 'a', 'a', 'ae', 'c', 'e', 'e', 'e', 'e', 'i', 'i', 'i', 'i', 'n', 'o', 'o', 'o', 'o', 'o', 'o', 'u', 'u', 'u', 'u', 'y', 'y', 'A', 'a', 'A', 'a', 'A', 'a', 'C', 'c', 'C', 'c', 'C', 'c', 'C', 'c', 'D', 'd', 'D', 'd', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'G', 'g', 'G', 'g', 'G', 'g', 'G', 'g', 'H', 'h', 'H', 'h', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'IJ', 'ij', 'J', 'j', 'K', 'k', 'L', 'l', 'L', 'l', 'L', 'l', 'L', 'l', 'l', 'l', 'N', 'n', 'N', 'n', 'N', 'n', 'n', 'O', 'o', 'O', 'o', 'O', 'o', 'OE', 'oe', 'R', 'r', 'R', 'r', 'R', 'r', 'S', 's', 'S', 's', 'S', 's', 'S', 's', 'T', 't', 'T', 't', 'T', 't', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'W', 'w', 'Y', 'y', 'Y', 'Z', 'z', 'Z', 'z', 'Z', 'z', 's', 'f', 'O', 'o', 'U', 'u', 'A', 'a', 'I', 'i', 'O', 'o', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'A', 'a', 'AE', 'ae', 'O', 'o');
148		return str_replace($a, $b, $str);
149		}
150
151	View Code Duplication	function create_username($str, $a_char = array("'", "-", "."))
152		{
153		$string = replace_accents(mb_strtolower(strtok($str, '@')));
154		foreach ($a_char as $temp) {
155		$pos = strpos($string, $temp);
156		if ($pos) {
157		$mend = '';
158		$a_split = explode($temp, $string);
159		foreach ($a_split as $temp2) {
160		$mend .= ucfirst($temp2).$temp;
161		}
162		$string = substr($mend, 0, -1);
163		}
164		}
165		return ucfirst($string);
166		}
167
168		elgg_ws_expose_function(
169		"query.posts",
170		"query_the_posts",
171		array(
172		"user" => array('type' => 'string', 'required' => true),
173		"password" => array('type' => 'string', 'required' => true),
174		"object" => array('type' => 'string', 'required' => false, 'default' => ""),
175		"query" => array('type' => 'string', 'required' => false, 'default' => ""),
176		"group" => array('type' => 'string', 'required' => false, 'default' => ""),
177		"limit" => array('type' => 'int', 'required' => false, 'default' => 10),
178		"offset" => array('type' => 'int', 'required' => false, 'default' => 0),
179		"lang" => array('type' => 'string', 'required' => false, 'default' => "en")
180		),
181		'Query GCcollab data based on user-given parameters',
182		'POST',
183		false,
184		false
185		);
186
187		function query_the_posts($user, $password, $object, $query, $group, $limit, $offset, $lang)
188		{
189		$user_entity = is_numeric($user) ? get_user($user) : (strpos($user, '@') !== false ? get_user_by_email($user)[0] : get_user_by_username($user));
190		if (!$user_entity) {
191		return "User was not found. Please try a different GUID, username, or email address";
192		}
193		if (!$user_entity instanceof ElggUser) {
194		return "Invalid user. Please try a different GUID, username, or email address";
195		}
196
197		$valid = elgg_authenticate($user_entity->username, $password);
198
199		$type = "object";
200		$subtype = "";
201		switch ($object) {
202		case "blog":
203		$subtype = "blog";
204		break;
205		case "discussion":
206		$subtype = "groupforumtopic";
207		break;
208		case "event":
209		$subtype = "event_calendar";
210		break;
211		case "group":
212		$type = "group";
213		break;
214		case "opportunity":
215		$subtype = "mission";
216		break;
217		case "wire":
218		$subtype = "thewire";
219		break;
220		default:
221		return "Please use one of the following object types: 'blog', 'discussion', 'event', 'group', 'opportunity', 'wire'";
222		break;
		0 ignored issues – show Unused Code introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
223		}
224
225		$data = "Username/password combination is not correct.";
226		if ($valid === true) {
227		if (!elgg_is_logged_in()) {
228		login($user_entity);
229		}
230
231		$params = array(
232		'type' => $type,
233		'subtype' => $subtype,
234		'limit' => $limit,
235		'offset' => $offset
236		);
237
238	View Code Duplication	if ($query) {
239		$db_prefix = elgg_get_config('dbprefix');
240		$params['joins'] = array("JOIN {$db_prefix}objects_entity oe ON e.guid = oe.guid");
241		$params['wheres'] = array("(oe.title LIKE '%" . $query . "%' OR oe.description LIKE '%" . $query . "%')");
242		}
243
244		if ($group) {
245		$params['container_guid'] = $group;
246		}
247
248		$ia = elgg_set_ignore_access(true);
249		$data = json_decode(elgg_list_entities_from_metadata($params));
250		elgg_set_ignore_access($ia);
251		}
252
253		return $data;
254		}
255
256		elgg_ws_expose_function(
257		"login.redirect",
258		"login_and_redirect",
259		array(
260		"user" => array('type' => 'string', 'required' => true),
261		"password" => array('type' => 'string', 'required' => true),
262		"redirect_en" => array('type' => 'string', 'required' => true),
263		"redirect_fr" => array('type' => 'string', 'required' => true),
264		"lang" => array('type' => 'string', 'required' => false, 'default' => "en")
265		),
266		'Login user into GCcollab and redirect them',
267		'POST',
268		false,
269		false
270		);
271
272		function login_and_redirect($user, $password, $redirect_en, $redirect_fr, $lang)
		0 ignored issues – show Coding Style introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report `login_and_redirect` uses the super-global variable `$_SERVER` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
273		{
274		$user_entity = is_numeric($user) ? get_user($user) : (strpos($user, '@') !== false ? get_user_by_email($user)[0] : get_user_by_username($user));
275		if (!$user_entity) {
276		header("Location: " . $_SERVER['HTTP_REFERER']);
277		exit();
		0 ignored issues – show Coding Style Compatibility introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report The function login_and_redirect() contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
278		}
279		if (!$user_entity instanceof ElggUser) {
280		header("Location: " . $_SERVER['HTTP_REFERER']);
281		exit();
		0 ignored issues – show Coding Style Compatibility introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report The function login_and_redirect() contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
282		}
283
284		$valid = elgg_authenticate($user_entity->username, $password);
285
286		if ($valid === true) {
287		login($user_entity);
288
289		if($lang == "fr"){
290		setcookie("gcconnex_lang", "fr");
291		header("Location: $redirect_fr");
292		exit();
		0 ignored issues – show Coding Style Compatibility introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report The function login_and_redirect() contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
293		} else {
294		setcookie("gcconnex_lang", "en");
295		header("Location: $redirect_en");
296		exit();
		0 ignored issues – show Coding Style Compatibility introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report The function login_and_redirect() contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
297		}
298		} else {
299		header("Location: " . $_SERVER['HTTP_REFERER']);
300		exit();
		0 ignored issues – show Coding Style Compatibility introduced 2018-03-09 04:10 UTC by Report Bug Copy Issue Report The function login_and_redirect() contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
301		}
302		}
303

gctools-outilsgc / gcconnex

Push — Api_opt ( 47a261...24829b )

functions.php ➔ query_the_posts() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity Many Parameters

Long Method

Many Parameters

Duplication Side-by-Side

Filter issues like