Mongi\Mongicommerce\Http\Controllers\shop\ShopShipmentController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	107
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	page()	0	41	2
A	goToCheckout()	0	61	2

<?php
namespace Mongi\Mongicommerce\Http\Controllers\shop;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash;
use Mongi\Mongicommerce\Models\Cart;
use Mongi\Mongicommerce\Models\User;
use Mongi\Mongicommerce\Models\ProductItem;
use Mongi\Mongicommerce\Http\Controllers\Controller;
use Mongi\Mongicommerce\Libraries\Template;

class ShopShipmentController extends Controller
{
    public function page(){
        //if logged in
        $piva = '';
        $rag_soc = '';
        $first_name = '';
        $last_name = '';
        $address = '';
        $telephone = '';
        $email_sped = '';
        $floor = '';
        $province = '';
        $city = '';
        $cap = '';
        if(Auth::check()){
                $user = Auth::user();
                $piva = $user->piva;

Accessing piva on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $rag_soc = $user->company;

Accessing company on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $first_name = $user->first_name;

Accessing first_name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $last_name = $user->last_name;

Accessing last_name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $address = $user->address;

Accessing address on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $telephone = $user->telephone;

Accessing telephone on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $email_sped = $user->email;

Accessing email on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $floor = $user->floor;

Accessing floor on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $province = $user->province;

Accessing province on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $city = $user->city;

Accessing city on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                $cap = $user->cap;

Accessing cap on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                Template::MoveSessionToCart($user->id);

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        }

        return view('mongicommerce.pages.shipment',compact(
            'piva',
            'rag_soc',
            'first_name',
            'last_name',
            'address',
            'telephone',
            'email_sped',
            'floor',
            'province',
            'city',
            'cap'
        ));
    }

    public function goToCheckout(Request $r)
    {
        $r->validate([
            'first_name' => 'required',
            'last_name' => 'required',
            'address' => 'required',
            'telephone' => 'required',
            'email_sped' => 'required',
            'floor' => 'required',
            'province' => 'required',
            'city' => 'required',
            'cap' => 'required',
        ]);


        $piva = $r->get('piva');
        $rag_soc = $r->get('rag_soc');
        $first_name = $r->get('first_name');
        $last_name = $r->get('last_name');
        $address = $r->get('address');
        $telephone = $r->get('telephone');
        $email_sped = $r->get('email_sped');
        $floor = $r->get('floor');
        $province = $r->get('province');
        $city = $r->get('city');
        $cap = $r->get('cap');

        $user = User::where('email',$email_sped)->first();
        //if user exist
        if($user){
                $user->piva = $piva;
                $user->company = $rag_soc;
                $user->first_name = $first_name;
                $user->last_name = $last_name;
                $user->address = $address;
                $user->telephone = $telephone;
                $user->floor = $floor;
                $user->province = $province;
                $user->city = $city;
                $user->cap = $cap;
                $user->save();
        //if user doesn't exist
        }else{
            $user = new User();
            $user->piva = $piva;
            $user->company = $rag_soc;
            $user->first_name = $first_name;
            $user->last_name = $last_name;
            $user->email = $email_sped;
            $user->password = Hash::make('ciaociao');
            $user->address = $address;
            $user->telephone = $telephone;
            $user->floor = $floor;
            $user->province = $province;
            $user->city = $city;
            $user->cap = $cap;
            $user->save();
            Auth::login($user);
        }
        Template::MoveSessionToCart($user->id);
        return response()->json(['link' => route('shop.checkout')]);

    }
}