mQueue\View\Helper\AlternateFormats

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

91.67%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	alternateFormats()	0	23	5

<?php

namespace mQueue\View\Helper;

use Zend_Registry;
use Zend_View_Helper_Abstract;

class AlternateFormats extends Zend_View_Helper_Abstract
{
    protected static $supportedFormats = [
        'rss' => [
            'name' => 'RSS',
        ],
        'csv' => [
            'name' => 'CSV',
        ],
    ];

    /**
     * Returns a string of HTML links for end-user, and also append 'alternate' links to HTML header
     * @param array $formats
     * @param string $title

Should the type for parameter $title not be string|null?

     * @return string
     */
    public function alternateFormats(array $formats, $title = null)

Please use __construct() instead of a PHP4-style constructor that is named after the class.

    {
        $formatLinks = [];
        foreach ($formats as $format => $url) {
            // Inject format and locale parameters
            if (mb_strpos($url, '?') === false) {
                $url .= '?';
            } else {
                $url .= '&';
            }

            $url .= 'format=' . $format . '&lang=' . Zend_Registry::get('Zend_Locale')->getLanguage();

            $formatLinks[] = '<a class="' . $format . '" href="' . $url . '">' . self::$supportedFormats[$format]['name'] . '</a>';
            if ($title && isset(self::$supportedFormats[$format]['mime'])) {
                $this->view->headLink()->appendAlternate($url, self::$supportedFormats[$format]['mime'], $title);
            }
        }

        $result = '<p class="alternateFormats">' . $this->view->translate('Also available in:') . ' ' . implode(' | ', $formatLinks) . '</p>';

        return $result;
    }
}