ContactTable::sendHttpRequest() - Code Metrics - Inspection of "test cases" - emtiazzahid/laravel-wpuser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 0c0b0d...09e0a9 )

by Md. Emtiaz

created 2022-01-29 07:38 UTC

ContactTable::sendHttpRequest() A

↳ Parent: ContactTable

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	7
c	0
b	0
f	0
nc	1
nop	2
dl	0
loc	9
rs	10

<?php

namespace App\Http\Livewire;

use App\Models\Contact;
use App\Models\Setting;
use Illuminate\Support\Facades\Http;
use Jantinnerezo\LivewireAlert\LivewireAlert;
use Livewire\Component;
use Livewire\WithPagination;

class ContactTable extends Component
{
    use WithPagination, LivewireAlert;


    public $perPage = 10;
    public $search = '';
    public $orderBy = 'id';
    public $orderAsc = true;

    public $contact = null;

    public function render()
    {
        return view('components.contact-table', [
            'contacts' => Contact::search($this->search)
                ->orderBy($this->orderBy, $this->orderAsc ? 'asc' : 'desc')
                ->paginate($this->perPage),
        ]);
    }

    public function view($id)
    {
        $this->contact = Contact::findOrFail($id);
        $this->openViewModal();
    }

    /**
     * @param $id
     */
    public function createWPAccount($id)
    {
        $settings = Setting::pluck('value', 'key')->toArray();

        if (!isset($settings['site_url']) || !isset($settings['username']) || !isset($settings['password'])) {
            return $this->alert('warning',  'Missing WP site settings');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        }

        $contact = Contact::findOrFail($id);

        $response = $this->sendHttpRequest($settings, $contact);

        if ($response->successful()) {
            $contact->update(['is_wp_synced' => 1]);
            $this->alert('success', 'Data sync successfull');
        } else {
            $body = json_decode($response->body(), true);
            if(isset($body['message'])) {
                $detail = $body['message'];
            } else {
                $detail = 'Unknown';
            }
            
            $this->alert('warning',  'Something wrong happened. Please try again. DETAIL: ' . $detail);
        }
    }

    public function openViewModal()
    {
        $this->dispatchBrowserEvent('openContactViewModal');
    }

    public function closeViewModal()
    {
        $this->dispatchBrowserEvent('closeContactViewModal');
    }

    private function getBaseUrl($fullUrl)
    {
        $url_info = parse_url($fullUrl);
        return $url_info['scheme'] . '://' . $url_info['host'] . '/wp-json/wlu/v1/';
    }

    public function sendHttpRequest($settings, $contact) 
    {
        return Http::withBasicAuth($settings['username'], $settings['password'])->post($this->getBaseUrl($settings['site_url']) . 'customers', [
            'name' => $contact->name,
            'phone' => $contact->phone_number,
            'email' => $contact->email,
            'budget' => $contact->budget,
            'message' => $contact->message,
            'source' => route('contacts.show', ['id' => $contact->id])
        ]);
    }
}


1			<?php
2
3			namespace App\Http\Livewire;
4
5			use App\Models\Contact;
6			use App\Models\Setting;
7			use Illuminate\Support\Facades\Http;
8			use Jantinnerezo\LivewireAlert\LivewireAlert;
9			use Livewire\Component;
10			use Livewire\WithPagination;
11
12			class ContactTable extends Component
13			{
14			use WithPagination, LivewireAlert;
			0 ignored issues – show Bug introduced 2022-01-29 07:21 UTC by Report Bug Copy Issue Report The trait `Livewire\WithPagination` requires the property `$paginationTheme` which is not provided by `App\Http\Livewire\ContactTable`. Loading history...
15
16			public $perPage = 10;
17			public $search = '';
18			public $orderBy = 'id';
19			public $orderAsc = true;
20
21			public $contact = null;
22
23			public function render()
24			{
25			return view('components.contact-table', [
26			'contacts' => Contact::search($this->search)
27			->orderBy($this->orderBy, $this->orderAsc ? 'asc' : 'desc')
28			->paginate($this->perPage),
29			]);
30			}
31
32			public function view($id)
33			{
34			$this->contact = Contact::findOrFail($id);
35			$this->openViewModal();
36			}
37
38			/**
39			* @param $id
40			*/
41			public function createWPAccount($id)
42			{
43			$settings = Setting::pluck('value', 'key')->toArray();
44
45			if (!isset($settings['site_url']) \|\| !isset($settings['username']) \|\| !isset($settings['password'])) {
46			return $this->alert('warning', 'Missing WP site settings');
			0 ignored issues – show Bug introduced 2022-01-29 07:21 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->alert('warning', ...sing WP site settings')` targeting `App\Http\Livewire\ContactTable::alert()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
47			}
48
49			$contact = Contact::findOrFail($id);
50
51			$response = $this->sendHttpRequest($settings, $contact);
52
53			if ($response->successful()) {
54			$contact->update(['is_wp_synced' => 1]);
55			$this->alert('success', 'Data sync successfull');
56			} else {
57			$body = json_decode($response->body(), true);
58			if(isset($body['message'])) {
59			$detail = $body['message'];
60			} else {
61			$detail = 'Unknown';
62			}
63
64			$this->alert('warning', 'Something wrong happened. Please try again. DETAIL: ' . $detail);
65			}
66			}
67
68			public function openViewModal()
69			{
70			$this->dispatchBrowserEvent('openContactViewModal');
71			}
72
73			public function closeViewModal()
74			{
75			$this->dispatchBrowserEvent('closeContactViewModal');
76			}
77
78			private function getBaseUrl($fullUrl)
79			{
80			$url_info = parse_url($fullUrl);
81			return $url_info['scheme'] . '://' . $url_info['host'] . '/wp-json/wlu/v1/';
82			}
83
84			public function sendHttpRequest($settings, $contact)
85			{
86			return Http::withBasicAuth($settings['username'], $settings['password'])->post($this->getBaseUrl($settings['site_url']) . 'customers', [
87			'name' => $contact->name,
88			'phone' => $contact->phone_number,
89			'email' => $contact->email,
90			'budget' => $contact->budget,
91			'message' => $contact->message,
92			'source' => route('contacts.show', ['id' => $contact->id])
93			]);
94			}
95			}
96

emtiazzahid / laravel-wpuser

Push — main ( 0c0b0d...09e0a9 )

ContactTable::sendHttpRequest() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like