ByTIC\DocumentGenerator\PdfLetters\Controllers\Admin\Fields\PdfLetterFieldsControllerTrait::addNewModel()

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

0

<?php
    public function addNewModel()
    {
        /** @var PdfLetterTrait $item */
        $item = $this->getModelManager()->getNew();

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

        if ($this->pdfLetter) {
            $item->populateFromLetter($this->pdfLetter);

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

            $this->getView()->Breadcrumbs()->addItem(
                $this->getModelManager()->getLetterManager()->getLabel('add'),
                '#'
            );
            return $item;
        }

        return $this->forward('index', 'error');

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

    }