Inspection of "Fix fatal errors" - lubusIN/laravel-gymie - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 09f0b1...3d0663 )

by Ajit

created 2019-01-18 13:35 UTC

Status

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -43,12 +43,12 @@
 block discarded – undo
     public function registerMediaConversions()
     {
         $this->addMediaConversion('thumb')
-             ->setManipulations(['w' => 50, 'h' => 50, 'q' => 100, 'fit' => 'crop'])
-             ->performOnCollections('staff');
+                ->setManipulations(['w' => 50, 'h' => 50, 'q' => 100, 'fit' => 'crop'])
+                ->performOnCollections('staff');
 
         $this->addMediaConversion('form')
-             ->setManipulations(['w' => 70, 'h' => 70, 'q' => 100, 'fit' => 'crop'])
-             ->performOnCollections('staff');
+                ->setManipulations(['w' => 70, 'h' => 70, 'q' => 100, 'fit' => 'crop'])
+                ->performOnCollections('staff');
     }
 
     public function scopeExcludeArchive($query)

Please login to merge, or discard this patch.

app/Invoice.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@
 block discarded – undo
             'discount_note',
             'created_by',
             'updated_by',
-     ];
+        ];
 
     protected $dates = ['created_at', 'updated_at'];
 

Please login to merge, or discard this patch.

app/Console/Kernel.php 1 patch

Indentation +14 added lines, -14 removed lines patch added patch discarded remove patch

@@ -36,39 +36,39 @@
 block discarded – undo
     protected function schedule(Schedule $schedule)
     {
         $schedule->command('reshoot:offlineSms')
-                 ->hourly();
+                    ->hourly();
 
         $schedule->command('birthday:sms')
-                 ->dailyAt('00:01');
+                    ->dailyAt('00:01');
 
         $schedule->command('set:expired')
-                 ->dailyAt('00:05');
+                    ->dailyAt('00:05');
 
         $schedule->command('sms:event')
-                 ->dailyAt('09:00');
+                    ->dailyAt('09:00');
 
         $schedule->command('sms:expiring')
-                 ->dailyAt('10:00');
+                    ->dailyAt('10:00');
 
         $schedule->command('expense:alert')
-                 ->dailyAt('10:30')
-                 ->when(function () {
-                     return \Utilities::getSetting('primary_contact') != null;
-                 });
+                    ->dailyAt('10:30')
+                    ->when(function () {
+                        return \Utilities::getSetting('primary_contact') != null;
+                    });
 
         $schedule->command('pending:invoice')
-                 ->dailyAt('11:00');
+                    ->dailyAt('11:00');
 
         $schedule->command('followup:sms')
-                 ->dailyAt('11:30');
+                    ->dailyAt('11:30');
 
         $schedule->command('sms:expired')
-                 ->dailyAt('11:45');
+                    ->dailyAt('11:45');
 
         $schedule->command('repeat:expense')
-                 ->dailyAt('23:00');
+                    ->dailyAt('23:00');
 
         $schedule->command('sms:status')
-                 ->dailyAt('23:45');
+                    ->dailyAt('23:45');
     }
 }

Please login to merge, or discard this patch.

app/Console/Commands/RepeatExpense.php 1 patch

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

@@ -44,53 +44,53 @@
 block discarded – undo
         foreach ($expenses as $expense) {
             if ($expense->repeat == 1) {
                 $expenseData = ['name' => $expense->name,
-                                     'category_id' => $expense->category_id,
-                                     'due_date' => $expense->due_date->addDays(1),
-                                     'repeat' => $expense->repeat,
-                                     'note' => $expense->note,
-                                     'amount' => $expense->amount,
-                                     'paid' => 0,
-                                     'created_by' => 1,
-                                     'updated_by' => 1, ];
+                                        'category_id' => $expense->category_id,
+                                        'due_date' => $expense->due_date->addDays(1),
+                                        'repeat' => $expense->repeat,
+                                        'note' => $expense->note,
+                                        'amount' => $expense->amount,
+                                        'paid' => 0,
+                                        'created_by' => 1,
+                                        'updated_by' => 1, ];
 
                 $newExpense = new Expense($expenseData);
                 $newExpense->save();
             } elseif ($expense->repeat == 2) {
                 $expenseData = ['name' => $expense->name,
-                                     'category_id' => $expense->category_id,
-                                     'due_date' => $expense->due_date->addWeek(),
-                                     'repeat' => $expense->repeat,
-                                     'note' => $expense->note,
-                                     'amount' => $expense->amount,
-                                     'paid' => 0,
-                                     'created_by' => 1,
-                                     'updated_by' => 1, ];
+                                        'category_id' => $expense->category_id,
+                                        'due_date' => $expense->due_date->addWeek(),
+                                        'repeat' => $expense->repeat,
+                                        'note' => $expense->note,
+                                        'amount' => $expense->amount,
+                                        'paid' => 0,
+                                        'created_by' => 1,
+                                        'updated_by' => 1, ];
 
                 $newExpense = new Expense($expenseData);
                 $newExpense->save();
             } elseif ($expense->repeat == 3) {
                 $expenseData = ['name' => $expense->name,
-                                     'category_id' => $expense->category_id,
-                                     'due_date' => $expense->due_date->addMonth(),
-                                     'repeat' => $expense->repeat,
-                                     'note' => $expense->note,
-                                     'amount' => $expense->amount,
-                                     'paid' => 0,
-                                     'created_by' => 1,
-                                     'updated_by' => 1, ];
+                                        'category_id' => $expense->category_id,
+                                        'due_date' => $expense->due_date->addMonth(),
+                                        'repeat' => $expense->repeat,
+                                        'note' => $expense->note,
+                                        'amount' => $expense->amount,
+                                        'paid' => 0,
+                                        'created_by' => 1,
+                                        'updated_by' => 1, ];
 
                 $newExpense = new Expense($expenseData);
                 $newExpense->save();
             } elseif ($expense->repeat == 4) {
                 $expenseData = ['name' => $expense->name,
-                                     'category_id' => $expense->category_id,
-                                     'due_date' => $expense->due_date->addYear(),
-                                     'repeat' => $expense->repeat,
-                                     'note' => $expense->note,
-                                     'amount' => $expense->amount,
-                                     'paid' => 0,
-                                     'created_by' => 1,
-                                     'updated_by' => 1, ];
+                                        'category_id' => $expense->category_id,
+                                        'due_date' => $expense->due_date->addYear(),
+                                        'repeat' => $expense->repeat,
+                                        'note' => $expense->note,
+                                        'amount' => $expense->amount,
+                                        'paid' => 0,
+                                        'created_by' => 1,
+                                        'updated_by' => 1, ];
 
                 $newExpense = new Expense($expenseData);
                 $newExpense->save();

