offline-agency / laravel-mongo-auto-sync

src/Traits/PlainMongoTrait.php

<?php

namespace OfflineAgency\MongoAutoSync\Traits;

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

    }
}