Apply fixes from StyleCI

app/Console/Kernel.php

@@ -30,7 +30,6 @@ class Kernel extends ConsoleKernel
     protected function schedule(Schedule $schedule)
     {
         if (env('DB_INSTALL') == 1) {
-
             if ($this->getCurrentQueue() != 'sync') {
                 $schedule->command('queue:listen '.$this->getCurrentQueue().' --sleep 60')->everyMinute();
             }

app/FaveoStorage/Controllers/SettingsController.php

@@ -11,24 +11,26 @@
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;
 
-class SettingsController extends Controller {
-
-    public function settingsIcon() {
+class SettingsController extends Controller
+{
+    public function settingsIcon()
+    {
         return ' <div class="col-md-2 col-sm-6">
                     <div class="settingiconblue">
                         <div class="settingdivblue">
-                            <a href="' . url('storage') . '">
+                            <a href="'.url('storage').'">
                                 <span class="fa-stack fa-2x">
                                     <i class="fa fa-save fa-stack-1x"></i>
                                 </span>
                             </a>
                         </div>
-                        <p class="box-title" >' . Lang::get('storage::lang.storage') . '</p>
+                        <p class="box-title" >'.Lang::get('storage::lang.storage').'</p>
                     </div>
                 </div>';
     }
 
-    public function settings() {
+    public function settings()
+    {
         try {
             $settings = new CommonSettings();
             $directories = $this->directories();
@@ -49,7 +51,8 @@ public function settings() {
         }
     }
 
-    public function postSettings(Request $request) {
+    public function postSettings(Request $request)
+    {
         try {
             $requests = $request->except('_token');
             $this->delete();
@@ -67,7 +70,8 @@ public function postSettings(Request $request) {
         }
     }
 
-    public function delete() {
+    public function delete()
+    {
         $settings = CommonSettings::where('option_name', 'storage')->get();
         if ($settings->count() > 0) {
             foreach ($settings as $setting) {
@@ -76,15 +80,17 @@ public function delete() {
         }
     }
 
-    public function save($key, $value) {
+    public function save($key, $value)
+    {
         CommonSettings::create([
-            'option_name' => 'storage',
+            'option_name'    => 'storage',
             'optional_field' => $key,
-            'option_value' => $value,
+            'option_value'   => $value,
         ]);
     }
 
-    public function directories($root = '') {
+    public function directories($root = '')
+    {
         if ($root == '') {
             $root = base_path();
         }
@@ -103,12 +109,12 @@ public function directories($root = '') {
         return $paths;
     }
 
-    public function attachment() {
+    public function attachment()
+    {
         $storage = new StorageController();
         $storage->upload();
     }
 
-
     public function activate()
     {
         if (!\Schema::hasColumn('ticket_attachment', 'driver')) {
@@ -119,5 +125,4 @@ public function activate()
             ]);
         }
     }
-
 }

app/Helper/Finder.php

@@ -58,26 +58,26 @@ public static function group($id, $custom = null)
         return $group->first()->name;
     }
 
-        /**
-         * STATUS TYPE
-         * This function is used for returning status type name with respect to id.
-         *
-         * @param $id type int
-         * @param $custom type array/null
-         *
-         * @return type string
-         */
-        public static function statusType($id, $custom = null)
-        {
-            if ($custom == null) {
-                $status_type = TicketStatusType::whereId($id)->select(['name']);
-            } elseif (isset($custom)) {
-                $status_type = TicketStatusType::whereId($id)->select($custom);
-            }
-
-            return $status_type->first()->name;
+    /**
+     * STATUS TYPE
+     * This function is used for returning status type name with respect to id.
+     *
+     * @param $id type int
+     * @param $custom type array/null
+     *
+     * @return type string
+     */
+    public static function statusType($id, $custom = null)
+    {
+        if ($custom == null) {
+            $status_type = TicketStatusType::whereId($id)->select(['name']);
+        } elseif (isset($custom)) {
+            $status_type = TicketStatusType::whereId($id)->select($custom);
         }
 
+        return $status_type->first()->name;
+    }
+
     /**
      * STATUS
      * This function is used for returning status name with respect to id.
@@ -137,48 +137,48 @@ public static function rolesGroup($id)
                 }
     }
 
-        /**
-         * ANY TYPE STATUS
-         * This function is used to return the set of status which are of any type passed in the param.
-         *
-         * @param type $id
-         *
-         * @return type array
-         */
-        public static function anyTypeStatus($id)
-        {
-            $status_group = Ticket_Status::where('purpose_of_status', '=', $id)->select(['id'])->get();
-            foreach ($status_group as $status) {
-                $status_group2[] = $status->id;
-            }
-
-            return $status_group2;
+    /**
+     * ANY TYPE STATUS
+     * This function is used to return the set of status which are of any type passed in the param.
+     *
+     * @param type $id
+     *
+     * @return type array
+     */
+    public static function anyTypeStatus($id)
+    {
+        $status_group = Ticket_Status::where('purpose_of_status', '=', $id)->select(['id'])->get();
+        foreach ($status_group as $status) {
+            $status_group2[] = $status->id;
         }
 
-        /**
-         * RETURNS ALL STATUS
-         * This function is used to return all the status given in the system.
-         *
-         * @return type array
-         */
-        public static function getAllStatus()
-        {
-            $status = Ticket_Status::where('purpose_of_status', '!=', 3)->orwhere('purpose_of_status', '!=', 4)->get();
-
-            return $status;
-        }
+        return $status_group2;
+    }
+
+    /**
+     * RETURNS ALL STATUS
+     * This function is used to return all the status given in the system.
+     *
+     * @return type array
+     */
+    public static function getAllStatus()
+    {
+        $status = Ticket_Status::where('purpose_of_status', '!=', 3)->orwhere('purpose_of_status', '!=', 4)->get();
 
-        /**
-         * VARIABLE REPLACEMENT
-         * This function is used to replace the replaceable variables form a given content for templates.
-         */
-        public static function replaceTemplateVariables($variables, $data, $contents)
-        {
-            foreach ($variables as $key => $variable) {
-                $messagebody = str_replace($variables[$key], $data[$key], $contents);
-                $contents = $messagebody;
-            }
-
-            return $contents;
+        return $status;
+    }
+
+    /**
+     * VARIABLE REPLACEMENT
+     * This function is used to replace the replaceable variables form a given content for templates.
+     */
+    public static function replaceTemplateVariables($variables, $data, $contents)
+    {
+        foreach ($variables as $key => $variable) {
+            $messagebody = str_replace($variables[$key], $data[$key], $contents);
+            $contents = $messagebody;
         }
+
+        return $contents;
+    }
 }

app/Http/Controllers/Admin/helpdesk/AgentController.php

@@ -271,10 +271,12 @@ public function destroy($id, User $user, Assign_team_agent $team_assign_agent)
         $team_assign_agent = $team_assign_agent->where('agent_id', $id);
         $team_assign_agent->delete();
         $user = $user->whereId($id)->first();
+
         try {
             $error = Lang::get('lang.this_staff_is_related_to_some_tickets');
             $user->id;
             $user->delete();
+
             throw new \Exception($error);
             return redirect('agents')->with('success', Lang::get('lang.agent_deleted_sucessfully'));
         } catch (\Exception $e) {

app/Http/Controllers/Admin/helpdesk/CloseWrokflowController.php

@@ -54,7 +54,7 @@ public function update($id, WorkflowCloseRequest $request)
             $security = new WorkflowClose();
             $securitys = $security->whereId($id)->first();
             $securitys->days = $request->input('days');
-//            $securitys->condition = $request->input('condition');
+            //            $securitys->condition = $request->input('condition');
             $securitys->send_email = $request->input('send_email');
             $securitys->status = $request->input('status');
             $securitys->save();

app/Http/Controllers/Admin/helpdesk/EmailsController.php

@@ -428,6 +428,7 @@ public function destroy($id, Emails $email)
                 return redirect('emails')->with('fails', Lang::get('lang.you_cannot_delete_system_default_email'));
             }
         }
+
         try {
             // fetching the database instance of the current email
             $emails = $email->whereId($id)->first();

app/Http/Controllers/Admin/helpdesk/FormController.php

@@ -106,6 +106,7 @@ public function show($id)
 
                 return view('themes.default1.admin.helpdesk.manage.form.preview', compact('form', 'fields'));
             }
+
             throw new Exception("Sorry we can't find your request");
         } catch (Exception $ex) {
             return redirect()->back()->with('fails', $ex->getMessage());
@@ -125,6 +126,7 @@ public function store(Request $request)
             'name.*'   => 'required',
             'type.*'   => 'required',
         ]);
+
         try {
             $forms = new Forms();
             $require = Input::get('required');
@@ -195,6 +197,7 @@ public function edit($id)
                 //dd($fields);
                 return view('themes.default1.admin.helpdesk.manage.form.edit', compact('form', 'fields', 'select_forms'));
             }
+
             throw new Exception("Sorry we can't find your request");
         } catch (Exception $ex) {
             return redirect()->back()->with('fails', $ex->getMessage());
@@ -213,6 +216,7 @@ public function addChildForm($id)
                 //dd($fields);
                 return view('themes.default1.admin.helpdesk.manage.form.add-child', compact('form', 'fields', 'select_forms'));
             }
+
             throw new Exception("Sorry we can't find your request");
         } catch (Exception $ex) {
             return redirect()->back()->with('fails', $ex->getMessage());
@@ -227,6 +231,7 @@ public function update($id, Request $request)
             'name.*'   => 'required',
             'type.*'   => 'required',
         ]);
+
         try {
             if (!$request->input('formname')) {
                 throw new Exception(Lang::get('lang.please_fill_form_name'));
@@ -391,6 +396,7 @@ public function createValues($fieldid, $values, $childid = null, $key = '')
     public function addChild($fieldid, Request $request)
     {
         $ids = $request->except('_token');
+
         try {
             foreach ($ids as $valueid => $formid) {
                 $field_value = new \App\Model\helpdesk\Form\FieldValue();

app/Http/Controllers/Admin/helpdesk/GroupController.php

@@ -15,7 +15,6 @@
 use Exception;
 // classes
 use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Input;
 use Lang;
 
 /**

app/Http/Controllers/Admin/helpdesk/PriorityController.php

@@ -20,7 +20,6 @@
 // classes
 use Illuminate\Http\Request;
 use Illuminate\support\Collection;
-use Input;
 use Lang;
 
 /**
@@ -47,7 +46,7 @@ public function priorityIndex()
         $user_status = CommonSettings::where('option_name', '=', 'user_priority')->first();
         // dd( $user_status);
 
-       return view('themes.default1.admin.helpdesk.manage.ticket_priority.index', compact('user_status'));
+        return view('themes.default1.admin.helpdesk.manage.ticket_priority.index', compact('user_status'));
     }
 
     /**
@@ -173,7 +172,7 @@ public function priorityEdit1(PriorityRequest $request)
     public function destroy($priority_id)
     {
         $default_priority = Ticket_Priority::where('is_default', '>', '0')->first();
-// dd($default_priority->is_default);
+        // dd($default_priority->is_default);
         $topic = DB::table('help_topic')->where('priority', '=', $priority_id)->update(['priority' => $default_priority->is_default]);
         // if ($topic > 0) {
         //     if ($topic > 1) {

app/Http/Controllers/Admin/helpdesk/SecurityController.php

@@ -10,7 +10,6 @@
 // Request
 use Illuminate\Http\Request;
 // Class
-use Input;
 use Lang;
 use Redirect;
 
@@ -64,10 +63,10 @@ public function create()
      *
      * @return Response
      */
-//    public function show($id)
-//    {
-//        return view('themes.default1.admin.helpdesk.setting.security.preview', compact('id'));
-//    }
+    //    public function show($id)
+    //    {
+    //        return view('themes.default1.admin.helpdesk.setting.security.preview', compact('id'));
+    //    }
 
     /**
      * Update security details.

app/Http/Controllers/Admin/helpdesk/SettingsController.php

@@ -823,7 +823,7 @@ public function PostRatingSettings($id, Rating $ratings, RatingUpdateRequest $re
             $rating->display_order = $request->input('display_order');
             $rating->allow_modification = $request->input('allow_modification');
             $rating->rating_scale = $request->input('rating_scale');
-//            $rating->rating_area = $request->input('rating_area');
+            //            $rating->rating_area = $request->input('rating_area');
             $rating->restrict = $request->input('restrict');
             $rating->save();