Modules\Core\Internationalisation\Translatable

Complexity

Total Complexity

15

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

1 Method

Rating	Name	Duplication	Size	Complexity
B	save()	0	30	5

<?php namespace Modules\Core\Internationalisation;

trait Translatable
{
    use \Dimsav\Translatable\Translatable;

    public function save(array $options = array())
    {
        $tempTranslations = $this->translations;

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

        if ($this->exists) {

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

            if (count($this->getDirty()) > 0) {

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

                // If $this->exists and dirty, parent::save() has to return true. If not,
                // an error has occurred. Therefore we shouldn't save the translations.
                if (parent::save($options)) {
                    $this->translations = $tempTranslations;

                    return $this->saveTranslations();
                }

                return false;
            } else {
                // If $this->exists and not dirty, parent::save() skips saving and returns
                // false. So we have to save the translations
                $this->translations = $tempTranslations;

                return $this->saveTranslations();
            }
        } elseif (parent::save($options)) {
            // We save the translations only if the instance is saved in the database.
            $this->translations = $tempTranslations;

            return $this->saveTranslations();
        }

        return false;
    }
}