HoomanMirghasemi\Sms\Listeners\DbLogListener

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	saveLogInDb()	0	11	1
A	handle()	0	3	1

<?php

namespace HoomanMirghasemi\Sms\Listeners;

use HoomanMirghasemi\Sms\Abstracts\Driver;
use HoomanMirghasemi\Sms\Contracts\SmsEvent;
use HoomanMirghasemi\Sms\Models\SmsReport;

class DbLogListener
{
    /**
     * Handle the event.
     *
     * @param object $event
     *
     * @return void
     */
    public function handle(SmsEvent $event): void
    {
        $this->saveLogInDb($event->smsDriver);

Accessing smsDriver on the interface HoomanMirghasemi\Sms\Contracts\SmsEvent suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    /**
     * This method should call in all driver send method in last line.
     *
     * @return void
     */
    protected function saveLogInDb(Driver $smsDriver): void
    {
        $sendSmsReport = new SmsReport();
        $sendSmsReport->mobile = $smsDriver->getRecipient();
        $sendSmsReport->message = $smsDriver->getMessage();
        $reflect = new \ReflectionClass($smsDriver);
        $sendSmsReport->from = strtolower($reflect->getShortName());
        $sendSmsReport->number = $smsDriver->getSenderNumber();
        $sendSmsReport->web_service_response = $smsDriver->getWebServiceResponce();
        $sendSmsReport->success = $smsDriver->getResult();
        $sendSmsReport->save();
    }
}