CWP\CWP\PageTypes\NewsHolder

Complexity

Total Complexity

1

Size/Duplication

Total Lines	49
Duplicated Lines	100 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	AllUpdates()	11	11	1

<?php

namespace CWP\CWP\PageTypes;

use SilverStripe\ORM\DataList;
use SilverStripe\ORM\PaginatedList;

class NewsHolder extends DatedUpdateHolder

This class seems to be duplicated in your project.

{
    private static $description = 'Container page for News Pages, provides news filtering and pagination';

The private property $description is not used, and could be removed.

    private static $allowed_children = [

The private property $allowed_children is not used, and could be removed.

        NewsPage::class,
    ];

    private static $default_child = NewsPage::class;

The private property $default_child is not used, and could be removed.

    private static $update_name = 'News';

The private property $update_name is not used, and could be removed.

    private static $update_class = NewsPage::class;

The private property $update_class is not used, and could be removed.

    private static $icon = 'cwp/images/icons/sitetree_images/news_listing.png';

The private property $icon is not used, and could be removed.

    public $pageIcon =  'images/icons/sitetree_images/news_listing.png';

    private static $singular_name = 'News Holder';

The private property $singular_name is not used, and could be removed.

    private static $plural_name = 'News Holders';

The private property $plural_name is not used, and could be removed.

    private static $table_name = 'NewsHolder';

The private property $table_name is not used, and could be removed.

    /**
     * Find all site's news items, based on some filters.
     * Omitting parameters will prevent relevant filters from being applied. The filters are ANDed together.
     *
     * @param string $className The name of the class to fetch.
     * @param int $parentID The ID of the holder to extract the news items from.
     * @param int $tagID The ID of the tag to filter the news items by.
     * @param string $dateFrom The beginning of a date filter range.
     * @param string $dateTo The end of the date filter range. If empty, only one day will be searched for.
     * @param int $year Numeric value of the year to show.
     * @param int $monthNumber Numeric value of the month to show.
     *
     * @returns DataList|PaginatedList
     */
    public static function AllUpdates(
        $className = NewsPage::class,
        $parentID = null,
        $tagID = null,
        $dateFrom = null,
        $dateTo = null,
        $year = null,
        $monthNumber = null
    ) {
        return parent::AllUpdates($className, $parentID, $tagID, $dateFrom, $dateTo, $year, $monthNumber)
            ->Sort('Date', 'DESC');
    }
}