Grido\Components\Columns\Link

Complexity

Total Complexity

6

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	formatValue()	0	4	1
A	formatHref()	0	8	2
A	formatText()	0	4	1
A	getAnchor()	0	22	2

<?php

/**
 * This file is part of the Grido (https://github.com/o5/grido)
 *
 * Copyright (c) 2011 Petr Bugyík (http://petr.bugyik.cz)
 *
 * For the full copyright and license information, please view
 * the file LICENSE.md that was distributed with this source code.
 */

namespace Grido\Components\Columns;

/**
 * Link column.
 *
 * @package     Grido
 * @subpackage  Components\Columns
 * @author      Petr Bugyík
 */
class Link extends Text
{
    /**
     * @param mixed $value
     * @return \Nette\Utils\Html
     */
    protected function formatValue($value)
    {
        return $this->getAnchor($value);
    }

    /**
     * @param string $value
     * @return string
     */
    protected function formatHref($value)
    {
        if (!preg_match('~^\w+://~i', $value)) {
            $value = "http://" . $value;
        }

        return $value;
    }

    /**
     * @param string $value
     * @return string
     */
    protected function formatText($value)
    {
        return preg_replace('~^https?://~i', '', $value);
    }

    /**
     * @param mixed $value
     * @return \Nette\Utils\Html
     */
    protected function getAnchor($value)
    {
        $truncate = $this->truncate;
        $this->truncate = NULL;

        $value = parent::formatValue($value);

It seems like you call parent on a different method (formatValue() instead of getAnchor()). Are you sure this is correct? If so, you might want to change this to $this->formatValue().

        $href = $this->formatHref($value);
        $text = $this->formatText($value);

        $anchor = \Nette\Utils\Html::el('a')
            ->setHref($href)
            ->setText($text)
            ->setTarget('_blank')
            ->setRel('noreferrer');

        if ($truncate) {
            $anchor->setText($truncate($text))
                ->setTitle($value);
        }

        return $anchor;
    }
}