LIN3S\WPSymfonyForm\Admin\Views\FormsTable

Complexity

Total Complexity

8

Size/Duplication

Total Lines	87
Duplicated Lines	16.09 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	no_items()	0	4	1
A	column_default()	0	10	3
A	get_columns()	0	7	1
A	get_sortable_columns()	0	7	1
A	prepare_items()	14	14	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 16 and the first side effect is on line 8.

namespace LIN3S\WPSymfonyForm\Admin\Views;

use LIN3S\WPSymfonyForm\Admin\Storage\Storage;

if (!class_exists('WP_List_Table')) {
    require_once(ABSPATH . 'wp-admin/includes/class-wp-list-table.php');
}

/**
 * Forms table.
 *
 * @author Beñat Espiña <benatespina@gmail.com>
 */
class FormsTable extends \WP_List_Table
{
    /**
     * The storage.
     *
     * @var Storage
     */
    private $storage;

    /**
     * Constructor.
     *
     * @param Storage $storage The storage
     */
    public function __construct(Storage $storage)
    {
        parent::__construct([
            'singular' => __('Form', \WPSymfonyForm::TRANSLATION_DOMAIN),
            'plural'   => __('Forms', \WPSymfonyForm::TRANSLATION_DOMAIN),
            'ajax'     => false,
        ]);

        $this->storage = $storage;
    }

    /**
     * {@inheritdoc}
     */
    public function no_items()
    {
        _e('No forms register yet', \WPSymfonyForm::TRANSLATION_DOMAIN);
    }

    /**
     * {@inheritdoc}
     */
    public function column_default($item, $column_name)
    {
        switch ($column_name) {
            case 'name':
            case 'link':
                return $item[$column_name];
            default:
                return print_r($item, true);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function get_columns()
    {
        return [
            'name' => __('Name', \WPSymfonyForm::TRANSLATION_DOMAIN),
            'link' => __('Link', \WPSymfonyForm::TRANSLATION_DOMAIN),
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function get_sortable_columns()
    {
        return [
            'name' => ['name', true],
            'link' => ['link', false],
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function prepare_items()

This method seems to be duplicated in your project.

    {
        $this->_column_headers = $this->get_column_info();

        $limit = $this->get_items_per_page('forms_per_page', 10);
        $offset = $this->get_pagenum();
        $this->items = $this->storage->findAll($limit, $offset);
        $total = count($this->items);

        $this->set_pagination_args([
            'total_items' => $total,
            'per_page'    => $limit,
        ]);
    }
}