bakaphp / phalcon-api

api/controllers/AppsPlansController.php

<?php

declare(strict_types=1);

namespace Gewaer\Api\Controllers;

use Gewaer\Models\AppsPlans;
use Gewaer\Exception\UnauthorizedHttpException;
use Gewaer\Exception\NotFoundHttpException;
use Stripe\Token as StripeToken;
use Phalcon\Http\Response;
use Phalcon\Validation;
use Phalcon\Validation\Validator\PresenceOf;
use Gewaer\Exception\UnprocessableEntityHttpException;
use Phalcon\Cashier\Subscription;
use Baka\Auth\Models\UserCompanyApps;

/**
 * Class LanguagesController
 *
 * @package Gewaer\Api\Controllers
 *
 */
class AppsPlansController extends BaseController
{
    /*
     * fields we accept to create
     *
     * @var array
     */
    protected $createFields = [];

    /*
     * fields we accept to create
     *
     * @var array
     */
    protected $updateFields = [];

    /**
     * set objects
     *
     * @return void
     */
    public function onConstruct()
    {
        $this->model = new AppsPlans();
        $this->additionalSearchFields = [
            ['is_deleted', ':', 0],
            ['apps_id', ':', $this->app->getId()],

The property app does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

        ];
    }

    /**
     * Given the app plan stripe id , subscribe the current company to this aps
     *
     * @param string $stripeId
     * @return void
     */
    public function create(): Response
    {
        if (!$this->userData->hasRole('Default.Admins')) {

The property userData does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

            throw new UnauthorizedHttpException(_('You dont have permission to subscribe this apps'));
        }

        //Ok let validate user password
        $validation = new Validation();
        $validation->add('stripe_id', new PresenceOf(['message' => _('The plan is required.')]));
        $validation->add('number', new PresenceOf(['message' => _('Credit Card Number is required.')]));
        $validation->add('exp_month', new PresenceOf(['message' => _('Credit Card Number is required.')]));
        $validation->add('exp_year', new PresenceOf(['message' => _('Credit Card Number is required.')]));
        $validation->add('cvc', new PresenceOf(['message' => _('CVC is required.')]));

        //validate this form for password
        $messages = $validation->validate($this->request->getPost());
        if (count($messages)) {
            foreach ($messages as $message) {
                throw new UnprocessableEntityHttpException($message);
            }
        }

        $stripeId = $this->request->getPost('stripe_id');
        $company = $this->userData->defaultCompany;
        $cardNumber = $this->request->getPost('number');
        $expMonth = $this->request->getPost('exp_month');
        $expYear = $this->request->getPost('exp_year');
        $cvc = $this->request->getPost('cvc');

        $appPlan = $this->model->findFirstByStripeId($stripeId);

        if (!$appPlan) {
            throw new NotFoundHttpException(_('This plan doesnt exist'));
        }

        $userSubscription = Subscription::findFirst([
            'conditions' => 'user_id = ?0 and company_id = ?1 and apps_id = ?2 and is_deleted  = 0',
            'bind' => [$this->userData->getId(), $this->userData->default_company, $this->app->getId()]

The property app does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

        ]);

        if ($userSubscription) {

$userSubscription is of type Phalcon\Mvc\Model, thus it always evaluated to true.

            throw new NotFoundHttpException(_('You are already subscribed to this plan'));
        }

        $card = StripeToken::create([
            'card' => [
                'number' => $cardNumber,
                'exp_month' => $expMonth,
                'exp_year' => $expYear,
                'cvc' => $cvc,
            ],
        ], [
            'api_key' => $this->config->stripe->secret
        ])->id;

        //if fails it will throw exception
        if ($appPlan->free_trial_dates == 0) {
            $this->userData->newSubscription($appPlan->name, $appPlan->stripe_id, $company, $this->app)->create($card);
        } else {
            $this->userData->newSubscription($appPlan->name, $appPlan->stripe_id, $company, $this->app)->trialDays($appPlan->free_trial_dates)->create($card);
        }

        //update company app
        $companyApp = UserCompanyApps::findFirst([
            'conditions' => 'company_id = ?0 and apps_id = ?1',
            'bind' => [$this->userData->defaultCompany->getId(), $this->app->getId()]
        ]);

        //udpate the company app to the new plan
        if ($companyApp) {
            $companyApp->strip_id = $stripeId;
            $companyApp->subscriptions_id = $this->userData->subscription($appPlan->stripe_plan)->getId();
            $companyApp->update();
        }

        //sucess
        return $this->response($appPlan);
    }

    /**
     * Cancel a given subscription
     *
     * @param string $stripeId
     * @return boolean
     */
    public function edit($stripeId) : Response
    {
        $appPlan = $this->model->findFirstByStripeId($stripeId);

        if (!$appPlan) {
            throw new NotFoundHttpException(_('This plan doesnt exist'));
        }

        $userSubscription = Subscription::findFirst([
            'conditions' => 'user_id = ?0 and company_id = ?1 and apps_id = ?2 and is_deleted  = 0',
            'bind' => [$this->userData->getId(), $this->userData->default_company, $this->app->getId()]

The property app does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

The property userData does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

        ]);

        if (!$userSubscription) {

$userSubscription is of type Phalcon\Mvc\Model, thus it always evaluated to true.

            throw new NotFoundHttpException(_('No current subscription found'));
        }

        $this->userData->subscription($userSubscription->name)->swap($stripeId);

        //udpate the company app to the new plan
        if ($companyApp) {

The variable $companyApp seems to be never defined.

            $companyApp->strip_id = $stripeId;
            $companyApp->subscriptions_id = $this->userData->subscription($stripeId)->getId();
            $companyApp->update();
        }

        //return the new subscription plan
        return $this->response($appPlan);

The expression return $this->response($appPlan) returns the type Phalcon\Http\Response which is incompatible with the documented return type boolean.

    }

    /**
     * Cancel a given subscription
     *
     * @param string $stripeId
     * @return boolean
     */
    public function delete($stripeId): Response
    {
        $appPlan = $this->model->findFirstByStripeId($stripeId);

        if (!$appPlan) {
            throw new NotFoundHttpException(_('This plan doesnt exist'));
        }

        $userSubscription = Subscription::findFirst([
            'conditions' => 'user_id = ?0 and company_id = ?1 and apps_id = ?2 and is_deleted  = 0',
            'bind' => [$this->userData->getId(), $this->userData->default_company, $this->app->getId()]

The property userData does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

The property app does not exist on Gewaer\Api\Controllers\AppsPlansController. Since you implemented __get, consider adding a @property annotation.

        ]);

        if (!$userSubscription) {

$userSubscription is of type Phalcon\Mvc\Model, thus it always evaluated to true.

            throw new NotFoundHttpException(_('No current subscription found'));
        }

        $subscription = $this->userData->subscription($userSubscription->name)->cancel();
        $subscription->is_deleted = 1;
        $subscription->update();

        return $this->response($appPlan);

The expression return $this->response($appPlan) returns the type Phalcon\Http\Response which is incompatible with the documented return type boolean.

    }
}