ModPleio\AccessRequest

Complexity

Total Complexity

12

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getURL()	0	3	1
A	getIconURL()	0	3	1
A	getType()	0	3	1
B	approve()	0	25	3
A	decline()	0	14	2
A	remove()	0	3	1
A	sendEmail()	0	11	2

<?php
namespace ModPleio;

class AccessRequest {
    public function __construct($data) {
        $this->id = (int) $data->id;

The property id does not exist. Did you maybe forget to declare it?

        $this->user = unserialize($data->user);

The property user does not exist. Did you maybe forget to declare it?

        $this->time_created = $data->time_created;

The property time_created does not exist. Did you maybe forget to declare it?

    }

    public function getURL() {
        return $this->user["url"];
    }

    public function getIconURL() {
        return $this->user["icon"];
    }

    public function getType() {
        return "accessRequest";
    }

    public function approve() {
        $resourceOwner = new ResourceOwner($this->user);
        $loginHandler = new LoginHandler($resourceOwner);
        $site = elgg_get_site_entity();
        
        try {
            $user = $loginHandler->createUser();
            if ($user) {
                $this->remove();
                $this->sendEmail(
                    elgg_echo("pleio:approved:subject", [$site->name]), 
                    elgg_echo("pleio:approved:body", [
                        $site->name,
                        $site->url,
                        $site->url . "notifications/personal/" . $user->username
                    ])
                );
                return true;
            }
        } catch (\RegistrationException $e) {
            register_error($e->getMessage());
        }

        return false;
    }

    public function decline() {
        $site = elgg_get_site_entity();

        if ($this->remove()) {

The expression $this->remove() of type integer|false is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

            $this->sendEmail(
                elgg_echo("pleio:declined:subject", [$site->name]), 
                elgg_echo("pleio:declined:body", [$site->name])
            );

            return true;
        }

        return false;
    }

    private function remove() {
        return delete_data("DELETE FROM pleio_request_access WHERE id = {$this->id}");
    }

    private function sendEmail($subject, $body, $params = null) {
        $site = elgg_get_site_entity();

        if ($site->email) {
            $from = $site->email;
        } else {
            $from = "noreply@" . get_site_domain($site->guid);
        }

        return elgg_send_email($from, $this->user["email"], $subject, $body, $params);
    }
}