App\Http\Controllers\RssController::rss()

Complexity

Conditions	30
Paths	> 20000

Size

Total Lines	114
Code Lines	73

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	RssController::showRssDesc()	0	43	3
F	RssController::fullFeedRss()	0	18	12
A	RssController::MyShowsRss()	0	12	6
A	RssController::MyMoviesRss()	0	15	4

<?php

namespace App\Http\Controllers;

use App\Models\User;
use App\Models\Category;
use Blacklight\http\RSS;
use App\Models\UserRequest;
use Illuminate\Http\Request;
use Blacklight\utility\Utility;
use Illuminate\Support\Facades\Auth;

class RssController extends BasePageController
{
    /**
     * @param \Illuminate\Http\Request $request
     * @return void
     * @throws \Exception
     * @throws \Throwable
     */
    public function MyMoviesRss(Request $request)
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);
        $offset = 0;

        $user = $this->userCheck($request->all());

$request->all() of type array is incompatible with the type Illuminate\Http\Request expected by parameter $request of App\Http\Controllers\RssController::userCheck().

        $outputXML = (! ($request->has('o') && $request->input('o') === 'json'));

        $userNum = ($request->has('num') && is_numeric($request->input('num')) ? abs($request->input('num')) : 0);

        $relData = $rss->getMyMoviesRss($userNum, $user['user_id'], User::getCategoryExclusionById($user['user_id']));

It seems like $userNum can also be of type double; however, parameter $limit of Blacklight\http\RSS::getMyMoviesRss() does only seem to accept integer, maybe add an additional type check?

        $rss->output($relData, $user['params'], $outputXML, $offset, 'rss');
    }

    /**
     * @param \Illuminate\Http\Request $request
     * @throws \Exception
     * @throws \Throwable
     */
    public function MyShowsRss(Request $request)
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);
        $offset = 0;
        $user = $this->userCheck($request->all());

$request->all() of type array is incompatible with the type Illuminate\Http\Request expected by parameter $request of App\Http\Controllers\RssController::userCheck().

        $userAirDate = $request->has('airdate') && is_numeric($request->input('airdate')) ? abs($request->input('airdate')) : -1;
        $userNum = ($request->has('num') && is_numeric($request->input('num')) ? abs($request->input('num')) : 0);
        $relData = $rss->getShowsRss($userNum, $user['user_id'], User::getCategoryExclusionById($user['user_id']), $userAirDate);

It seems like $userAirDate can also be of type double; however, parameter $airDate of Blacklight\http\RSS::getShowsRss() does only seem to accept integer, maybe add an additional type check?

It seems like $userNum can also be of type double; however, parameter $limit of Blacklight\http\RSS::getShowsRss() does only seem to accept integer, maybe add an additional type check?

        $outputXML = (! ($request->has('o') && $request->input('o') === 'json'));

        $rss->output($relData, $user['params'], $outputXML, $offset, 'rss');
    }

    /**
     * @param \Illuminate\Http\Request $request
     * @throws \Exception
     * @throws \Throwable
     */
    public function fullFeedRss(Request $request)
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);
        $offset = 0;
        $user = $this->userCheck($request->all());

$request->all() of type array is incompatible with the type Illuminate\Http\Request expected by parameter $request of App\Http\Controllers\RssController::userCheck().

        $userAirDate = $request->has('airdate') && is_numeric($request->input('airdate')) ? abs($request->input('airdate')) : -1;
        $userNum = ($request->has('num') && is_numeric($request->input('num')) ? abs($request->input('num')) : 0);
        $userLimit = $request->has('limit') && is_numeric($request->input('limit')) ? $request->input('limit') : 100;
        $userShow = $userAnidb = -1;
        if ($request->has('show')) {
            $userShow = ((int) $request->input('show') === 0 ? -1 : $request->input('show') + 0);
        } elseif ($request->has('anidb')) {
            $userAnidb = ((int) $request->input('anidb') === 0 ? -1 : $request->input('anidb') + 0);
        }
        $outputXML = (! ($request->has('o') && $request->input('o') === 'json'));
        $relData = $rss->getRss([0], $userShow, $userAnidb, $user['user_id'], $userAirDate, $userLimit, $userNum);

It seems like $userAirDate can also be of type double; however, parameter $airDate of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

It seems like $userNum can also be of type double; however, parameter $offset of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

        $rss->output($relData, $user['params'], $outputXML, $offset, 'rss');
    }

    /**
     * @throws \Exception
     */
    public function showRssDesc()
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);

        $title = 'Rss Info';
        $meta_title = 'Rss Nzb Info';
        $meta_keywords = 'view,nzb,description,details,rss,atom';
        $meta_description = 'View information about NNTmux RSS Feeds.';

        $firstShow = $rss->getFirstInstance('videos_id', 'releases', 'id');
        $firstAni = $rss->getFirstInstance('anidbid', 'releases', 'id');

        if ($firstShow !== null) {
            $this->smarty->assign('show', $firstShow->videos_id);

The method assign() does not exist on Illuminate\Foundation\Application.

The property videos_id does not seem to exist on Illuminate\Database\Query\Builder.

        } else {
            $this->smarty->assign('show', 1);
        }

        if ($firstAni !== null) {
            $this->smarty->assign('anidb', $firstAni->anidbid);

The property anidbid does not seem to exist on Illuminate\Database\Query\Builder.

        } else {
            $this->smarty->assign('anidb', 1);
        }

        $this->smarty->assign(
            [
                'categorylist'       => Category::getCategories(true, $this->userdata['categoryexclusions']),
                'parentcategorylist' => Category::getForMenu($this->userdata['categoryexclusions']),
            ]
        );

        $content = $this->smarty->fetch('rssdesc.tpl');

