fanbinit/rhymix
1
0
Fork 0
mirror of https://github.com/Lastorder-DC/rhymix.git synced 2026-01-07 18:51:41 +09:00
Code Issues Projects Releases Packages Wiki Activity

커스텀 알림을 위한 각 메서드들의 기본값을 제거

Browse source 
커스텀알림을 위한 메서드들에서 대부분 notify_srl 과같은 parameter에 기본값을 사용하고 있습니다.
이 기본값들이 대부분 알림센터의 내부에서 사용되고, 이 커밋에서 지웠던 부분 모두 디비쿼리에 필요한 notify_srl 값을 null으로 전송하여 입력이나 가져오는 것에 대부분 문제가 있었을 것으로 추정하였습니다.
문제가 있을 경우 백지가 나타나게 되어서 문제점을 알 수 없을 수 있는 위험은 있으나 대부분의 동작 코드에서 notify_srl 값이 강제적으로 필요한 것으로 작동되고 있기 때문에 일딴은 parameter의 기본값은 삭제합니다.
This commit is contained in:
BJRambo 2018-09-19 17:50:30 +09:00
parent 58932d22c9
commit a94420908d
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
modules/ncenterlite/ncenterlite.model.php
View file

@ -76,7 +76,7 @@ class ncenterliteModel extends ncenterlite
return self::$config;
}
function getNotifyTypebySrl($notify_srl='')
function getNotifyTypebySrl($notify_srl)
{
$args = new stdClass();
$args->notify_type_srl = $notify_srl;
@ -86,7 +86,7 @@ class ncenterliteModel extends ncenterlite
return $output;
}
function getNotifyTypeString($notify_srl='',$notify_args)
function getNotifyTypeString($notify_srl, $notify_args)
{
$this->notify_args = $notify_args;
@ -113,7 +113,7 @@ class ncenterliteModel extends ncenterlite
return $this->notify_args->{$match[1]};
}
function isNotifyTypeExistsbySrl($notify_srl='')
function isNotifyTypeExistsbySrl($notify_srl)
{
$args = new stdClass();
$args->notify_type_srl = $notify_srl;