OfflineAgency\MongoAutoSync\Traits\PlainMongoTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	storeEditAllItems()	0	43	9

<?php

namespace OfflineAgency\MongoAutoSync\Traits;

use Exception;
use Illuminate\Http\Request;
use MongoDB\BSON\UTCDateTime;

trait PlainMongoTrait
{
    /**
     * @param Request $request
     * @param string $event
     * @param array $options
     * @throws Exception
     */
    public function storeEditAllItems(Request $request, string $event, array $options)

The parameter $options is not used and could be removed.

    {
        //Get the item name
        $items = $this->getItems();

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

        //Current Obj Create
        foreach ($items as $key => $item) {
            $is_ML = isML($item);
            $is_MD = isMD($item);

            $is_fillable = isFillable($item, $event);
            $is_skippable = $this->getIsSkippable($request->has($key));

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

            if ($is_skippable) {
                continue;
            } else {
                $this->checkRequestExistence(

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

                    $request,
                    $key
                );
            }

            if ($is_fillable) {
                if ($is_ML) {
                    if (is_null($this->$key)) {
                        $old_value = [];
                    } else {
                        $old_value = $this->$key;
                    }
                    $this->$key = ml($old_value, $request->input($key));
                } elseif ($is_MD) {
                    if ($request->input($key) == '' || $request->input($key) == null) {
                        $this->$key = null;
                    } else {
                        $this->$key = new UTCDateTime(new DateTime($request->input($key)));

The type OfflineAgency\MongoAutoSync\Traits\DateTime was not found. Did you mean DateTime? If so, make sure to prefix the type with \.

new OfflineAgency\MongoAutoSync\Traits\DateTime($request->input($key)) of type OfflineAgency\MongoAutoSync\Traits\DateTime is incompatible with the type integer expected by parameter $milliseconds of MongoDB\BSON\UTCDateTime::__construct().

                    }
                } else {
                    $this->$key = $request->input($key);
                }
            }
        }

        $this->save();

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

    }
}