Please login to merge, or discard this patch.

app/Http/Controllers/EnquiriesController.php 1 patch

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -80,10 +80,10 @@
 block discarded – undo
 
             //Store the followup details
             $followupData = ['enquiry_id'=>$enquiry->id,
-                                     'followup_by'=>$request->followup_by,
-                                     'due_date'=>$request->due_date,
-                                     'status'=> \constFollowUpStatus::Pending,
-                                     'outcome'=>'', ];
+                                        'followup_by'=>$request->followup_by,
+                                        'due_date'=>$request->due_date,
+                                        'status'=> \constFollowUpStatus::Pending,
+                                        'outcome'=>'', ];
 
             $followup = new Followup($followupData);
             $followup->createdBy()->associate(Auth::user());

Please login to merge, or discard this patch.

app/Http/Controllers/PlansController.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -59,7 +59,7 @@
 block discarded – undo
     {
         //Model Validation
         $this->validate($request, ['plan_code' => 'unique:mst_plans,plan_code',
-                                   'plan_name' => 'unique:mst_plans,plan_name', ]);
+                                    'plan_name' => 'unique:mst_plans,plan_name', ]);
 
         $plan = new Plan($request->all());
 

Please login to merge, or discard this patch.

app/Http/Controllers/ExpensesController.php 1 patch

Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -66,11 +66,11 @@
 block discarded – undo
     public function store(Request $request)
     {
         $expenseData = ['name' => $request->name,
-                             'category_id' => $request->category_id,
-                             'due_date' => $request->due_date,
-                             'repeat' => $request->repeat,
-                             'note' => $request->note,
-                             'amount' => $request->amount, ];
+                                'category_id' => $request->category_id,
+                                'due_date' => $request->due_date,
+                                'repeat' => $request->repeat,
+                                'note' => $request->note,
+                                'amount' => $request->amount, ];
 
         $expense = new Expense($expenseData);
         $expense->createdBy()->associate(Auth::user());

Please login to merge, or discard this patch.

app/Setting.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@
 block discarded – undo
     protected $fillable = [
             'key',
             'value',
-     ];
+        ];
 
     const CREATED_AT = null;
 

Please login to merge, or discard this patch.

app/Lubus/Utilities.php 2 patches

Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -650,11 +650,11 @@
 block discarded – undo
             if (str_contains($response, 'SMS-SHOOT-ID')) {
                 //Log entry for SMS_log table
                 $log->update(['shoot_id' => substr($response, strpos($response, 'SMS-SHOOT-ID/') + 13),
-                              'number' => $member_contact,
-                              'message' => $sms_text,
-                              'sender_id' => $sender_id,
-                              'send_time' => Carbon::now(),
-                              'status' => 'NA', ]);
+                                'number' => $member_contact,
+                                'message' => $sms_text,
+                                'sender_id' => $sender_id,
+                                'send_time' => Carbon::now(),
+                                'status' => 'NA', ]);
                 $log->save();
             }
             //Update SMS balance

Please login to merge, or discard this patch.

Switch Indentation +213 added lines, -213 removed lines patch added patch discarded remove patch

@@ -43,13 +43,13 @@  discard block
 block discarded – undo
     public static function getFollowUpStatus($status)
     {
         switch ($status) {
-        case '1':
-            return 'Done';
-            break;
+            case '1':
+                return 'Done';
+                break;
 
-        default:
-            return 'Pending';
-            break;
+            default:
+                return 'Pending';
+                break;
     }
     }
 
@@ -57,17 +57,17 @@  discard block
 block discarded – undo
     public static function getFollowupBy($followUpBy)
     {
         switch ($followUpBy) {
-        case '1':
-            return 'SMS';
-            break;
+            case '1':
+                return 'SMS';
+                break;
 
-        case '2':
-            return 'Personal';
-            break;
+            case '2':
+                return 'Personal';
+                break;
 
-        default:
-            return 'Call';
-            break;
+            default:
+                return 'Call';
+                break;
     }
     }
 
@@ -75,13 +75,13 @@  discard block
 block discarded – undo
     public static function getIconBg($status)
     {
         switch ($status) {
-        case '1':
-            return 'bg-blue-400 border-blue-700';
-            break;
+            case '1':
+                return 'bg-blue-400 border-blue-700';
+                break;
 
-        default:
-            return 'bg-orange-400 border-orange-700';
-            break;
+            default:
+                return 'bg-orange-400 border-orange-700';
+                break;
     }
     }
 
@@ -89,13 +89,13 @@  discard block
 block discarded – undo
     public static function getStatusIcon($status)
     {
         switch ($status) {
-        case '1':
-            return 'fa fa-thumbs-up';
-            break;
+            case '1':
+                return 'fa fa-thumbs-up';
+                break;
 
-        default:
-            return 'fa fa-refresh';
-            break;
+            default:
+                return 'fa fa-refresh';
+                break;
     }
     }
 
