OfflineAgency\MongoAutoSync\Traits\PlainMongoTrait::storeEditAllItems()

Complexity

Conditions	9
Paths	8

Size

Total Lines	43
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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)));
                    }
                } 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?

    }