Skip to content

Commit 60768ec

Browse files
committed
Merge branch 'master' of github.com:arif98741/laravelbdsms
2 parents 5c7e0ec + 3ab1076 commit 60768ec

5 files changed

Lines changed: 36 additions & 3 deletions

File tree

composer.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,10 @@
1515
"autoload": {
1616
"psr-4": {
1717
"Xenon\\LaravelBDSms\\": "src/"
18-
}
18+
},
19+
"files": [
20+
"Helper/Helper.php"
21+
]
1922
},
2023
"extra": {
2124
"laravel": {

src/Helper/Helper.php

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
<?php
2+
3+
namespace Xenon\LaravelBDSms\Helper;
4+
5+
class Helper
6+
{
7+
public static function numberValidation($number)
8+
{
9+
$validCheckPattern = "/^(?:\+88|01)?(?:\d{11}|\d{13})$/";
10+
if (preg_match($validCheckPattern, $number)) {
11+
if (preg_match('/^(?:01)\d+$/', $number)) {
12+
$number = '+88' . $number;
13+
}
14+
15+
return true;
16+
}
17+
18+
return false;
19+
}
20+
}

src/Provider/DianaHost.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
use GuzzleHttp\Exception\GuzzleException;
1616
use Illuminate\Http\JsonResponse;
1717
use Xenon\LaravelBDSms\Handler\RenderException;
18+
use Xenon\LaravelBDSms\Helper\Helper;
1819
use Xenon\LaravelBDSms\Sender;
1920

2021
class DianaHost extends AbstractProvider
@@ -80,6 +81,11 @@ public function errorException()
8081
if (strlen($this->senderObject->getMobile()) > 11 || strlen($this->senderObject->getMobile()) < 11) {
8182
throw new RenderException('Invalid mobile number. It should be 11 digit');
8283
}
84+
85+
if (Helper::numberValidation($this->senderObject->getMobile()) == false) {
86+
throw new RenderException('Invalid Mobile Number');
87+
}
88+
8389
if (empty($this->senderObject->getMessage())) {
8490
throw new RenderException('Message should not be empty');
8591
}

src/Provider/Ssl.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,10 @@
1111

1212
namespace Xenon\LaravelBDSms\Provider;
1313

14-
14+
use Xenon\LaravelBDSms\Helper\Helper;
1515
use GuzzleHttp\Client;
1616
use GuzzleHttp\Exception\GuzzleException;
1717
use Xenon\LaravelBDSms\Handler\RenderException;
18-
use Xenon\LaravelBDSms\Handler\XenonException;
1918
use Xenon\LaravelBDSms\Sender;
2019

2120

@@ -86,6 +85,9 @@ public function errorException()
8685
if (strlen($this->senderObject->getMobile()) > 11 || strlen($this->senderObject->getMobile()) < 11)
8786
throw new RenderException('Invalid mobile number. It should be 11 digit');
8887

88+
if (Helper::numberValidation($this->senderObject->getMobile()) == false) {
89+
throw new RenderException('Invalid Mobile Number');
90+
}
8991
if (empty($this->senderObject->getMessage()))
9092
throw new RenderException('Message should not be empty');
9193

src/Sender.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
use Exception;
1616
use Xenon\LaravelBDSms\Handler\RenderException;
1717
use Xenon\LaravelBDSms\Handler\XenonException;
18+
use Xenon\LaravelBDSms\Handler\RenderException;
1819

1920
class Sender
2021
{
@@ -84,6 +85,7 @@ public function getConfig()
8485
*/
8586
public function setConfig($config): Sender
8687
{
88+
8789
try {
8890
if (!is_array($config)) {
8991
throw new XenonException('config must be an array');

0 commit comments

Comments
 (0)