Nip\Mail\MessageBuilder\CanBuild

Complexity

Total Complexity

5

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	8	3
A	guardIsBuild()	0	7	2

<?php

namespace Nip\Mail\MessageBuilder;

trait CanBuild
{
    protected $isBuild = false;
    protected $buildMethods = ['buildFrom', 'buildRecipients', 'buildSubject', 'buildBody', 'buildAttachments'];

    protected function guardIsBuild()
    {
        if ($this->isBuild) {
            return;
        }
        $this->build();
        $this->isBuild = true;
    }

    protected function build()
    {
        foreach ($this->buildMethods as $method) {
            if (method_exists($this, $method)) {
                $this->{$method}();
            }
        }
        $this->runCallbacks();

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

    }
}