General cleanup of nonstandard syntax in board module classes

This commit is contained in:
Kijin Sung 2024-12-10 23:02:42 +09:00
parent 3f08dd1de2
commit 2f0ec84cc2
3 changed files with 26 additions and 38 deletions

View file

@ -5,21 +5,14 @@
* @class boardController * @class boardController
* @author NAVER (developers@xpressengine.com) * @author NAVER (developers@xpressengine.com)
* @brief board module Controller class * @brief board module Controller class
**/ */
class BoardController extends Board class BoardController extends Board
{ {
/**
* @brief initialization
**/
function init()
{
}
/** /**
* @brief insert document * @brief insert document
**/ */
function procBoardInsertDocument() public function procBoardInsertDocument()
{ {
// check grant // check grant
if(!$this->grant->write_document) if(!$this->grant->write_document)
@ -295,7 +288,7 @@ class BoardController extends Board
$this->setMessage($msg_code); $this->setMessage($msg_code);
} }
function procBoardRevertDocument() public function procBoardRevertDocument()
{ {
$update_id = Context::get('update_id'); $update_id = Context::get('update_id');
$logged_info = Context::get('logged_info'); $logged_info = Context::get('logged_info');
@ -338,8 +331,8 @@ class BoardController extends Board
/** /**
* @brief delete the document * @brief delete the document
**/ */
function procBoardDeleteDocument() public function procBoardDeleteDocument()
{ {
// get the document_srl // get the document_srl
$document_srl = Context::get('document_srl'); $document_srl = Context::get('document_srl');
@ -416,8 +409,8 @@ class BoardController extends Board
/** /**
* @brief vote * @brief vote
**/ */
function procBoardVoteDocument() public function procBoardVoteDocument()
{ {
// Check document_srl // Check document_srl
$document_srl = intval(Context::get('document_srl')); $document_srl = intval(Context::get('document_srl'));
@ -434,8 +427,8 @@ class BoardController extends Board
/** /**
* @brief insert comments * @brief insert comments
**/ */
function procBoardInsertComment() public function procBoardInsertComment()
{ {
// check grant // check grant
if(!$this->grant->write_comment) if(!$this->grant->write_comment)
@ -625,8 +618,8 @@ class BoardController extends Board
/** /**
* @brief delete the comment * @brief delete the comment
**/ */
function procBoardDeleteComment() public function procBoardDeleteComment()
{ {
// get the comment_srl // get the comment_srl
$comment_srl = Context::get('comment_srl'); $comment_srl = Context::get('comment_srl');
@ -741,8 +734,8 @@ class BoardController extends Board
/** /**
* @brief delete the tracjback * @brief delete the tracjback
**/ */
function procBoardDeleteTrackback() public function procBoardDeleteTrackback()
{ {
$trackback_srl = Context::get('trackback_srl'); $trackback_srl = Context::get('trackback_srl');
@ -765,8 +758,8 @@ class BoardController extends Board
/** /**
* @brief check the password for document and comment * @brief check the password for document and comment
**/ */
function procBoardVerificationPassword() public function procBoardVerificationPassword()
{ {
// get the id number of the document and the comment // get the id number of the document and the comment
$password = Context::get('password'); $password = Context::get('password');
@ -790,7 +783,9 @@ class BoardController extends Board
} }
$oComment->setGrantForSession(); $oComment->setGrantForSession();
} else { }
else
{
// get the document information // get the document information
$oDocument = DocumentModel::getDocument($document_srl); $oDocument = DocumentModel::getDocument($document_srl);
if(!$oDocument->isExists()) if(!$oDocument->isExists())
@ -810,8 +805,8 @@ class BoardController extends Board
/** /**
* @brief the trigger for displaying 'view document' link when click the user ID * @brief the trigger for displaying 'view document' link when click the user ID
**/ */
function triggerMemberMenu($obj) public function triggerMemberMenu($obj)
{ {
if(!$mid = Context::get('cur_mid')) if(!$mid = Context::get('cur_mid'))
{ {

View file

@ -3,7 +3,7 @@
class BoardMobile extends BoardView class BoardMobile extends BoardView
{ {
function getBoardCommentPage() public function getBoardCommentPage()
{ {
$this->dispBoardCommentPage(); $this->dispBoardCommentPage();
$oTemplate = TemplateHandler::getInstance(); $oTemplate = TemplateHandler::getInstance();

View file

@ -5,19 +5,12 @@
* @class boardModel * @class boardModel
* @author NAVER (developers@xpressengine.com) * @author NAVER (developers@xpressengine.com)
* @brief board module Model class * @brief board module Model class
**/ */
class BoardModel extends Board class BoardModel extends Board
{ {
/**
* @brief initialization
**/
function init()
{
}
/** /**
* @brief get the list configuration * @brief get the list configuration
**/ */
public static function getListConfig($module_srl) public static function getListConfig($module_srl)
{ {
// get the list config value, if it is not exitsted then setup the default value // get the list config value, if it is not exitsted then setup the default value
@ -54,7 +47,7 @@ class BoardModel extends Board
/** /**
* @brief return the default list configration value * @brief return the default list configration value
**/ */
public static function getDefaultListConfig($module_srl) public static function getDefaultListConfig($module_srl)
{ {
$extra_vars = []; $extra_vars = [];
@ -85,7 +78,7 @@ class BoardModel extends Board
/** /**
* @brief return module name in sitemap * @brief return module name in sitemap
**/ */
public function triggerModuleListInSitemap(&$obj) public function triggerModuleListInSitemap(&$obj)
{ {
array_push($obj, 'board'); array_push($obj, 'board');