mirror of
https://github.com/Lastorder-DC/rhymix.git
synced 2026-01-04 17:21:39 +09:00
Fix #993 읽지 않은 보관함에 있는 쪽지를 삭제할 수 없는 문제 고침
읽지않은 쪽지함으로 사용할때 부득이하게 코드의 절약과 구동 개선을 위해서 message_type 변수를 사용하였는데, 이 변수가 메세지의 보관함에 보관하는 것과 받은 사람, 보낸 사람 이렇게 나뉘는 부분을 확인하였습니다. 그래서 아래와 같은 동작으로 작동하도록 하엿습니다. 1. 메세지 타입이 N(읽지않은 경우) 의 값으로 호출이 된다면 무조건 메세지는 R(받은사람)의 쪽지를 기준으로 선택하도록 합니다. 2. 보관함의 경우 기본 동작은 메세지 읽기-> 보관함이동을 통해서 보관함으로 이동하기 때문에 보관함의 기준으로 삭제하는 기능의 대해서는 별 다른 조치를 취하지 않습니다. 위와 같이 동작을 확인하였으며 일단 전체적인 액션을 테스트 해본 결과 이상이 없어 보입니다. 이 커밋의 내용은 단순히 증상만 정상적으로 보이도록 하였을 수 있으며, 코드 리뷰를 통해서 좀 더 세밀하게 두고두고 볼 필요가 있을 것 입니다. by @BJRambo
This commit is contained in:
parent
f8e00fac76
commit
271fd78c08
1 changed files with 11 additions and 3 deletions
|
|
@ -393,7 +393,7 @@ class communicationController extends communication
|
|||
}
|
||||
|
||||
$message_type = Context::get('message_type');
|
||||
if(!$message_type || !in_array($message_type, array('R', 'S', 'T')))
|
||||
if(!$message_type || !in_array($message_type, array('R', 'S', 'T', 'N')))
|
||||
{
|
||||
return $this->setError('msg_invalid_request');
|
||||
}
|
||||
|
|
@ -418,8 +418,16 @@ class communicationController extends communication
|
|||
// Delete
|
||||
$args = new stdClass();
|
||||
$args->message_srls = implode(',', $target);
|
||||
$args->message_type = $message_type;
|
||||
|
||||
|
||||
if ($message_type === 'N')
|
||||
{
|
||||
$args->message_type = 'R';
|
||||
}
|
||||
else
|
||||
{
|
||||
$args->message_type = $message_type;
|
||||
}
|
||||
|
||||
if($message_type == 'S')
|
||||
{
|
||||
$args->sender_srl = $member_srl;
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue