MySociety\TheyWorkForYou\Subscription

Complexity

Total Complexity

73

Size/Duplication

Total Lines	345
Duplicated Lines	0 %

Test Coverage

Coverage

17.39%

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	delete_from_redis()	0	4	1
A	update_email()	0	2	1
C	__construct()	0	79	13
A	update_payment_method()	0	6	1
F	update_subscription()	0	73	20
A	quota_status()	0	6	1
A	createOrUpdateFromForm()	0	21	4
A	checkValidPlan()	0	2	2
A	invoices()	0	7	1
C	checkForErrors()	0	36	12
B	add_subscription()	0	40	6
A	getFields()	0	5	2
A	redis_reset_quota()	0	6	2
A	checkPaymentGivenIfNeeded()	0	5	5
A	update_customer()	0	2	1
A	redis_update_max()	0	5	1

<?php

namespace MySociety\TheyWorkForYou;

function add_vat($pence) {
    return round($pence * 1.2 / 100, 2);
}

class Subscription {
    public $stripe;
    public $upcoming;
    public $has_payment_data = false;

    private static $plans = ['twfy-1k', 'twfy-5k', 'twfy-10k', 'twfy-0k'];
    private static $prices = [2000, 5000, 10000, 30000];

    public function __construct($arg) {
        # User ID
        if (is_int($arg)) {
            $user = new \USER();
            $user->init($arg);
            $arg = $user;
        }

        $this->db = new \ParlDB();

The property db does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->redis = new Redis();

The property redis does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if (defined('TESTING')) {
            $this->api = new TestStripe("");

The property api does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        } else {
            $this->api = new Stripe(STRIPE_SECRET_KEY);
        }

        if (is_a($arg, 'User')) {
            # User object
            $this->user = $arg;

The property user does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->redis_prefix = "user:{$this->user->user_id}:quota:" . REDIS_API_NAME;

The property redis_prefix does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $q = $this->db->query('SELECT * FROM api_subscription WHERE user_id = :user_id', [
                ':user_id' => $this->user->user_id()])->first();
            if ($q) {
                $id = $q['stripe_id'];
            } else {
                return;
            }
        } else {
            # Assume Stripe ID string
            $id = $arg;
            $q = $this->db->query('SELECT * FROM api_subscription WHERE stripe_id = :stripe_id', [
                ':stripe_id' => $id])->first();
            if ($q) {
                $user = new \USER();
                $user->init($q['user_id']);
                $this->user = $user;
                $this->redis_prefix = "user:{$this->user->user_id}:quota:" . REDIS_API_NAME;
            } else {
                return;
            }
        }

        try {
            $this->stripe = $this->api->getSubscription([
                'id' => $id,
                'expand' => [
                    'customer.default_source',
                    'customer.invoice_settings.default_payment_method',
                    'latest_invoice.payment_intent',
                    'schedule.phases.items.plan',
                ],
            ]);
        } catch (\Stripe\Exception\InvalidRequestException $e) {
            $this->db->query('DELETE FROM api_subscription WHERE stripe_id = :stripe_id', [':stripe_id' => $id]);
            $this->delete_from_redis();
            return;
        }

        $this->has_payment_data = $this->stripe->customer->default_source || $this->stripe->customer->invoice_settings->default_payment_method;
        if ($this->stripe->customer->invoice_settings->default_payment_method) {
            $this->card_info = $this->stripe->customer->invoice_settings->default_payment_method->card;

The property card_info does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        } else {
            $this->card_info = $this->stripe->customer->default_source;
        }

        $data = $this->stripe;
        if ($data->discount && $data->discount->coupon && $data->discount->coupon->percent_off) {
            $this->actual_paid = add_vat(floor(

The property actual_paid does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                $data->plan->amount * (100 - $data->discount->coupon->percent_off) / 100
            ));
            $data->plan->amount = add_vat($data->plan->amount);
        } else {
            $data->plan->amount = add_vat($data->plan->amount);
            $this->actual_paid = $data->plan->amount;
        }

        try {
            $this->upcoming = $this->api->getUpcomingInvoice(["customer" => $this->stripe->customer->id]);
        } catch (\Stripe\Exception\ApiErrorException $e) {

Consider adding a comment why this CATCH block is empty.

        }
    }

    private function update_subscription($form_data) {
        if ($form_data['payment_method']) {
            $this->update_payment_method($form_data['payment_method']);
        }

        foreach ($this::$plans as $i => $plan) {
            if ($plan == $form_data['plan']) {
                $new_price = $this::$prices[$i];
                if ($form_data['coupon'] == 'charitable100') {
                    $new_price = 0;
                } elseif ($form_data['coupon'] == 'charitable50') {
                    $new_price /= 2;
                }
            }
            if ($plan == $this->stripe->plan->id) {
                $old_price = $this::$prices[$i];
                if ($this->stripe->discount && ($coupon = $this->stripe->discount->coupon)) {
                    if ($coupon->percent_off == 100) {
                        $old_price = 0;
                    } elseif ($coupon->percent_off == 50) {
                        $old_price /= 2;
                    }
                }
            }
        }

       if ($old_price >= $new_price) {

The variable $old_price does not seem to be defined for all execution paths leading up to this point.

The variable $new_price does not seem to be defined for all execution paths leading up to this point.

            if ($this->stripe->schedule) {
                \Stripe\SubscriptionSchedule::release($this->stripe->schedule);

The method Stripe\SubscriptionSchedule::release() is not static, but was called statically.

            }
            $schedule = \Stripe\SubscriptionSchedule::create(['from_subscription' => $this->stripe->id]);
            $phases = [
                [
                    'items' => [['price' => $schedule->phases[0]->items[0]->price]],
                    'start_date' => $schedule->phases[0]->start_date,
                    'end_date' => $schedule->phases[0]->end_date,
                    'proration_behavior' => 'none',
                    'default_tax_rates' => [STRIPE_TAX_RATE],

The constant MySociety\TheyWorkForYou\STRIPE_TAX_RATE was not found. Maybe you did not declare it correctly or list all dependencies?

                ],
                [
                    'items' => [['price' => $form_data['plan']]],
                    'iterations' => 1,
                    'metadata' => $form_data['metadata'],
                    'proration_behavior' => 'none',
                    'default_tax_rates' => [STRIPE_TAX_RATE],
                ],
            ];
            if ($schedule->phases[0]->discounts && $schedule->phases[0]->discounts[0]->coupon) {
                $phases[0]['discounts'] = [['coupon' => $schedule->phases[0]->discounts[0]->coupon]];
            }
            if ($form_data['coupon']) {
                $phases[1]['coupon'] = $form_data['coupon'];
            }
            \Stripe\SubscriptionSchedule::update($schedule->id, ['phases' => $phases]);
        }

        if ($old_price < $new_price) {
            $args = [
                'payment_behavior' => 'allow_incomplete',
                'plan' => $form_data['plan'],
                'metadata' => $form_data['metadata'],
                'cancel_at_period_end' => false, # Needed in Stripe 2018-02-28
                'proration_behavior' => 'always_invoice',
            ];
            if ($form_data['coupon']) {
                $args['coupon'] = $form_data['coupon'];
            } elseif ($this->stripe->discount) {
                $args['coupon'] = '';
            }
            if ($this->stripe->schedule) {
                \Stripe\SubscriptionSchedule::release($this->stripe->schedule);
            }
            \Stripe\Subscription::update($this->stripe->id, $args);
        }
    }

    private function update_customer($args) {
        $this->api->updateCustomer($this->stripe->customer->id, $args);
    }

    public function update_email($email) {
        $this->update_customer([ 'email' => $email ]);
    }

    public function update_payment_method($payment_method) {
        $payment_method = \Stripe\PaymentMethod::retrieve($payment_method);
        $payment_method->attach(['customer' => $this->stripe->customer->id]);
        $this->update_customer([
            'invoice_settings' => [
                'default_payment_method' => $payment_method,
            ],
        ]);
    }

    private function add_subscription($form_data) {
        # Create new Stripe customer and subscription
        $cust_params = ['email' => $this->user->email()];
        if ($form_data['stripeToken']) {
            $cust_params['source'] = $form_data['stripeToken'];
        }

        # At the point the customer is created, details such as postcode and
        # security code can be checked, and therefore fail
        try {
            $obj = $this->api->createCustomer($cust_params);
        } catch (\Stripe\Exception\CardException $e) {
            $body = $e->getJsonBody();
            $err  = $body['error'];
            $error = 'Sorry, we could not process your payment, please try again. ';
            $error .= 'Our payment processor returned: ' . $err['message'];
            unset($_POST['stripeToken']); # So card form is shown again
            return [ $error ];
        }

        $customer = $obj->id;

        if (!$form_data['stripeToken'] && !($form_data['plan'] == $this::$plans[0] && $form_data['coupon'] == 'charitable100')) {
            exit(1); # Should never reach here!

Using exit here is not recommended.

        }

        $obj = $this->api->createSubscription([
            'payment_behavior' => 'allow_incomplete',
            'expand' => ['latest_invoice.payment_intent'],
            'default_tax_rates' => [STRIPE_TAX_RATE],

The constant MySociety\TheyWorkForYou\STRIPE_TAX_RATE was not found. Maybe you did not declare it correctly or list all dependencies?

            'customer' => $customer,
            'plan' => $form_data['plan'],
            'coupon' => $form_data['coupon'],
            'metadata' => $form_data['metadata'],
        ]);
        $stripe_id = $obj->id;

        $this->db->query('INSERT INTO api_subscription (user_id, stripe_id) VALUES (:user_id, :stripe_id)', [
            ':user_id' => $this->user->user_id(),
            ':stripe_id' => $stripe_id,
        ]);
    }

    public function invoices() {
        $invoices = $this->api->getInvoices([
            'subscription' => $this->stripe->id,
            'limit' => 24,
        ]);
        $invoices = $invoices->data;
        return $invoices;
    }

    private function getFields() {
        $fields = ['plan', 'charitable_tick', 'charitable', 'charity_number', 'description', 'tandcs_tick', 'stripeToken', 'payment_method'];
        $this->form_data = [];

The property form_data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        foreach ($fields as $field) {
            $this->form_data[$field] = get_http_var($field);
        }
    }

    private function checkValidPlan() {
        return ($this->form_data['plan'] && in_array($this->form_data['plan'], $this::$plans));
    }

    private function checkPaymentGivenIfNeeded() {
        $payment_data = $this->form_data['stripeToken'] || $this->form_data['payment_method'];
        return ($this->has_payment_data || $payment_data || (
            $this->form_data['plan'] == $this::$plans[0]
                && in_array($this->form_data['charitable'], ['c', 'i'])
        ));
    }

    public function checkForErrors() {
        $this->getFields();
        $form_data = &$this->form_data;

        $errors = [];
        if ($form_data['charitable'] && !in_array($form_data['charitable'], ['c', 'i', 'o'])) {
            $form_data['charitable'] = '';
        }

        if (!$this->checkValidPlan()) {
            $errors[] = 'Please pick a plan';
        }

        if (!$this->checkPaymentGivenIfNeeded()) {
            $errors[] = 'You need to submit payment';
        }

        if (!$this->stripe && !$form_data['tandcs_tick']) {
            $errors[] = 'Please agree to the terms and conditions';
        }

        if (!$form_data['charitable_tick']) {
            $form_data['charitable'] = '';
            $form_data['charity_number'] = '';
            $form_data['description'] = '';
            return $errors;
        }

        if ($form_data['charitable'] == 'c' && !$form_data['charity_number']) {
            $errors[] = 'Please provide your charity number';
        }
        if ($form_data['charitable'] == 'i' && !$form_data['description']) {
            $errors[] = 'Please provide details of your project';
        }

        return $errors;
    }

    public function createOrUpdateFromForm() {
        $form_data = $this->form_data;

        $form_data['coupon'] = null;
        if (in_array($form_data['charitable'], ['c', 'i'])) {
            $form_data['coupon'] = 'charitable50';
            if ($form_data['plan'] == $this::$plans[0]) {
                $form_data['coupon'] = 'charitable100';
            }
        }

        $form_data['metadata'] = [
            'charitable' => $form_data['charitable'],
            'charity_number' => $form_data['charity_number'],
            'description' => $form_data['description'],
        ];

        if ($this->stripe) {
            $this->update_subscription($form_data);
        } else {
            return $this->add_subscription($form_data);
        }
    }

    public function redis_update_max($plan) {
        preg_match('#^twfy-(\d+)k#', $plan, $m);
        $max = $m[1] * 1000;
        $this->redis->set("$this->redis_prefix:max", $max);

The method set() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

        $this->redis->del("$this->redis_prefix:blocked");

The method del() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

    }

    public function redis_reset_quota() {
        $count = $this->redis->getset("$this->redis_prefix:count", 0);

The method getset() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

        if ($count !== null) {
            $this->redis->rpush("$this->redis_prefix:history", $count);

The method rpush() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

        }
        $this->redis->del("$this->redis_prefix:blocked");
    }

    public function delete_from_redis() {
        $this->redis->del("$this->redis_prefix:max");
        $this->redis->del("$this->redis_prefix:count");
        $this->redis->del("$this->redis_prefix:blocked");
    }

    public function quota_status() {
        return [
            'count' => floor($this->redis->get("$this->redis_prefix:count")),

The method get() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

            'blocked' => floor($this->redis->get("$this->redis_prefix:blocked")),
            'quota' => floor($this->redis->get("$this->redis_prefix:max")),
            'history' => $this->redis->lrange("$this->redis_prefix:history", 0, -1),

The method lrange() does not exist on MySociety\TheyWorkForYou\Redis. Since you implemented __call, consider adding a @method annotation.

        ];
    }
}