AdminNewSingleProductController::createNewSingleProduct()

Complexity

Conditions	3
Paths	4

Size

Total Lines	58
Code Lines	47

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function createNewSingleProduct(Request $r){
        $r->validate([
            'category_id' => 'required',
            'quantity' => 'required',
            'price' => 'required',
            'image' => 'required',
        ]);

        $configuration_fields = json_decode($r->get('configuration_fields'),true);

It seems like $r->get('configuration_fields') can also be of type null; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

        $quantity = $r->get('quantity');
        $price = $r->get('price');
        $product_name = $r->get('product_name');
        $product_description = $r->get('product_description');
        $get_image = $r->get('image');
        $category_id = $r->get('category_id');

        $product = new Product();
        $product->category_id = $category_id;
        $product->name = $product_name;
        $product->single_product = true;
        $product->description = $product_description;
        $product->image = null;
        $product->save();

        $base64_str = substr($get_image, strpos($get_image, ",")+1);

It seems like $get_image can also be of type null; however, parameter $haystack of strpos() does only seem to accept string, maybe add an additional type check?

It seems like $get_image can also be of type null; however, parameter $string of substr() does only seem to accept string, maybe add an additional type check?

        $image = base64_decode($base64_str);
        $destinationPath = public_path().'/uploads/products_img/'.$product->id.'/'.$product->id.'/';
        $destinationPathDB = url('/').'/uploads/products_img/'.$product->id.'/'.$product->id.'/';

        if(!File::isDirectory($destinationPath)){
            File::makeDirectory($destinationPath, $mode = 0777, true, true);
        }

        $image_name = time().'.'.'jpeg';
        $path_file = $destinationPath.$image_name;
        $dbPath = $destinationPathDB.$image_name;
        file_put_contents($path_file, $image);

        $product->image = $dbPath;
        $product->save();

        $product_item = new ProductItem();
        $product_item->product_id = $product->id;
        $product_item->category_id = $category_id;
        $product_item->name = $product_name;
        $product_item->description = $product_description;
        $product_item->image = $dbPath;
        $product_item->price = $price;
        $product_item->quantity = $quantity;
        $product_item->save();

        foreach ($configuration_fields as $conf_field){
            $conf_fields_obj = (object) $conf_field;
            $configuration_field = new ProductConfigurationField();
            $configuration_field->product_item_id = $product_item->id;
            $configuration_field->config_field_id = $conf_fields_obj->configuration_field_id;
            $configuration_field->value = $conf_fields_obj->configuration_field_value;
            $configuration_field->save();
        }