diff --git a/NexmoClient/NexmoClient.php b/NexmoClient/NexmoClient.php index a546cbe..ed96bba 100644 --- a/NexmoClient/NexmoClient.php +++ b/NexmoClient/NexmoClient.php @@ -3,6 +3,7 @@ namespace Jhg\NexmoBundle\NexmoClient; use Jhg\NexmoBundle\NexmoClient\Exceptions\QuotaExcededException; use Jhg\NexmoBundle\NexmoClient\Exceptions\UnroutableSmsMessageException; +use Psr\Log\LoggerInterface; class NexmoClient { @@ -36,20 +37,27 @@ class NexmoClient { */ protected $disable_delivery; + /** + * @var LoggerInterface + */ + protected $logger; + /** * @param $api_key * @param $api_secret * @param string $api_method GET|POST configured in Nexmo API preferences * @param string $delivery_phone * @param boolean $disable_delivery + * @param LoggerInterface $logger */ - public function __construct($api_key,$api_secret,$api_method='GET',$delivery_phone,$disable_delivery=false) { + public function __construct($api_key,$api_secret,$api_method='GET',$delivery_phone,$disable_delivery=false,LoggerInterface $logger) { $this->rest_url = 'https://rest.nexmo.com'; $this->api_key = $api_key; $this->api_secret = $api_secret; $this->api_method = $api_method; $this->delivery_phone = $delivery_phone; $this->disable_delivery = $disable_delivery; + $this->logger = $logger; } /** @@ -107,9 +115,13 @@ class NexmoClient { * @throws \Exception */ public function sendTextMessage($fromName,$toNumber,$text,$status_report_req=0) { + $this->logger->debug("Nexmo sendTextMessage from $fromName to $toNumber with text '$text'"); + // delivery phone for development if($this->delivery_phone) { $toNumber = $this->delivery_phone; + + $this->logger->debug("Nexmo sendTextMessage delivery to $toNumber"); } $params = array( @@ -120,6 +132,7 @@ class NexmoClient { ); if($this->disable_delivery) { + $this->logger->debug("Nexmo sendTextMessage delivery disabled by config"); return null; } diff --git a/Resources/config/services.yml b/Resources/config/services.yml index 9c2013e..3a76213 100644 --- a/Resources/config/services.yml +++ b/Resources/config/services.yml @@ -1,9 +1,7 @@ services: jhg_nexmo_client: class: Jhg\NexmoBundle\NexmoClient\NexmoClient - arguments: ["%jhg_nexmo.api_key%","%jhg_nexmo.api_secret%", - "%jhg_nexmo.api_method%","%jhg_nexmo.delivery_phone%", - "%jhg_nexmo.disable_delivery%"] + arguments: ["%jhg_nexmo.api_key%","%jhg_nexmo.api_secret%","%jhg_nexmo.api_method%","%jhg_nexmo.delivery_phone%","%jhg_nexmo.disable_delivery%",@logger] jhg_nexmo_account: class: Jhg\NexmoBundle\Managers\AccountManager