From eba3ecd63d692b245a22a5b8b0a3dbf8ec1609b3 Mon Sep 17 00:00:00 2001 From: Gergely POLONKAI Date: Sun, 4 Nov 2012 15:14:27 +0100 Subject: [PATCH] Moved login parameter fetching to the service itself --- Command/SendCommand.php | 5 +---- Service/Sender.php | 4 ++-- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/Command/SendCommand.php b/Command/SendCommand.php index 31b837a..26793ed 100644 --- a/Command/SendCommand.php +++ b/Command/SendCommand.php @@ -20,10 +20,7 @@ class SendCommand extends ContainerAwareCommand $container = $this->getContainer(); $sender = $container->get('gergelypolonkai_smssender.sender'); - $sender->login( - $container->getParameter('username'), - $container->getParameter('password') - ); + $sender->login(); $sender->sendMessage($input->getArgument('recipient'), $input->getArgument('message'), array()); $sender->logout(); } diff --git a/Service/Sender.php b/Service/Sender.php index 731faea..dcc643b 100644 --- a/Service/Sender.php +++ b/Service/Sender.php @@ -34,9 +34,9 @@ class Sender ); } - public function login($username, $password) + public function login() { - if ($this->sender->login($username, $password) === true) { + if ($this->sender->login($this->container->getParameter('username'), $this->container->getParameter('password')) === true) { $this->loggedIn = true; } }