Skip to content

Commit a67e780

Browse files
uploadFile 메소드에서 Exception 제어하지 않도록 수정, BaseException 추가
1 parent 53ece71 commit a67e780

3 files changed

Lines changed: 30 additions & 12 deletions

File tree

src/Exceptions/BaseException.php

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
<?php
2+
3+
namespace Nurigo\Solapi\Exceptions;
4+
5+
use Exception;
6+
7+
class BaseException extends Exception
8+
{
9+
10+
/**
11+
* @var string
12+
*/
13+
public $errorCode;
14+
15+
public function __construct($message = "", $errorCode = "")
16+
{
17+
parent::__construct($message);
18+
$this->errorCode = $errorCode;
19+
}
20+
}

src/Exceptions/MessageNotReceivedException.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ class MessageNotReceivedException extends Exception
1717
*/
1818
public function __construct($failedMessageList)
1919
{
20-
$this->failedMessageList = $failedMessageList;
2120
parent::__construct("메시지 접수에 실패했습니다.");
21+
$this->failedMessageList = $failedMessageList;
2222
}
2323

2424
/**

src/Services/SolapiMessageService.php

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
use DateTime;
66
use Exception;
7+
use Nurigo\Solapi\Exceptions\BaseException;
78
use Nurigo\Solapi\Exceptions\CurlException;
89
use Nurigo\Solapi\Exceptions\MessageNotReceivedException;
910
use Nurigo\Solapi\Libraries\Fetcher;
@@ -55,9 +56,10 @@ public function send($messages, DateTime $scheduledDateTime = null): SendRespons
5556
/**
5657
* @param string $filePath 파일 경로
5758
* @param string $type 파일 유형(MMS, RCS, DOCUMENT, KAKAO)
58-
* @return string|null
59+
* @throws Exception|CurlException
60+
* @return string
5961
*/
60-
public function uploadFile(string $filePath, string $type = "MMS", $name = null, $link = null)
62+
public function uploadFile(string $filePath, string $type = "MMS", $name = null, $link = null): string
6163
{
6264
$fileContent = file_get_contents($filePath);
6365
$encodedFile = base64_encode($fileContent);
@@ -74,15 +76,11 @@ public function uploadFile(string $filePath, string $type = "MMS", $name = null,
7476
$parameter->setLink($link);
7577
}
7678

77-
try {
78-
$result = $this->fetcherInstance->request("POST", "/storage/v1/files", $parameter);
79-
if (isset($result->errorCode)) {
80-
throw new Exception();
81-
}
82-
$response = new UploadFileResponse($result);
83-
return $response->fileId;
84-
} catch (Exception $exception) {
85-
return null;
79+
$result = $this->fetcherInstance->request("POST", "/storage/v1/files", $parameter);
80+
if (isset($result->errorCode)) {
81+
throw new BaseException($result->errorMessage, $result->errorCode);
8682
}
83+
$response = new UploadFileResponse($result);
84+
return $response->fileId;
8785
}
8886
}

0 commit comments

Comments
 (0)