Helpers::getSiteEmail() - Code Metrics - Inspection of "Adding mod_pleio to Elgg" - gctools-outilsgc/gcconnex - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add-pleio-mod ( 6d68a5 )

unknown

created 2018-03-13 15:32 UTC

Helpers::getSiteEmail() A

↳ Parent: Helpers

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
cc	2
eloc	7
nc	2
nop	0
dl	0
loc	10
rs	9.4285
c	0
b	0
f	0
ccs	0
cts	9
cp	0
crap	6

<?php
namespace ModPleio;

class Helpers {
    public static function generateUsername($username) {
        $hidden = access_show_hidden_entities(true);

        while (strlen($username) < 4) {
            $username .= "0";
        }

        if (get_user_by_username($username)) {
            $i = 1;

            while (get_user_by_username($username . $i)) {
                $i++;
            }

            $result = $username . $i;
        } else {
            $result = $username;
        }

        access_show_hidden_entities($hidden);
        return $result;
    }

    public static function removeUser(\ElggUser $user) {
        if (!$user || !$user instanceof \ElggUser) {
            return false;
        }

        // make sure the user cannot login any more, also not with the "rememberme" cookie
        $user->ban("banned");
        $user->removeAdmin();
        $user->save();

        $result = Helpers::removeAllRelationships($user);
        $result &= Helpers::removeAllMetadata($user);
        $result &= Helpers::removeAllPrivateSettings($user);
        $result &= Helpers::anonymizeAccount($user);

        return $result;
    }

    public function removeAllRelationships(\ElggUser $user) {
        $dbprefix = elgg_get_config("dbprefix");
        $result = delete_data("DELETE FROM {$dbprefix}entity_relationships WHERE guid_one = {$user->guid} OR guid_two = {$user->guid}");

        return ($result !== false);
    }

    public function removeAllMetadata(\ElggUser $user) {
        $dbprefix = elgg_get_config("dbprefix");
        $result = delete_data("DELETE FROM {$dbprefix}metadata WHERE entity_guid = {$user->guid}");

        return ($result !== false);
    }

    public function removeAllPrivateSettings(\ElggUser $user) {
        $dbprefix = elgg_get_config("dbprefix");
        $result = delete_data("DELETE FROM {$dbprefix}private_settings WHERE entity_guid = {$user->guid}");

        return ($result !== false);
    }

    public static function anonymizeAccount(\ElggUser $user) {
        $dbprefix = elgg_get_config("dbprefix");

        // set site_guid to 0 as we do not want to display the entity in the listing any more
        $result = update_data("UPDATE {$dbprefix}entities SET
            site_guid = 0
            WHERE guid = {$user->guid}"
        );

        $result &= update_data("UPDATE {$dbprefix}users_entity SET 
            password = NULL,
            salt = NULL,
            password_hash = NULL,
            name = 'Verwijderde gebruiker',
            username = 'verwijderd{$user->guid}',
            email = NULL,
            pleio_guid = NULL,
            language = NULL,
            last_action = 0,
            prev_last_action = 0,
            last_login = 0,
            prev_last_login = 0
            WHERE guid = {$user->guid}"
        );

        _elgg_invalidate_cache_for_entity($user->guid);

        if (is_memcache_available()) {
            $newentity_cache = new \ElggMemcache("new_entity_cache");
            $newentity_cache->delete($user->guid);
        }

        elgg_trigger_event("update", "user", get_entity($user->guid));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        return ($result !== false);
    }

    public static function getSiteEmail() {
        global $CONFIG;

        $site = elgg_get_site_entity();
        if ($site->email) {
            return $site->email;
        } else {
            return "noreply@" . get_site_domain($CONFIG->site_guid);
        }
    }

    public static function signData($data) {
        $data = base64_encode(json_encode($data)) . ":" . time();
        $hash = hash_hmac("sha256", $data, get_site_secret());
        return "{$hash}:{$data}";
    }

    public static function loadSignedData($data) {
        list($input_hash, $data, $timestamp) = explode(":", $data);
        $verification_hash = hash_hmac("sha256", $data . ":" . $timestamp, get_site_secret());

        if ($verification_hash !== $input_hash) {
            return false;
        }

        if (time() > $timestamp + 3600*24) {
            return false;
        }

        $data = json_decode(base64_decode($data), true);

        if (!$data) {
            return false;
        }

        return $data;
    }

    public static function emailInWhitelist($email) {
        $domain = pleio_get_domain_from_email($email);
        return pleio_domain_in_whitelist($domain);
    }
}

