Anax\Content\TFBCUtilities

Complexity

Total Complexity

6

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	3
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getActiveRoute()	0	8	2
B	getDataForAdditionalRoute()	0	28	2
A	addBaseurl2AnchorUrls()	0	13	1
A	getPublishTime()	0	5	1

<?php

namespace Anax\Content;

/**
 * File Based Content, code for loading additional content into view through 
 * data["meta"].
 */
trait TFBCUtilities
{
    /**
     * Support relative routes.
     *
     * @param string $route      to load.
     * @param string $routeIndex to use.
     *
     * @return string with active route.
     */
    private function getActiveRoute($route, $routeIndex)

This method is not used, and could be removed.

    {
        if (substr_compare($route, "./", 0, 2) === 0) {
            $route = dirname($routeIndex) . "/" . substr($route, 2);
        }

        return $route;
    }



    /**
     * Load view data for additional route, merged with meta if any.
     *
     * @param string $route to load.
     *
     * @return array with view data details.
     */
    private function getDataForAdditionalRoute($route)
    {
        // From configuration
         $filter = $this->config["textfilter"];

The property config does not exist. Did you maybe forget to declare it?

        // Get filtered content from route
        list(, , $filtered) =
            $this->mapRoute2Content($route);

It seems like mapRoute2Content() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        // Set data to be content of frontmatter, merged with meta
        $meta = $this->getMetaForRoute($route);

It seems like getMetaForRoute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $data = $filtered->frontmatter;
        $data = array_merge_recursive_distinct($meta, $data);
        unset($data["__toc__"]);
        unset($data["views"]);

        // Do phase 2 processing
        $new = $this->di->get("textFilter")->parse($filtered->text, $filter);

The property di does not exist. Did you maybe forget to declare it?

        
        // Creates urls based on baseurl
        $baseurl = isset($data["baseurl"])
            ? isset($data["baseurl"])
            : null;
        $this->addBaseurl2AnchorUrls($new, $baseurl);

$baseurl is of type boolean|null, but the function expects a string.

        $data["content"] = $new->text;

        return $data;
    }



    /**
     * Parse text, find and update all a href to use baseurl.
     *
     * @param object &$filtered with text and excerpt to process.
     * @param string $baseurl   add as baseurl for all relative urls.
     *
     * @return void.

The doc-type void. could not be parsed: Unknown type name "void." at position 0. (view supported doc-types)

     */
    private function addBaseurl2AnchorUrls(&$filtered, $baseurl)
    {
        $textf  = $this->di->get("textFilter");
        $url    = $this->di->get("url");

        // Use callback to url->create() instead of string concat
        $callback = function ($route) use ($url, $baseurl) {
            return $url->create($route, $baseurl);
        };

        $filtered->text =
            $textf->addBaseurlToRelativeLinks($filtered->text, $baseurl, $callback);
    }



    /**
     * Get published date.
     *
     * @param array $frontmatter with details on dates.
     *
     * @return integer as time for publish time.
     */
    private function getPublishTime($frontmatter)
    {
        list(, $date) = $this->di->get("view")->getPublishedDate($frontmatter);
        return strtotime($date);
    }
}