ilateral\SilverStripe\SlightlyBetterBulkLoader\BulkLoader_Result

Complexity

Total Complexity

15

Size/Duplication

Total Lines	169
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	ErrorCount()	0	4	1
A	addError()	0	12	1
A	merge()	0	7	1
A	getTotal()	0	4	1
A	getCreated()	0	4	1
A	getUpdated()	0	4	1
A	getDeleted()	0	4	1
A	getErrors()	0	4	1
A	getMessagesArray()	0	31	4
A	getMessagesString()	0	4	1
A	getMessageType()	0	10	2

<?php

namespace ilateral\SilverStripe\SlightlyBetterBulkLoader;

use SilverStripe\ORM\ValidationResult;
use SilverStripe\Dev\BulkLoader_Result as SS_BulkLoader_Result;

/**
 * Custom bulk loader result that also tracks errors in submission
 */
class BulkLoader_Result extends SS_BulkLoader_Result
{

    /**
     * List of errors tracked, each error should be an error message
     * (as a string).
     *
     * @var array (see {@link $created})
     */
    protected $errors = [];

    /**
     * Return the number of errors
     *
     * @return int
     */
    public function ErrorCount()
    {
        return count($this->errors);
    }

    /**
     * Add an error message to the stack
     *
     * @param string $message The error message
     *
     * @return self
     */
    public function addError($message, $id = null)
    {
        $this->lastChange = [
            'Message' => $message,
            'ID' => $id,
            '_BulkLoaderMessage' => $message
        ];
        $this->errors[] = $message;
        $this->lastChange['ChangeType'] = 'error';

        return $this;
    }

    /**
     * Merges another BulkLoader_Result into this one.
     *
     * @param BulkLoader_Result $other

Should the type for parameter $other not be SS_BulkLoader_Result?

     */
    public function merge(SS_BulkLoader_Result $other)
    {
        $this->created = array_merge($this->created, $other->getCreated());

The method getCreated() does not exist on SilverStripe\Dev\BulkLoader_Result. Did you maybe mean create()?

        $this->updated = array_merge($this->updated, $other->getUpdated());

The method getUpdated() does not exist on SilverStripe\Dev\BulkLoader_Result. Did you maybe mean Updated()?

        $this->deleted = array_merge($this->deleted, $other->getDeleted());

The method getDeleted() does not exist on SilverStripe\Dev\BulkLoader_Result. Did you maybe mean Deleted()?

        $this->errors = array_merge($this->errors, $other->getErrors());

It seems like you code against a specific sub-type and not the parent class SilverStripe\Dev\BulkLoader_Result as the method getErrors() does only exist in the following sub-classes of SilverStripe\Dev\BulkLoader_Result: ilateral\SilverStripe\SlightlyBetterBulkLoader\BulkLoader_Result. Maybe you want to instanceof check for one of these explicitly?

    }

    /**
     * Get he total number of results tracked
     *
     * @return int
     */
    public function getTotal()
    {
        return $this->CreatedCount() + $this->UpdatedCount() + $this->DeletedCount() + $this->ErrorCount();
    }

    /**
     * Get created array
     *
     * @return array
     */
    public function getCreated()
    {
        return $this->created;
    }

    /**
     * Get updated array
     *
     * @return array
     */
    public function getUpdated()
    {
        return $this->updated;
    }

    /**
     * Get updated array
     *
     * @return array
     */
    public function getDeleted()
    {
        return $this->deleted;
    }

    /**
     * Get errors array
     *
     * @return array
     */
    public function getErrors()
    {
        return $this->errors;
    }

    /**
     * Get an array of all messages stored
     *
     * @return array
     */
    public function getMessagesArray()
    {
        $messages = [];

        if ($this->CreatedCount() > 0) {
            $messages[] = _t(
                'SilverStripe\\Admin\\ModelAdmin.IMPORTEDRECORDS',
                "Imported {count} records.",
                ['count' => $this->CreatedCount()]
            );
        }

        if ($this->UpdatedCount() > 0) {
            $messages[] = _t(
                'SilverStripe\\Admin\\ModelAdmin.UPDATEDRECORDS',
                "Updated {count} records.",
                ['count' => $this->UpdatedCount()]
            );
        }

        if ($this->DeletedCount() > 0) {
            $messages[] = _t(
                'SilverStripe\\Admin\\ModelAdmin.DELETEDRECORDS',
                "Deleted {count} records.",
                ['count' => $this->DeletedCount()]
            );
        }
        
        // Finally include any errors
        return array_merge($messages, $this->getErrors());
    }

    /**
     * Return a string of all messages (that can be rendered into a message window)
     *
     * @param bool $html Add the newline as a HTML "<br/>"

There is no parameter named $html. Was it maybe removed?

     *
     * @return string
     */
    public function getMessagesString($delimiter = ";")
    {
        return implode($delimiter, $this->getMessagesArray());
    }

    /**
     * Get the "type" of message (using SilverStripe's Validation result)
     *
     * @return string
     */
    public function getMessageType()
    {
        $type = ValidationResult::TYPE_GOOD;

        if (count($this->getErrors()) > 0) {
            $type = ValidationResult::TYPE_ERROR;
        }

        return $type;
    }
}