Sunnysideup\EcommerceGoogleShoppingFeed\Controllers\GoogleShoppingFeedController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getFileName()	0	3	1
A	index()	0	13	1

<?php

namespace Sunnysideup\EcommerceGoogleShoppingFeed\Controllers;

use SilverStripe\Control\ContentNegotiator;
use SilverStripe\Control\Controller;
use SilverStripe\Core\ClassInfo;
use SilverStripe\Core\Config\Config;
use SilverStripe\View\SSViewer;
use Sunnysideup\EcommerceGoogleShoppingFeed\Api\ProductCollectionForGoogleShoppingFeed;

/**
 * Controller for displaying the xml feed.
 *
 * <code>
 * http://site.com/shoppingfeed.xml
 * </code>
 *
 */
class GoogleShoppingFeedController extends Controller
{
    /**
     * @var array
     */
    private static $allowed_actions = [
        'index',
    ];

    private static $api_class = ProductCollectionForGoogleShoppingFeed::class;

    /**
     * Specific controller action for displaying a particular list of links
     * for a class.
     *
     * @return mixed
     */
    public function index()
    {
        Config::modify()->set(SSViewer::class, 'set_source_file_comments', false);
        Config::modify()->set(ContentNegotiator::class, 'enabled', false);
        // response header
        $header = $this->getResponse();
        $header->addHeader('Pragma', 'no-cache');
        $header->addHeader('Expires', 0);
        $header->addHeader('Content-Type', $this->getContentType());

The method getContentType() does not exist on Sunnysideup\EcommerceGoogleShoppingFeed\Controllers\GoogleShoppingFeedController. Since you implemented __call, consider adding a @method annotation.

        $header->addHeader('Content-Disposition', 'attachment; filename=' . $this->getFilename());
        $header->addHeader('X-Robots-Tag', 'noindex');

        return $this->renderWith(ClassInfo::shortName(static::class));
    }

    protected function getFileName(): string
    {
        return 'shoppingfeed.' . $this->getExtension();

The method getExtension() does not exist on Sunnysideup\EcommerceGoogleShoppingFeed\Controllers\GoogleShoppingFeedController. Since you implemented __call, consider adding a @method annotation.

    }
}