@@ -103,29 +103,29 @@  discard block
 block discarded – undo
     public static function getAim($aim)
     {
         switch ($aim) {
-        case '1':
-            return 'Networking';
-            break;
+            case '1':
+                return 'Networking';
+                break;
 
-        case '2':
-            return 'Body Building';
-            break;
+            case '2':
+                return 'Body Building';
+                break;
 
-        case '3':
-            return 'Fatloss';
-            break;
+            case '3':
+                return 'Fatloss';
+                break;
 
-        case '4':
-            return 'Weightgain';
-            break;
+            case '4':
+                return 'Weightgain';
+                break;
 
-        case '5':
-            return 'Others';
-            break;
+            case '5':
+                return 'Others';
+                break;
 
-        default:
-            return 'Fitness';
-            break;
+            default:
+                return 'Fitness';
+                break;
     }
     }
 
@@ -133,21 +133,21 @@  discard block
 block discarded – undo
     public static function getInvoiceLabel($status)
     {
         switch ($status) {
-        case '0':
-            return 'label label-danger';
-            break;
+            case '0':
+                return 'label label-danger';
+                break;
 
-        case '1':
-            return 'label label-success';
-            break;
+            case '1':
+                return 'label label-success';
+                break;
 
-        case '3':
-            return 'label label-default';
-            break;
+            case '3':
+                return 'label label-default';
+                break;
 
-        default:
-            return 'label label-primary';
-            break;
+            default:
+                return 'label label-primary';
+                break;
     }
     }
 
@@ -155,25 +155,25 @@  discard block
 block discarded – undo
     public static function expenseRepeatIntervel($repeat)
     {
         switch ($repeat) {
-        case '0':
-            return 'Never Repeat';
-            break;
+            case '0':
+                return 'Never Repeat';
+                break;
 
-        case '1':
-            return 'Every Day';
-            break;
+            case '1':
+                return 'Every Day';
+                break;
 
-        case '2':
-            return 'Every Week';
-            break;
+            case '2':
+                return 'Every Week';
+                break;
 
-        case '3':
-            return 'Every Month';
-            break;
+            case '3':
+                return 'Every Month';
+                break;
 
-        default:
-            return 'Every Year';
-            break;
+            default:
+                return 'Every Year';
+                break;
     }
     }
 
@@ -181,13 +181,13 @@  discard block
 block discarded – undo
     public static function getPaidUnpaid($status)
     {
         switch ($status) {
-        case '0':
-            return 'label label-danger';
-            break;
+            case '0':
+                return 'label label-danger';
+                break;
 
-        default:
-            return 'label label-primary';
-            break;
+            default:
+                return 'label label-primary';
+                break;
     }
     }
 
@@ -195,13 +195,13 @@  discard block
 block discarded – undo
     public static function getActiveInactive($status)
     {
         switch ($status) {
-        case '0':
-            return 'label label-danger';
-            break;
+            case '0':
+                return 'label label-danger';
+                break;
 
-        default:
-            return 'label label-primary';
-            break;
+            default:
+                return 'label label-primary';
+                break;
     }
     }
 
@@ -209,29 +209,29 @@  discard block
 block discarded – undo
     public static function getOccupation($occupation)
     {
         switch ($occupation) {
-        case '1':
-            return 'Housewife';
-            break;
+            case '1':
+                return 'Housewife';
+                break;
 
-        case '2':
-            return 'Self Employed';
-            break;
+            case '2':
+                return 'Self Employed';
+                break;
 
-        case '3':
-            return 'Professional';
-            break;
+            case '3':
+                return 'Professional';
+                break;
 
-        case '4':
-            return 'Freelancer';
-            break;
+            case '4':
+                return 'Freelancer';
+                break;
 
-        case '5':
-            return 'Others';
-            break;
+            case '5':
+                return 'Others';
+                break;
 
-        default:
-            return 'Student';
-            break;
+            default:
+                return 'Student';
+                break;
     }
     }
 
@@ -239,17 +239,17 @@  discard block
 block discarded – undo
     public static function getSource($source)
     {
         switch ($source) {
-        case '1':
-            return 'Word of mouth';
-            break;
+            case '1':
+                return 'Word of mouth';
+                break;
 
-        case '2':
-            return 'Others';
-            break;
+            case '2':
+                return 'Others';
+                break;
 
-        default:
-            return 'Promotions';
-            break;
+            default:
+                return 'Promotions';
+                break;
     }
     }
 
@@ -257,17 +257,17 @@  discard block
 block discarded – undo
     public static function getStatusValue($status)
     {
         switch ($status) {
-        case '0':
-            return 'Inactive';
-            break;
+            case '0':
+                return 'Inactive';
+                break;
 
-        case '2':
-            return 'Archived';
-            break;
+            case '2':
+                return 'Archived';
+                break;
 
-        default:
-            return 'Active';
-            break;
+            default:
+                return 'Active';
+                break;
     }
     }
 
@@ -275,17 +275,17 @@  discard block
 block discarded – undo
     public static function getEnquiryStatus($status)
     {
         switch ($status) {
-        case '0':
-            return 'Lost';
-            break;
+            case '0':
+                return 'Lost';
+                break;
 
-        case '2':
-            return 'Member';
-            break;
+            case '2':
+                return 'Member';
+                break;
 
-        default:
-            return 'Lead';
-            break;
+            default:
+                return 'Lead';
+                break;
     }
     }
 
@@ -293,17 +293,17 @@  discard block
 block discarded – undo
     public static function getEnquiryLabel($status)
     {
         switch ($status) {
-        case '0':
-            return 'label label-danger';
-            break;
+            case '0':
+                return 'label label-danger';
+                break;
 
-        case '2':
-            return 'label label-success';
-            break;
+            case '2':
+                return 'label label-success';
+                break;
 
-        default:
-            return 'label label-primary';
-            break;
+            default:
+                return 'label label-primary';
+                break;
     }
     }
 
