App\Plugins\Pages\FrontendController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	sitemap()	0	11	2

<?php
/**
 * Created by PhpStorm.
 * User: Mark
 * Date: 20/05/2016
 * Time: 22:27.
 */

namespace App\Plugins\Pages;

use App\Model\Page;
use App\Plugins\PluginEngine;
use App\Classes\SitemapGenerator;
use App\Classes\Interfaces\Sitemap;
use App\Classes\Repositories\PageRepository;

/**
 * Class UserController.
 */
class FrontendController extends PluginEngine implements Sitemap
{
    /**
     * The sitemap function allows plugins to quickly and effectively
     * create and store new content for the SEO Sitemap Controller.
     *
     * @param SitemapGenerator $sitemap
     * @return SitemapGenerator
     */
    public function sitemap(SitemapGenerator $sitemap)
    {
        /** @var PageRepository $repository */
        $repository = app(PageRepository::class);

        /** @var Page $page */
        foreach ($repository->whereSitemap() as $page) {
            $sitemap->store(url($page->route()), $page->updated_at, 'bi-weekly', '1.0');

It seems like $page->route() can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $path of url() does only seem to accept string, maybe add an additional type check?

        }

        return $sitemap;
    }
}