GalleryPage

Complexity

Total Complexity

1

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	getCMSFields()	0	25	1

<?php

class GalleryPage extends Page

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * @var string
     */
    private static $icon = 'flex-images-gallery/images/icons/gallery.png';

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

    /**
     * @var string
     */
    private static $description = 'Gallery page';

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

    /**
     * @var array
     */
    private static $db = array(

The property $db is not used and could be removed.

        'Height' => 'int',
    );

    /**
     * @var array
     */
    private static $has_many = array(

The property $has_many is not used and could be removed.

        'GalleryBlocks' => 'ContentBlock_Gallery',
    );

    /**
     * @var string
     */
    private static $many_many = array(

The property $many_many is not used and could be removed.

        'Images' => 'Image',
    );

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        // remove the content block
        $fields->removeByName('Content');

        // Add a gallery gridfield for the images.
        $fields->addFieldToTab(
            'Root.Main',
            SortableGalleryField::create('Images'),
            'Metadata'
        );

        // add configuration tab for the height
        $fields->addFieldsToTab(
            'Root.Configuration',
            array(
                NoticeMessage::create('This value defines the height of an image row (in pixel).'),
                TextField::create('Height'),
            )
        );

        return $fields;
    }
}

class GalleryPage_Controller extends Page_Controller

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * add the requirements
     */
    public function init()
    {
        parent::init();

        // add the used libs in.
        Requirements::css(FLEX_IMAGES_GALLERY_MODULE_DIR . '/css/styles.css');
        Requirements::javascript(THIRDPARTY_DIR . '/jquery/jquery.js');
        Requirements::javascript(FLEX_IMAGES_GALLERY_MODULE_DIR . '/javascript/jquery.flex-images.js');
        Requirements::javascript(FLEX_IMAGES_GALLERY_MODULE_DIR . '/javascript/lightbox.js');

        // add the defined rowHeight to the js file
        Requirements::javascriptTemplate(
            FLEX_IMAGES_GALLERY_MODULE_DIR . '/javascript/GalleryPage.template.js',
            array(
                'rowHeight' => ($this->Height) ? $this->Height : 180,
            )
        );
    }
}