@@ -327,21 +327,21 @@  discard block
 block discarded – undo
     public static function getInvoiceStatus($status)
     {
         switch ($status) {
-        case '1':
-            return 'Paid';
-            break;
+            case '1':
+                return 'Paid';
+                break;
 
-        case '2':
-            return 'Partial';
-            break;
+            case '2':
+                return 'Partial';
+                break;
 
-        case '3':
-            return 'Overpaid';
-            break;
+            case '3':
+                return 'Overpaid';
+                break;
 
-        default:
-            return 'Unpaid';
-            break;
+            default:
+                return 'Unpaid';
+                break;
     }
     }
 
@@ -349,21 +349,21 @@  discard block
 block discarded – undo
     public static function getSubscriptionStatus($status)
     {
         switch ($status) {
-        case '0':
-            return 'Expired';
-            break;
+            case '0':
+                return 'Expired';
+                break;
 
-        case '2':
-            return 'Renewed';
-            break;
+            case '2':
+                return 'Renewed';
+                break;
 
-        case '3':
-            return 'Cancelled';
-            break;
+            case '3':
+                return 'Cancelled';
+                break;
 
-        default:
-            return 'OnGoing';
-            break;
+            default:
+                return 'OnGoing';
+                break;
     }
     }
 
@@ -371,21 +371,21 @@  discard block
 block discarded – undo
     public static function getSubscriptionLabel($status)
     {
         switch ($status) {
-        case '0':
-            return 'label label-danger';
-            break;
+            case '0':
+                return 'label label-danger';
+                break;
 
-        case '2':
-            return 'label label-success';
-            break;
+            case '2':
+                return 'label label-success';
+                break;
 
-        case '3':
-            return 'label label-default';
-            break;
+            case '3':
+                return 'label label-default';
+                break;
 
-        default:
-            return 'label label-primary';
-            break;
+            default:
+                return 'label label-primary';
+                break;
     }
     }
 
@@ -393,13 +393,13 @@  discard block
 block discarded – undo
     public static function getPaymentMode($status)
     {
         switch ($status) {
-        case '0':
-            return 'Cheque';
-            break;
+            case '0':
+                return 'Cheque';
+                break;
 
-        default:
-            return 'Cash';
-            break;
+            default:
+                return 'Cash';
+                break;
     }
     }
 
@@ -407,25 +407,25 @@  discard block
 block discarded – undo
     public static function getChequeStatus($status)
     {
         switch ($status) {
-        case '1':
-            return 'Deposited';
-            break;
+            case '1':
+                return 'Deposited';
+                break;
 
-        case '2':
-            return 'Cleared';
-            break;
+            case '2':
+                return 'Cleared';
+                break;
 
-        case '3':
-            return 'Bounced';
-            break;
+            case '3':
+                return 'Bounced';
+                break;
 
-        case '4':
-            return 'Reissued';
-            break;
+            case '4':
+                return 'Reissued';
+                break;
 
-        default:
-            return 'Recieved';
-            break;
+            default:
+                return 'Recieved';
+                break;
     }
     }
 
@@ -433,13 +433,13 @@  discard block
 block discarded – undo
     public static function getGender($gender)
     {
         switch ($gender) {
-        case 'm':
-            return 'Male';
-            break;
+            case 'm':
+                return 'Male';
+                break;
 
-        case 'f':
-            return 'Female';
-            break;
+            case 'f':
+                return 'Female';
+                break;
     }
     }
 
@@ -447,13 +447,13 @@  discard block
 block discarded – undo
     public static function getDisplay($display)
     {
         switch ($display) {
-        case 'gym_logo':
-            return 'Gym Logo';
-            break;
+            case 'gym_logo':
+                return 'Gym Logo';
+                break;
 
-        default:
-            return 'Gym Name';
-            break;
+            default:
+                return 'Gym Name';
+                break;
     }
     }
 
@@ -461,13 +461,13 @@  discard block
 block discarded – undo
     public static function getMode($mode)
     {
         switch ($mode) {
-        case '0':
-            return 'Manual';
-            break;
+            case '0':
+                return 'Manual';
+                break;
 
-        default:
-            return 'Automatic';
-            break;
+            default:
+                return 'Automatic';
+                break;
     }
     }
 

Please login to merge, or discard this patch.

		@@ -650,11 +650,11 @@
		block discarded – undo
650	650	if (str_contains($response, 'SMS-SHOOT-ID')) {
651	651	//Log entry for SMS_log table
652	652	$log->update(['shoot_id' => substr($response, strpos($response, 'SMS-SHOOT-ID/') + 13),
653		- 'number' => $member_contact,
654		- 'message' => $sms_text,
655		- 'sender_id' => $sender_id,
656		- 'send_time' => Carbon::now(),
657		- 'status' => 'NA', ]);
	653	+ 'number' => $member_contact,
	654	+ 'message' => $sms_text,
	655	+ 'sender_id' => $sender_id,
	656	+ 'send_time' => Carbon::now(),
	657	+ 'status' => 'NA', ]);
658	658	$log->save();
659	659	}
660	660	//Update SMS balance

		@@ -43,12 +43,12 @@
		block discarded – undo
43	43	public function registerMediaConversions()
44	44	{
45	45	$this->addMediaConversion('thumb')
46		- ->setManipulations(['w' => 50, 'h' => 50, 'q' => 100, 'fit' => 'crop'])
47		- ->performOnCollections('staff');
	46	+ ->setManipulations(['w' => 50, 'h' => 50, 'q' => 100, 'fit' => 'crop'])
	47	+ ->performOnCollections('staff');
48	48
49	49	$this->addMediaConversion('form')
50		- ->setManipulations(['w' => 70, 'h' => 70, 'q' => 100, 'fit' => 'crop'])
51		- ->performOnCollections('staff');
	50	+ ->setManipulations(['w' => 70, 'h' => 70, 'q' => 100, 'fit' => 'crop'])
	51	+ ->performOnCollections('staff');
52	52	}
53	53
54	54	public function scopeExcludeArchive($query)

		@@ -23,7 +23,7 @@
		block discarded – undo
