Canvas\Api\Controllers\UserWebhooksController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	execute()	0	29	6
A	onConstruct()	0	10	1

<?php

declare(strict_types=1);

namespace Canvas\Api\Controllers;

use Canvas\Http\Exception\UnprocessableEntityException;
use Canvas\Models\UserWebhooks;
use Canvas\Validation;
use Phalcon\Http\Response;

The type Phalcon\Http\Response was not found. Maybe you did not declare it correctly or list all dependencies?

use Canvas\Webhooks;
use Phalcon\Validation\Validator\PresenceOf;

The type Phalcon\Validation\Validator\PresenceOf was not found. Maybe you did not declare it correctly or list all dependencies?

use function Canvas\Core\isJson;

/**
 * Class LanguagesController.
 *
 * @package Canvas\Api\Controllers
 *
 * @property Users $userData
 * @property Request $request
 * @property Config $config
 * @property Apps $app
 *
 */
class UserWebhooksController extends BaseController
{
    /*
     * fields we accept to create
     *
     * @var array
     */
    protected $createFields = [
        'webhooks_id',
        'url',
        'method',
        'format'
    ];

    /*
     * fields we accept to create
     *
     * @var array
     */
    protected $updateFields = [
        'webhooks_id',
        'url',
        'method',
        'format'
    ];

    /**
     * set objects.
     *
     * @return void
     */
    public function onConstruct()
    {
        $this->model = new UserWebhooks();

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

        $this->model->users_id = $this->userData->getId();
        $this->model->companies_id = $this->userData->currentCompanyId();
        $this->model->apps_id = $this->app->getId();
        $this->additionalSearchFields = [

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

            ['is_deleted', ':', '0'],
            ['apps_id', ':', $this->app->getId()],
            ['companies_id', ':', '0|' . $this->userData->currentCompanyId()],
        ];
    }

    /**
    * Given the weebhook id, we run a test for it.
    *
    * @param integer $id
    * @return Response
    */
    public function execute(string $name): Response
    {
        $request = $this->request->getPostData();
        $validation = new Validation();
        $validateJson = false;
        $validation->add('module', new PresenceOf(['message' => 'module is required to know what webhook to run']));
        $validation->add('action', new PresenceOf(['message' => 'action is required']));

        if (isset($request['data']) && !empty($request['data'])) {
            if (!isJson($request['data'])) {
                throw new UnprocessableEntityException('Data is not a valid json format text');
            }
            $validation->add('data', new PresenceOf(['message' => 'data is required']));
            $validateJson = true;
        }

        $validation->validate($request);

        $systemModule = $request['module'];
        $data = $validateJson ? json_decode($request['data'], true) : [];
        $action = $request['action'];
        $headers = isJson((string) $request['headers']) ? json_decode($request['headers'], true) : [];

        return $this->response(
            Webhooks::process(
                $systemModule,
                $data,
                $action,
                $headers
            )
        );
    }
}