Kaliop\eZObjectWrapperBundle\Core\Traits\RichTextConvertingEntity

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

42.86%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setRichTextConverter()	0	5	1
A	getHtml()	0	4	1
A	getHtmlForField()	0	4	1

<?php

namespace Kaliop\eZObjectWrapperBundle\Core\Traits;

use eZ\Publish\Core\FieldType\XmlText\Converter;

/**
 * Adds the capability to generate html version of RichText fields
 */
trait RichTextConvertingEntity
{
    protected $richTextConverter;

    /**
     * @param Converter $richTextConverter
     * @return $this
     */
    public function setRichTextConverter(Converter $richTextConverter)
    {
        $this->richTextConverter = $richTextConverter;
        return $this; // fluent interfaces for setters
    }

    /**
     * @param string $xml the value of an xml-text field
     * @return string
     */
    protected function getHtml($xml)
    {
        return $this->richTextConverter->convert($xml);
    }

    /**
     * @param $fieldName the identifier of an xml-text field
     * @return string
     */
    protected function getHtmlForField($fieldName)
    {
        return $this->getHtml($this->content()->getFieldValue($fieldName)->xml);

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

    }
}