23	23	'discount_note',
24	24	'created_by',
25	25	'updated_by',
26		- ];
	26	+ ];
27	27
28	28	protected $dates = ['created_at', 'updated_at'];
29	29

		@@ -36,39 +36,39 @@
		block discarded – undo
36	36	protected function schedule(Schedule $schedule)
37	37	{
38	38	$schedule->command('reshoot:offlineSms')
39		- ->hourly();
	39	+ ->hourly();
40	40
41	41	$schedule->command('birthday:sms')
42		- ->dailyAt('00:01');
	42	+ ->dailyAt('00:01');
43	43
44	44	$schedule->command('set:expired')
45		- ->dailyAt('00:05');
	45	+ ->dailyAt('00:05');
46	46
47	47	$schedule->command('sms:event')
48		- ->dailyAt('09:00');
	48	+ ->dailyAt('09:00');
49	49
50	50	$schedule->command('sms:expiring')
51		- ->dailyAt('10:00');
	51	+ ->dailyAt('10:00');
52	52
53	53	$schedule->command('expense:alert')
54		- ->dailyAt('10:30')
55		- ->when(function () {
56		- return \Utilities::getSetting('primary_contact') != null;
57		- });
	54	+ ->dailyAt('10:30')
	55	+ ->when(function () {
	56	+ return \Utilities::getSetting('primary_contact') != null;
	57	+ });
58	58
59	59	$schedule->command('pending:invoice')
60		- ->dailyAt('11:00');
	60	+ ->dailyAt('11:00');
61	61
62	62	$schedule->command('followup:sms')
63		- ->dailyAt('11:30');
	63	+ ->dailyAt('11:30');
64	64
65	65	$schedule->command('sms:expired')
66		- ->dailyAt('11:45');
	66	+ ->dailyAt('11:45');
67	67
68	68	$schedule->command('repeat:expense')
69		- ->dailyAt('23:00');
	69	+ ->dailyAt('23:00');
70	70
71	71	$schedule->command('sms:status')
72		- ->dailyAt('23:45');
	72	+ ->dailyAt('23:45');
73	73	}
74	74	}

		@@ -44,53 +44,53 @@
		block discarded – undo
44	44	foreach ($expenses as $expense) {
45	45	if ($expense->repeat == 1) {
46	46	$expenseData = ['name' => $expense->name,
47		- 'category_id' => $expense->category_id,
48		- 'due_date' => $expense->due_date->addDays(1),
49		- 'repeat' => $expense->repeat,
50		- 'note' => $expense->note,
51		- 'amount' => $expense->amount,
52		- 'paid' => 0,
53		- 'created_by' => 1,
54		- 'updated_by' => 1, ];
	47	+ 'category_id' => $expense->category_id,
	48	+ 'due_date' => $expense->due_date->addDays(1),
	49	+ 'repeat' => $expense->repeat,
	50	+ 'note' => $expense->note,
	51	+ 'amount' => $expense->amount,
	52	+ 'paid' => 0,
	53	+ 'created_by' => 1,
	54	+ 'updated_by' => 1, ];
55	55
56	56	$newExpense = new Expense($expenseData);
57	57	$newExpense->save();
58	58	} elseif ($expense->repeat == 2) {
59	59	$expenseData = ['name' => $expense->name,
60		- 'category_id' => $expense->category_id,
61		- 'due_date' => $expense->due_date->addWeek(),
62		- 'repeat' => $expense->repeat,
63		- 'note' => $expense->note,
64		- 'amount' => $expense->amount,
65		- 'paid' => 0,
66		- 'created_by' => 1,
67		- 'updated_by' => 1, ];
	60	+ 'category_id' => $expense->category_id,
	61	+ 'due_date' => $expense->due_date->addWeek(),
	62	+ 'repeat' => $expense->repeat,
	63	+ 'note' => $expense->note,
	64	+ 'amount' => $expense->amount,
	65	+ 'paid' => 0,
	66	+ 'created_by' => 1,
	67	+ 'updated_by' => 1, ];
68	68
69	69	$newExpense = new Expense($expenseData);
70	70	$newExpense->save();
71	71	} elseif ($expense->repeat == 3) {
72	72	$expenseData = ['name' => $expense->name,
73		- 'category_id' => $expense->category_id,
74		- 'due_date' => $expense->due_date->addMonth(),
75		- 'repeat' => $expense->repeat,
76		- 'note' => $expense->note,
77		- 'amount' => $expense->amount,
78		- 'paid' => 0,
79		- 'created_by' => 1,
80		- 'updated_by' => 1, ];
	73	+ 'category_id' => $expense->category_id,
	74	+ 'due_date' => $expense->due_date->addMonth(),
	75	+ 'repeat' => $expense->repeat,
	76	+ 'note' => $expense->note,
	77	+ 'amount' => $expense->amount,
	78	+ 'paid' => 0,
	79	+ 'created_by' => 1,
	80	+ 'updated_by' => 1, ];
81	81
82	82	$newExpense = new Expense($expenseData);
83	83	$newExpense->save();
84	84	} elseif ($expense->repeat == 4) {
85	85	$expenseData = ['name' => $expense->name,
86		- 'category_id' => $expense->category_id,
87		- 'due_date' => $expense->due_date->addYear(),
88		- 'repeat' => $expense->repeat,
89		- 'note' => $expense->note,
90		- 'amount' => $expense->amount,
91		- 'paid' => 0,
92		- 'created_by' => 1,
93		- 'updated_by' => 1, ];
	86	+ 'category_id' => $expense->category_id,
	87	+ 'due_date' => $expense->due_date->addYear(),
	88	+ 'repeat' => $expense->repeat,
	89	+ 'note' => $expense->note,
	90	+ 'amount' => $expense->amount,
	91	+ 'paid' => 0,
	92	+ 'created_by' => 1,
	93	+ 'updated_by' => 1, ];
