cronario / messenger

src/Curl/Job.php

Doc Comments +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 
 
     /**
-     * @return null
+     * @return string
      */
     public function getUrl()
     {
@@ -35,7 +35,7 @@ 
     }
 
     /**
-     * @return null
+     * @return string
      */
     public function getMethod()
     {

src/Curl/Worker.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Cronario\AbstractJob;
 use Cronario\AbstractWorker;
-use Cronario\Logger;
 use Messenger\CurlWrapper;
 use Messenger\CurlWrapperCurlException;
 use Messenger\CurlWrapperException;

src/Sms/Job.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     const P_PARAM_TEXT = 'text';
 
     /**
-     * @return int|null|string
+     * @return string
      */
     public function getRecipient()
     {

src/Sms/Worker.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     const P_RESULT_DATA_ERRORS = 'errors';
 
     /**
-     * @param null $vendor_name
+     * @param string $vendor_name
      * @param null $vendor_id
      * @param null $success
      * @param null $errors

src/Mail/Job.php

Doc Comments +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
     const P_PARAM_ATTACHMENT__NAME = 'n';
 
     /**
-     * @return int|null|string
+     * @return string
      */
     public function getFromMail()
     {
@@ -36,7 +36,7 @@ 
     }
 
     /**
-     * @return int|null|string
+     * @return string
      */
     public function getFromName()
     {
@@ -54,7 +54,7 @@ 
     }
 
     /**
-     * @return int|null|string
+     * @return string
      */
     public function getToMail()
     {
@@ -90,7 +90,7 @@ 
     }
 
     /**
-     * @return int|null|string
+     * @return string
      */
     public function getBody()
     {

src/Mail/Worker.php

Spacing +5 added lines, -5 removed lines

@@ -62,8 +62,8 @@ 
         $mail = new \PHPMailer();
 
         $clientConfig = static::getConfig('client');
-        if(is_array($clientConfig) && count($clientConfig) > 0){
-            foreach($clientConfig as $key => $value){
+        if (is_array($clientConfig) && count($clientConfig) > 0) {
+            foreach ($clientConfig as $key => $value) {
                 $mail->{$key} = $value;
             }
         }
@@ -92,8 +92,8 @@ 
         $attachments = $job->getAttachment();
         if (!empty($attachments) && is_array($attachments) && count($attachments) > 0) {
             foreach ($attachments as $key => $attach) {
-                if(isset($attach[Job::P_PARAM_ATTACHMENT__NAME])){
-                    $mail->addAttachment($attach[Job::P_PARAM_ATTACHMENT__PATH] , $attach[Job::P_PARAM_ATTACHMENT__NAME]);
+                if (isset($attach[Job::P_PARAM_ATTACHMENT__NAME])) {
+                    $mail->addAttachment($attach[Job::P_PARAM_ATTACHMENT__PATH], $attach[Job::P_PARAM_ATTACHMENT__NAME]);
                 } else {
                     $mail->addAttachment($attach[Job::P_PARAM_ATTACHMENT__PATH]);
                 }
@@ -125,7 +125,7 @@ 
             throw new ResultException(ResultException::RETRY_TRANSPORT_ERROR);
         }
 
-        if(!$resultData['response']['success']){
+        if (!$resultData['response']['success']) {
             throw new ResultException(ResultException::R_FAILURE, $resultData);
         }