diff --git a/app/Http/Controllers/Agent/helpdesk/MailController.php b/app/Http/Controllers/Agent/helpdesk/MailController.php
index 4138f351f..d73360c08 100644
--- a/app/Http/Controllers/Agent/helpdesk/MailController.php
+++ b/app/Http/Controllers/Agent/helpdesk/MailController.php
@@ -172,7 +172,7 @@ class MailController extends Controller
             }
             $server->setFlag($cert);
             $server->setAuthentication($username, $password);
-            $date = date("d M Y", strToTime("-1 days"));
+            $date = date('d M Y', strtotime('-1 days'));
             $messages = $server->search("SINCE \"$date\" UNSEEN");
             $this->message($messages, $email);
         }
@@ -338,20 +338,21 @@ class MailController extends Controller
      *
      * @return type file
      */
-    public function get_data($id) {
+    public function get_data($id)
+    {
         $attachment = \App\Model\helpdesk\Ticket\Ticket_attachments::where('id', '=', $id)->first();
         if (mime($attachment->type) == true) {
-            echo "<img src=data:$attachment->type;base64," . $attachment->file . ">";
+            echo "<img src=data:$attachment->type;base64,".$attachment->file.'>';
         } else {
             $file = base64_decode($attachment->file);
+
             return response($file)
                             ->header('Cache-Control', 'no-cache private')
                             ->header('Content-Description', 'File Transfer')
                             ->header('Content-Type', $attachment->type)
                             ->header('Content-length', strlen($file))
-                            ->header('Content-Disposition', 'attachment; filename=' . $attachment->name)
+                            ->header('Content-Disposition', 'attachment; filename='.$attachment->name)
                             ->header('Content-Transfer-Encoding', 'binary');
         }
     }
-
 }
diff --git a/app/Model/helpdesk/Ticket/Ticket_attachments.php b/app/Model/helpdesk/Ticket/Ticket_attachments.php
index e2addb4a8..65ef748ee 100644
--- a/app/Model/helpdesk/Ticket/Ticket_attachments.php
+++ b/app/Model/helpdesk/Ticket/Ticket_attachments.php
@@ -10,20 +10,23 @@ class Ticket_attachments extends Model
     protected $fillable = [
                             'id', 'thread_id', 'name', 'size', 'type', 'file', 'data', 'poster', 'updated_at', 'created_at',
                             ];
-    
-    public function getFile() {
+
+    public function getFile()
+    {
         $size = $this->size;
-        $units = array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB');
+        $units = ['B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB'];
         $power = $size > 0 ? floor(log($size, 1024)) : 0;
-        $value = number_format($size / pow(1024, $power), 2, '.', ',') . ' ' . $units[$power];
+        $value = number_format($size / pow(1024, $power), 2, '.', ',').' '.$units[$power];
         if ($this->poster == 'ATTACHMENT') {
             if (mime($this->type) == true) {
-                $var = '<a href="' . \URL::route('image', array('image_id' => $this->id)) . '" target="_blank"><img style="max-width:200px;height:133px;" src="data:image/jpg;base64,' . $this->file . '"/></a>';
-                return  '<li style="background-color:#f4f4f4;"><span class="mailbox-attachment-icon has-img">' . $var . '</span><div class="mailbox-attachment-info"><b style="word-wrap: break-word;">' . $this->name . '</b><br/><p>' . $value . '</p></div></li>';
+                $var = '<a href="'.\URL::route('image', ['image_id' => $this->id]).'" target="_blank"><img style="max-width:200px;height:133px;" src="data:image/jpg;base64,'.$this->file.'"/></a>';
+
+                return  '<li style="background-color:#f4f4f4;"><span class="mailbox-attachment-icon has-img">'.$var.'</span><div class="mailbox-attachment-info"><b style="word-wrap: break-word;">'.$this->name.'</b><br/><p>'.$value.'</p></div></li>';
             } else {
                 //$var = '<a href="' . URL::route('image', array('image_id' => $attachment->id)) . '" target="_blank"><img style="max-width:200px;height:133px;" src="data:'.$attachment->type.';base64,' . base64_encode($data) . '"/></a>';
-                $var = '<a style="max-width:200px;height:133px;color:#666;" href="' . \URL::route('image', array('image_id' => $this->id)) . '" target="_blank"><span class="mailbox-attachment-icon" style="background-color:#fff; font-size:18px;">' . strtoupper($this->type) . '</span><div class="mailbox-attachment-info"><span ><b style="word-wrap: break-word;">' . $this->name . '</b><br/><p>' . $value . '</p></span></div></a>';
-                return  '<li style="background-color:#f4f4f4;">' . $var . '</li>';
+                $var = '<a style="max-width:200px;height:133px;color:#666;" href="'.\URL::route('image', ['image_id' => $this->id]).'" target="_blank"><span class="mailbox-attachment-icon" style="background-color:#fff; font-size:18px;">'.strtoupper($this->type).'</span><div class="mailbox-attachment-info"><span ><b style="word-wrap: break-word;">'.$this->name.'</b><br/><p>'.$value.'</p></span></div></a>';
+
+                return  '<li style="background-color:#f4f4f4;">'.$var.'</li>';
             }
         }
     }