94	94
95	95	$newExpense = new Expense($expenseData);
96	96	$newExpense->save();

		@@ -80,10 +80,10 @@
		block discarded – undo
80	80
81	81	//Store the followup details
82	82	$followupData = ['enquiry_id'=>$enquiry->id,
83		- 'followup_by'=>$request->followup_by,
84		- 'due_date'=>$request->due_date,
85		- 'status'=> \constFollowUpStatus::Pending,
86		- 'outcome'=>'', ];
	83	+ 'followup_by'=>$request->followup_by,
	84	+ 'due_date'=>$request->due_date,
	85	+ 'status'=> \constFollowUpStatus::Pending,
	86	+ 'outcome'=>'', ];
87	87
88	88	$followup = new Followup($followupData);
89	89	$followup->createdBy()->associate(Auth::user());

		@@ -59,7 +59,7 @@
		block discarded – undo
59	59	{
60	60	//Model Validation
61	61	$this->validate($request, ['plan_code' => 'unique:mst_plans,plan_code',
62		- 'plan_name' => 'unique:mst_plans,plan_name', ]);
	62	+ 'plan_name' => 'unique:mst_plans,plan_name', ]);
63	63
64	64	$plan = new Plan($request->all());
65	65

		@@ -66,11 +66,11 @@
		block discarded – undo
66	66	public function store(Request $request)
67	67	{
68	68	$expenseData = ['name' => $request->name,
69		- 'category_id' => $request->category_id,
70		- 'due_date' => $request->due_date,
71		- 'repeat' => $request->repeat,
72		- 'note' => $request->note,
73		- 'amount' => $request->amount, ];
	69	+ 'category_id' => $request->category_id,
	70	+ 'due_date' => $request->due_date,
	71	+ 'repeat' => $request->repeat,
	72	+ 'note' => $request->note,
	73	+ 'amount' => $request->amount, ];
74	74
75	75	$expense = new Expense($expenseData);
76	76	$expense->createdBy()->associate(Auth::user());

		@@ -11,7 +11,7 @@
		block discarded – undo
11	11	protected $fillable = [
12	12	'key',
13	13	'value',
14		- ];
	14	+ ];
15	15
16	16	const CREATED_AT = null;
17	17

		@@ -43,13 +43,13 @@ discard block
		block discarded – undo
43	43	public static function getFollowUpStatus($status)
44	44	{
45	45	switch ($status) {
46		- case '1':
47		- return 'Done';
48		- break;
	46	+ case '1':
	47	+ return 'Done';
	48	+ break;
49	49
50		- default:
51		- return 'Pending';
52		- break;
	50	+ default:
	51	+ return 'Pending';
	52	+ break;
53	53	}
54	54	}
55	55
		@@ -57,17 +57,17 @@ discard block
		block discarded – undo
57	57	public static function getFollowupBy($followUpBy)
58	58	{
59	59	switch ($followUpBy) {
60		- case '1':
61		- return 'SMS';
62		- break;
	60	+ case '1':
	61	+ return 'SMS';
	62	+ break;
63	63
64		- case '2':
65		- return 'Personal';
66		- break;
	64	+ case '2':
	65	+ return 'Personal';
	66	+ break;
67	67
68		- default:
69		- return 'Call';
70		- break;
	68	+ default:
	69	+ return 'Call';
	70	+ break;
71	71	}
72	72	}
73	73
		@@ -75,13 +75,13 @@ discard block
		block discarded – undo
75	75	public static function getIconBg($status)
76	76	{
77	77	switch ($status) {
78		- case '1':
79		- return 'bg-blue-400 border-blue-700';
80		- break;
	78	+ case '1':
	79	+ return 'bg-blue-400 border-blue-700';
	80	+ break;
81	81
82		- default:
83		- return 'bg-orange-400 border-orange-700';
84		- break;
	82	+ default:
	83	+ return 'bg-orange-400 border-orange-700';
	84	+ break;
85	85	}
86	86	}
87	87
		@@ -89,13 +89,13 @@ discard block
		block discarded – undo
89	89	public static function getStatusIcon($status)
90	90	{
91	91	switch ($status) {
92		- case '1':
93		- return 'fa fa-thumbs-up';
94		- break;
	92	+ case '1':
	93	+ return 'fa fa-thumbs-up';
	94	+ break;
95	95
96		- default:
97		- return 'fa fa-refresh';
98		- break;
	96	+ default:
	97	+ return 'fa fa-refresh';
	98	+ break;
99	99	}
100	100	}
101	101
		@@ -103,29 +103,29 @@ discard block
		block discarded – undo
103	103	public static function getAim($aim)
104	104	{
105	105	switch ($aim) {
106		- case '1':
107		- return 'Networking';
108		- break;
	106	+ case '1':
	107	+ return 'Networking';
	108	+ break;
109	109
110		- case '2':
111		- return 'Body Building';
112		- break;
	110	+ case '2':
	111	+ return 'Body Building';
	112	+ break;
113	113
114		- case '3':
115		- return 'Fatloss';
116		- break;
	114	+ case '3':
	115	+ return 'Fatloss';
	116	+ break;
117	117
118		- case '4':
119		- return 'Weightgain';
120		- break;
	118	+ case '4':
	119	+ return 'Weightgain';
	120	+ break;
121	121
122		- case '5':
123		- return 'Others';
124		- break;
	122	+ case '5':
	123	+ return 'Others';
	124	+ break;
125	125
126		- default:
127		- return 'Fitness';
128		- break;
	126	+ default:
	127	+ return 'Fitness';
	128	+ break;
129	129	}
130	130	}
131	131
		@@ -133,21 +133,21 @@ discard block
		block discarded – undo