1		<?php
2		namespace ModPleio;
3
4		class Helpers {
5		public static function generateUsername($username) {
6		$hidden = access_show_hidden_entities(true);
7
8		while (strlen($username) < 4) {
9		$username .= "0";
10		}
11
12	View Code Duplication	if (get_user_by_username($username)) {
13		$i = 1;
14
15		while (get_user_by_username($username . $i)) {
16		$i++;
17		}
18
19		$result = $username . $i;
20		} else {
21		$result = $username;
22		}
23
24		access_show_hidden_entities($hidden);
25		return $result;
26		}
27
28		public static function removeUser(\ElggUser $user) {
29		if (!$user \|\| !$user instanceof \ElggUser) {
30		return false;
31		}
32
33		// make sure the user cannot login any more, also not with the "rememberme" cookie
34		$user->ban("banned");
35		$user->removeAdmin();
36		$user->save();
37
38		$result = Helpers::removeAllRelationships($user);
39		$result &= Helpers::removeAllMetadata($user);
40		$result &= Helpers::removeAllPrivateSettings($user);
41		$result &= Helpers::anonymizeAccount($user);
42
43		return $result;
44		}
45
46		public function removeAllRelationships(\ElggUser $user) {
47		$dbprefix = elgg_get_config("dbprefix");
48		$result = delete_data("DELETE FROM {$dbprefix}entity_relationships WHERE guid_one = {$user->guid} OR guid_two = {$user->guid}");
49
50		return ($result !== false);
51		}
52
53		public function removeAllMetadata(\ElggUser $user) {
54		$dbprefix = elgg_get_config("dbprefix");
55		$result = delete_data("DELETE FROM {$dbprefix}metadata WHERE entity_guid = {$user->guid}");
56
57		return ($result !== false);
58		}
59
60		public function removeAllPrivateSettings(\ElggUser $user) {
61		$dbprefix = elgg_get_config("dbprefix");
62		$result = delete_data("DELETE FROM {$dbprefix}private_settings WHERE entity_guid = {$user->guid}");
63
64		return ($result !== false);
65		}
66
67		public static function anonymizeAccount(\ElggUser $user) {
68		$dbprefix = elgg_get_config("dbprefix");
69
70		// set site_guid to 0 as we do not want to display the entity in the listing any more
71		$result = update_data("UPDATE {$dbprefix}entities SET
72		site_guid = 0
73		WHERE guid = {$user->guid}"
74		);
75
76		$result &= update_data("UPDATE {$dbprefix}users_entity SET
77		password = NULL,
78		salt = NULL,
79		password_hash = NULL,
80		name = 'Verwijderde gebruiker',
81		username = 'verwijderd{$user->guid}',
82		email = NULL,
83		pleio_guid = NULL,
84		language = NULL,
85		last_action = 0,
86		prev_last_action = 0,
87		last_login = 0,
88		prev_last_login = 0
89		WHERE guid = {$user->guid}"
90		);
91
92		_elgg_invalidate_cache_for_entity($user->guid);
93
94		if (is_memcache_available()) {
95		$newentity_cache = new \ElggMemcache("new_entity_cache");
96		$newentity_cache->delete($user->guid);
97		}
98
99		elgg_trigger_event("update", "user", get_entity($user->guid));
		0 ignored issues – show Documentation introduced 2018-03-13 15:59 UTC by Report Bug Copy Issue Report `get_entity($user->guid)` is of type `object<ElggEntity>`, but the function expects a `string\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
100
101		return ($result !== false);
102		}
103
104		public static function getSiteEmail() {
105		global $CONFIG;
106
107		$site = elgg_get_site_entity();
108		if ($site->email) {
109		return $site->email;
110		} else {
111		return "noreply@" . get_site_domain($CONFIG->site_guid);
112		}
113		}
114
115		public static function signData($data) {
116		$data = base64_encode(json_encode($data)) . ":" . time();
117		$hash = hash_hmac("sha256", $data, get_site_secret());
118		return "{$hash}:{$data}";
119		}
120
121		public static function loadSignedData($data) {
122		list($input_hash, $data, $timestamp) = explode(":", $data);
123		$verification_hash = hash_hmac("sha256", $data . ":" . $timestamp, get_site_secret());
124
125		if ($verification_hash !== $input_hash) {
126		return false;
127		}
128
129		if (time() > $timestamp + 3600*24) {
130		return false;
131		}
132
133		$data = json_decode(base64_decode($data), true);
134
135		if (!$data) {
136		return false;
137		}
138
139		return $data;
140		}
141
142		public static function emailInWhitelist($email) {
143		$domain = pleio_get_domain_from_email($email);
144		return pleio_domain_in_whitelist($domain);
145		}
146		}

gctools-outilsgc / gcconnex

Push — add-pleio-mod ( 6d68a5 )

Helpers::getSiteEmail() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like