The method fetch() does not exist on Illuminate\Foundation\Application.

        $this->smarty->assign(
            [
                'content' => $content,
                'title' => $title,
                'meta_title' => $meta_title,
                'meta_keywords' => $meta_keywords,
                'meta_description' => $meta_description,
            ]
        );
        $this->pagerender();
    }

    public function cartRss(Request $request)
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);
        $offset = 0;
        $user = $this->userCheck($request->all());

$request->all() of type array is incompatible with the type Illuminate\Http\Request expected by parameter $request of App\Http\Controllers\RssController::userCheck().

        $outputXML = (! ($request->has('o') && $request->input('o') === 'json'));
        $userAirDate = $request->has('airdate') && is_numeric($request->input('airdate')) ? abs($request->input('airdate')) : -1;
        $userNum = ($request->has('num') && is_numeric($request->input('num')) ? abs($request->input('num')) : 0);
        $userLimit = $request->has('limit') && is_numeric($request->input('limit')) ? $request->input('limit') : 100;
        $userShow = $userAnidb = -1;
        if ($request->has('show')) {
            $userShow = ((int) $request->input('show') === 0 ? -1 : $request->input('show') + 0);
        } elseif ($request->has('anidb')) {
            $userAnidb = ((int) $request->input('anidb') === 0 ? -1 : $request->input('anidb') + 0);
        }

        $relData = $rss->getRss([-2], $userShow, $userAnidb, $user['user_id'], $userAirDate, $userLimit, $userNum);

It seems like $userNum can also be of type double; however, parameter $offset of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

It seems like $userAirDate can also be of type double; however, parameter $airDate of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

        $rss->output($relData, $user['params'], $outputXML, $offset, 'rss');
    }

    /**
     * @param \Illuminate\Http\Request $request
     * @return \Illuminate\Http\JsonResponse
     * @throws \Exception
     * @throws \Throwable
     */
    public function categoryFeedRss(Request $request)
    {
        $this->setPrefs();
        $rss = new RSS(['Settings' => $this->settings]);
        $offset = 0;
        if (! $request->has('id')) {
            return response()->json(['error' => 'Category ID is missing'], '403');
        }

        $user = $this->userCheck($request->all());

$request->all() of type array is incompatible with the type Illuminate\Http\Request expected by parameter $request of App\Http\Controllers\RssController::userCheck().

        $categoryId = (int) $request->input('id');
        $userAirDate = $request->has('airdate') && is_numeric($request->input('airdate')) ? abs($request->input('airdate')) : -1;
        $userNum = ($request->has('num') && is_numeric($request->input('num')) ? abs($request->input('num')) : 0);
        $userLimit = $request->has('limit') && is_numeric($request->input('limit')) ? $request->input('limit') : 100;
        $userShow = $userAnidb = -1;
        if ($request->has('show')) {
            $userShow = ((int) $request->input('show') === 0 ? -1 : $request->input('show') + 0);
        } elseif ($request->has('anidb')) {
            $userAnidb = ((int) $request->input('anidb') === 0 ? -1 : $request->input('anidb') + 0);
        }
        $outputXML = (! ($request->has('o') && $request->input('o') === 'json'));
        $relData = $rss->getRss($categoryId, $userShow, $userAnidb, $user['user_id'], $userAirDate, $userLimit, $userNum);

It seems like $userAirDate can also be of type double; however, parameter $airDate of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

It seems like $userNum can also be of type double; however, parameter $offset of Blacklight\http\RSS::getRss() does only seem to accept integer, maybe add an additional type check?

        $rss->output($relData, $user['params'], $outputXML, $offset, 'rss');
    }

    /**
     * @param \Illuminate\Http\Request $request
     * @return array|\Illuminate\Http\JsonResponse
     * @throws \Throwable
     */
    private function userCheck(Request $request)
    {
        // User requested a feed, ensure user is passing a valid api_token.
        if (Auth::check()) {
            $uid = $this->userdata->id;
            $rssToken = $this->userdata['api_token'];
            $maxRequests = $this->userdata->role->apirequests;

The property apirequests does not seem to exist on Spatie\Permission\Models\Role. Are you sure there is no database migration missing?

        } else {
            if (! $request->has('api_token')) {
                return response()->json(['error' => 'API key is required for viewing the RSS!'], 403);
            }

            $res = User::getByRssToken($request->input('api_token'));

            if ($res === null) {
                return response()->json(['error' => 'Invalid RSS token'], 403);
            }

            $uid = $res['id'];
            $rssToken = $res['api_token'];
            $maxRequests = $res->role->apirequests;

            if ($res->hasRole('Disabled')) {
                return response()->json(['error' => 'Your account is disabled'], 403);
            }
        }

        if (UserRequest::getApiRequests($uid) > $maxRequests) {
            return response()->json(['error' => 'You have reached your daily limit for API requests!'], 403);
        } else {
            UserRequest::addApiRequest($rssToken, $request->getRequestUri());

It seems like $rssToken can also be of type Illuminate\Database\Eloquent\Builder&Illuminate\Database\Eloquent\Relations\Relation and Illuminate\Database\Eloquent\Relations\Relation; however, parameter $token of App\Models\UserRequest::addApiRequest() does only seem to accept string, maybe add an additional type check?

        }
        $params =
            [
                'dl'       => $request->has('dl') && $request->input('dl') === '1' ? '1' : '0',
                'del'      => $request->has('del') && $request->input('del') === '1' ? '1' : '0',
                'extended' => 1,
                'uid'      => $uid,
                'token'    => $rssToken,
            ];

        return ['user_id' => $uid, 'rss_token' => $rssToken, 'max_requests' => $maxRequests, 'params' => $params];
    }
}