133	133	public static function getInvoiceLabel($status)
134	134	{
135	135	switch ($status) {
136		- case '0':
137		- return 'label label-danger';
138		- break;
	136	+ case '0':
	137	+ return 'label label-danger';
	138	+ break;
139	139
140		- case '1':
141		- return 'label label-success';
142		- break;
	140	+ case '1':
	141	+ return 'label label-success';
	142	+ break;
143	143
144		- case '3':
145		- return 'label label-default';
146		- break;
	144	+ case '3':
	145	+ return 'label label-default';
	146	+ break;
147	147
148		- default:
149		- return 'label label-primary';
150		- break;
	148	+ default:
	149	+ return 'label label-primary';
	150	+ break;
151	151	}
152	152	}
153	153
		@@ -155,25 +155,25 @@ discard block
		block discarded – undo
155	155	public static function expenseRepeatIntervel($repeat)
156	156	{
157	157	switch ($repeat) {
158		- case '0':
159		- return 'Never Repeat';
160		- break;
	158	+ case '0':
	159	+ return 'Never Repeat';
	160	+ break;
161	161
162		- case '1':
163		- return 'Every Day';
164		- break;
	162	+ case '1':
	163	+ return 'Every Day';
	164	+ break;
165	165
166		- case '2':
167		- return 'Every Week';
168		- break;
	166	+ case '2':
	167	+ return 'Every Week';
	168	+ break;
169	169
170		- case '3':
171		- return 'Every Month';
172		- break;
	170	+ case '3':
	171	+ return 'Every Month';
	172	+ break;
173	173
174		- default:
175		- return 'Every Year';
176		- break;
	174	+ default:
	175	+ return 'Every Year';
	176	+ break;
177	177	}
178	178	}
179	179
		@@ -181,13 +181,13 @@ discard block
		block discarded – undo
181	181	public static function getPaidUnpaid($status)
182	182	{
183	183	switch ($status) {
184		- case '0':
185		- return 'label label-danger';
186		- break;
	184	+ case '0':
	185	+ return 'label label-danger';
	186	+ break;
187	187
188		- default:
189		- return 'label label-primary';
190		- break;
	188	+ default:
	189	+ return 'label label-primary';
	190	+ break;
191	191	}
192	192	}
193	193
		@@ -195,13 +195,13 @@ discard block
		block discarded – undo
195	195	public static function getActiveInactive($status)
196	196	{
197	197	switch ($status) {
198		- case '0':
199		- return 'label label-danger';
200		- break;
	198	+ case '0':
	199	+ return 'label label-danger';
	200	+ break;
201	201
202		- default:
203		- return 'label label-primary';
204		- break;
	202	+ default:
	203	+ return 'label label-primary';
	204	+ break;
205	205	}
206	206	}
207	207
		@@ -209,29 +209,29 @@ discard block
		block discarded – undo
209	209	public static function getOccupation($occupation)
210	210	{
211	211	switch ($occupation) {
212		- case '1':
213		- return 'Housewife';
214		- break;
	212	+ case '1':
	213	+ return 'Housewife';
	214	+ break;
215	215
216		- case '2':
217		- return 'Self Employed';
218		- break;
	216	+ case '2':
	217	+ return 'Self Employed';
	218	+ break;
219	219
220		- case '3':
221		- return 'Professional';
222		- break;
	220	+ case '3':
	221	+ return 'Professional';
	222	+ break;
223	223
224		- case '4':
225		- return 'Freelancer';
226		- break;
	224	+ case '4':
	225	+ return 'Freelancer';
	226	+ break;
227	227
228		- case '5':
229		- return 'Others';
230		- break;
	228	+ case '5':
	229	+ return 'Others';
	230	+ break;
231	231
232		- default:
233		- return 'Student';
234		- break;
	232	+ default:
	233	+ return 'Student';
	234	+ break;
235	235	}
236	236	}
237	237
		@@ -239,17 +239,17 @@ discard block
		block discarded – undo
239	239	public static function getSource($source)
240	240	{
241	241	switch ($source) {
242		- case '1':
243		- return 'Word of mouth';
244		- break;
	242	+ case '1':
	243	+ return 'Word of mouth';
	244	+ break;
245	245
246		- case '2':
247		- return 'Others';
248		- break;
	246	+ case '2':
	247	+ return 'Others';
	248	+ break;
249	249
250		- default:
251		- return 'Promotions';
252		- break;
	250	+ default:
	251	+ return 'Promotions';
	252	+ break;
253	253	}
254	254	}
255	255
		@@ -257,17 +257,17 @@ discard block
		block discarded – undo
257	257	public static function getStatusValue($status)
258	258	{
259	259	switch ($status) {
260		- case '0':
261		- return 'Inactive';
262		- break;
	260	+ case '0':
	261	+ return 'Inactive';
	262	+ break;
263	263
264		- case '2':
265		- return 'Archived';
266		- break;
	264	+ case '2':
	265	+ return 'Archived';
	266	+ break;
267	267
268		- default:
269		- return 'Active';
270		- break;
	268	+ default:
	269	+ return 'Active';
	270	+ break;
271	271	}
272	272	}
273	273
		@@ -275,17 +275,17 @@ discard block
		block discarded – undo
275	275	public static function getEnquiryStatus($status)
276	276	{
277	277	switch ($status) {
278		- case '0':
279		- return 'Lost';
280		- break;
	278	+ case '0':
	279	+ return 'Lost';
	280	+ break;
281	281
282		- case '2':
283		- return 'Member';
284		- break;
	282	+ case '2':
	283	+ return 'Member';
	284	+ break;
285	285
286		- default:
287		- return 'Lead';
288		- break;
	286	+ default:
	287	+ return 'Lead';
	288	+ break;
289	289	}
290	290	}
291	291
		@@ -293,17 +293,17 @@ discard block
		block discarded – undo
293	293	public static function getEnquiryLabel($status)
294	294	{
295	295	switch ($status) {
296		- case '0':
297		- return 'label label-danger';
298		- break;
	296	+ case '0':
	297	+ return 'label label-danger';
	298	+ break;
299	299
300		- case '2':
301		- return 'label label-success';
302		- break;
	300	+ case '2':
	301	+ return 'label label-success';
	302	+ break;
303	303
304		- default:
305		- return 'label label-primary';
306		- break;
	304	+ default:
	305	+ return 'label label-primary';
	306	+ break;
307	307	}
308	308	}
309	309
		@@ -327,21 +327,21 @@ discard block
		block discarded – undo
