App\Http\Controllers\Front\CheckoutController::postCheckout()

Complexity

Conditions	8
Paths	148

Size

Total Lines	68
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	1	Features	0

<?php
    public function postCheckout(CheckoutRequest $request) {
        try {
            if (\Cart::getSubTotal() > 0) {
                $v = $this->validate($request, [

Are you sure the assignment to $v is correct as $this->validate($request, array('payment_gateway' => 'required')) (which targets Illuminate\Foundation\Validation\ValidatesRequests::validate()) seems to always return null.

$v is not used, you could remove the assignment.

                    'payment_gateway' => 'required',
                ]);
            }
            if (!$this->setting->where('id', 1)->first()) {
                return redirect()->back()->with('fails', 'Complete your settings');
            }

            $httpMethod = $request->method();
            //dd($httpMethod);
            if ($httpMethod == 'PATCH') {
                //do update the auth user

                \Auth::user()->fill($request->input())->save();
            } elseif ($httpMethod == 'POST') {
                
                
                //do saving user

                $str = str_random(8);
                $password = \Hash::make($str);
                $this->user->password = $password;

The property password does not exist on object<App\User>. Since you implemented __set, maybe consider adding a @property annotation.

                $this->user->fill($request->input())->save();

It seems like $request->input() targeting Illuminate\Http\Request::input() can also be of type string; however, Illuminate\Database\Eloquent\Model::fill() does only seem to accept array, maybe add an additional type check?

                
                $token = str_random(40);
                $activate = new \App\Model\User\AccountActivate();
                $activate->create(['email' => $this->user->email, 'token' => $token]);

The property email does not exist on object<App\User>. Since you implemented __get, maybe consider adding a @property annotation.

                $url = url("activate/$token");

                //send welcome note
                $settings = $this->setting->where('id', 1)->first();
                $from = $settings->email;
                $to = $this->user->email;

The property email does not exist on object<App\User>. Since you implemented __set, maybe consider adding a @property annotation.

                $data = $this->template->where('id', $settings->where('id', 1)->first()->welcome_mail)->first()->data;
                $replace = ['name' => $this->user->first_name . ' ' . $this->user->last_name, 'username' => $this->user->email, 'password' => $str,'url'=>$url];

The property first_name does not exist on object<App\User>. Since you implemented __get, maybe consider adding a @property annotation.

The property last_name does not exist on object<App\User>. Since you implemented __get, maybe consider adding a @property annotation.

The property email does not exist on object<App\User>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->templateController->Mailing($from, $to, $data, 'Welcome Email', $replace);

                \Auth::login($this->user);
            }

            /*
             * Do order, invoicing etc
             */
            $invoice_controller = new \App\Http\Controllers\Order\InvoiceController();
            $invoice = $invoice_controller->GenerateInvoice();
            $payment_method = $request->input('payment_gateway');
            if (!$payment_method) {
                $payment_method = 'free';
            }
            $invoiceid = $invoice->id;

The property id does not exist on object<App\Model\Order\Invoice>. Since you implemented __set, maybe consider adding a @property annotation.

            $amount = $invoice->grand_total;

The property grand_total does not exist on object<App\Model\Order\Invoice>. Since you implemented __set, maybe consider adding a @property annotation.

            $payment = $invoice_controller->doPayment($payment_method, $invoiceid, $amount);

Are you sure the assignment to $payment is correct as $invoice_controller->doPayment($payment_method, $invoiceid, $amount) (which targets App\Http\Controllers\Order\InvoiceController::doPayment()) seems to always return null.

$payment is not used, you could remove the assignment.

            //trasfer the control to event if cart price is not equal 0
            if (Cart::getSubTotal() != 0) {
                \Event::fire(new \App\Events\PaymentGateway(['request' => $request, 'cart' => Cart::getContent(), 'order' => []]));
            } else {
                $this->checkoutAction($invoice);
            }
            return redirect()->back()->with('success', \Lang::get('message.check-your-mail-for-further-datails'));
        } catch (\Exception $ex) {
            dd($ex);
            return redirect()->back()->with('fails', $ex->getMessage());
        }
    }