327	327	public static function getInvoiceStatus($status)
328	328	{
329	329	switch ($status) {
330		- case '1':
331		- return 'Paid';
332		- break;
	330	+ case '1':
	331	+ return 'Paid';
	332	+ break;
333	333
334		- case '2':
335		- return 'Partial';
336		- break;
	334	+ case '2':
	335	+ return 'Partial';
	336	+ break;
337	337
338		- case '3':
339		- return 'Overpaid';
340		- break;
	338	+ case '3':
	339	+ return 'Overpaid';
	340	+ break;
341	341
342		- default:
343		- return 'Unpaid';
344		- break;
	342	+ default:
	343	+ return 'Unpaid';
	344	+ break;
345	345	}
346	346	}
347	347
		@@ -349,21 +349,21 @@ discard block
		block discarded – undo
349	349	public static function getSubscriptionStatus($status)
350	350	{
351	351	switch ($status) {
352		- case '0':
353		- return 'Expired';
354		- break;
	352	+ case '0':
	353	+ return 'Expired';
	354	+ break;
355	355
356		- case '2':
357		- return 'Renewed';
358		- break;
	356	+ case '2':
	357	+ return 'Renewed';
	358	+ break;
359	359
360		- case '3':
361		- return 'Cancelled';
362		- break;
	360	+ case '3':
	361	+ return 'Cancelled';
	362	+ break;
363	363
364		- default:
365		- return 'OnGoing';
366		- break;
	364	+ default:
	365	+ return 'OnGoing';
	366	+ break;
367	367	}
368	368	}
369	369
		@@ -371,21 +371,21 @@ discard block
		block discarded – undo
371	371	public static function getSubscriptionLabel($status)
372	372	{
373	373	switch ($status) {
374		- case '0':
375		- return 'label label-danger';
376		- break;
	374	+ case '0':
	375	+ return 'label label-danger';
	376	+ break;
377	377
378		- case '2':
379		- return 'label label-success';
380		- break;
	378	+ case '2':
	379	+ return 'label label-success';
	380	+ break;
381	381
382		- case '3':
383		- return 'label label-default';
384		- break;
	382	+ case '3':
	383	+ return 'label label-default';
	384	+ break;
385	385
386		- default:
387		- return 'label label-primary';
388		- break;
	386	+ default:
	387	+ return 'label label-primary';
	388	+ break;
389	389	}
390	390	}
391	391
		@@ -393,13 +393,13 @@ discard block
		block discarded – undo
393	393	public static function getPaymentMode($status)
394	394	{
395	395	switch ($status) {
396		- case '0':
397		- return 'Cheque';
398		- break;
	396	+ case '0':
	397	+ return 'Cheque';
	398	+ break;
399	399
400		- default:
401		- return 'Cash';
402		- break;
	400	+ default:
	401	+ return 'Cash';
	402	+ break;
403	403	}
404	404	}
405	405
		@@ -407,25 +407,25 @@ discard block
		block discarded – undo
407	407	public static function getChequeStatus($status)
408	408	{
409	409	switch ($status) {
410		- case '1':
411		- return 'Deposited';
412		- break;
	410	+ case '1':
	411	+ return 'Deposited';
	412	+ break;
413	413
414		- case '2':
415		- return 'Cleared';
416		- break;
	414	+ case '2':
	415	+ return 'Cleared';
	416	+ break;
417	417
418		- case '3':
419		- return 'Bounced';
420		- break;
	418	+ case '3':
	419	+ return 'Bounced';
	420	+ break;
421	421
422		- case '4':
423		- return 'Reissued';
424		- break;
	422	+ case '4':
	423	+ return 'Reissued';
	424	+ break;
425	425
426		- default:
427		- return 'Recieved';
428		- break;
	426	+ default:
	427	+ return 'Recieved';
	428	+ break;
429	429	}
430	430	}
431	431
		@@ -433,13 +433,13 @@ discard block
		block discarded – undo
433	433	public static function getGender($gender)
434	434	{
435	435	switch ($gender) {
436		- case 'm':
437		- return 'Male';
438		- break;
	436	+ case 'm':
	437	+ return 'Male';
	438	+ break;
439	439
440		- case 'f':
441		- return 'Female';
442		- break;
	440	+ case 'f':
	441	+ return 'Female';
	442	+ break;
443	443	}
444	444	}
445	445
		@@ -447,13 +447,13 @@ discard block
		block discarded – undo
447	447	public static function getDisplay($display)
448	448	{
449	449	switch ($display) {
450		- case 'gym_logo':
451		- return 'Gym Logo';
452		- break;
	450	+ case 'gym_logo':
	451	+ return 'Gym Logo';
	452	+ break;
453	453
454		- default:
455		- return 'Gym Name';
456		- break;
	454	+ default:
	455	+ return 'Gym Name';
	456	+ break;
457	457	}
458	458	}
459	459
		@@ -461,13 +461,13 @@ discard block
		block discarded – undo
461	461	public static function getMode($mode)
462	462	{
463	463	switch ($mode) {
464		- case '0':
465		- return 'Manual';
466		- break;
	464	+ case '0':
	465	+ return 'Manual';
	466	+ break;
467	467
468		- default:
469		- return 'Automatic';
470		- break;
	468	+ default:
	469	+ return 'Automatic';
	470	+ break;
471	471	}
472	472	}
473	473

lubusIN / laravel-gymie

Push — master ( 09f0b1...3d0663 )

Status

Category

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +14 added lines, -14 removed lines patch added patch discarded remove patch

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Switch Indentation +213 added lines, -213 removed lines patch added patch discarded remove patch