From 64368f82a02c083cce94bba739c376348da6907d Mon Sep 17 00:00:00 2001 From: zhouxin Date: Tue, 30 May 2023 11:21:18 +0800 Subject: [PATCH] * Batch replace "else if" sequence to "elseif". --- module/api/model.php | 2 +- module/bug/control.php | 2 +- module/build/control.php | 4 ++-- module/caselib/model.php | 2 +- module/common/model.php | 2 +- module/doc/model.php | 24 ++++++++++++------------ module/file/model.php | 2 +- module/kanban/model.php | 2 +- module/mr/model.php | 4 ++-- module/program/model.php | 2 +- module/repo/control.php | 2 +- module/repo/model.php | 2 +- module/screen/model.php | 8 ++++---- module/search/model.php | 2 +- module/tree/control.php | 10 +++++----- module/user/control.php | 2 +- 16 files changed, 36 insertions(+), 36 deletions(-) diff --git a/module/api/model.php b/module/api/model.php index 73b3ce6ef3..8824421416 100644 --- a/module/api/model.php +++ b/module/api/model.php @@ -1021,7 +1021,7 @@ class apiModel extends model { $normalObjects['project'][$id] = $project->name; } - else if($project->status == 'done' or $project->status == 'closed') + elseif($project->status == 'done' or $project->status == 'closed') { $closedObjects['project'][$id] = $project->name; } diff --git a/module/bug/control.php b/module/bug/control.php index 29cb2cf6ce..c0e50bacbc 100755 --- a/module/bug/control.php +++ b/module/bug/control.php @@ -1074,7 +1074,7 @@ class bug extends control $bug->id = (int)$bugID; $this->bug->close($bug); } - else if($oldBug->status != 'closed') + elseif($oldBug->status != 'closed') { $skipBugs[$bugID] = $bugID; } diff --git a/module/build/control.php b/module/build/control.php index 3fb81d5d31..ebaf4fbce2 100644 --- a/module/build/control.php +++ b/module/build/control.php @@ -637,7 +637,7 @@ class build extends control { unset($this->config->product->search['fields']['plan']); } - else if(!$project->hasProduct and !$project->multiple) + elseif(!$project->hasProduct and !$project->multiple) { unset($this->config->product->search['fields']['plan']); } @@ -869,7 +869,7 @@ class build extends control { unset($this->config->bug->search['fields']['plan']); } - else if(!$project->hasProduct and !$project->multiple) + elseif(!$project->hasProduct and !$project->multiple) { unset($this->config->bug->search['fields']['plan']); } diff --git a/module/caselib/model.php b/module/caselib/model.php index cba16680af..a58b6468be 100644 --- a/module/caselib/model.php +++ b/module/caselib/model.php @@ -327,7 +327,7 @@ class caselibModel extends model $link = helper::createLink($module, $method, "libID=%s"); } } - else if($module == 'tree') + elseif($module == 'tree') { $link = helper::createLink($module, $method, "libID=%s&type=caselib¤tModuleID=0"); } diff --git a/module/common/model.php b/module/common/model.php index e5fcde1241..8766ec5c87 100644 --- a/module/common/model.php +++ b/module/common/model.php @@ -1718,7 +1718,7 @@ EOF; { $clickable = $extraEnabled; } - else if(is_object($object)) + elseif(is_object($object)) { if($app->getModuleName() != $module) $app->control->loadModel($module); $modelClass = class_exists("ext{$module}Model") ? "ext{$module}Model" : $module . "Model"; diff --git a/module/doc/model.php b/module/doc/model.php index ca4f7a8470..4914af5434 100644 --- a/module/doc/model.php +++ b/module/doc/model.php @@ -1752,11 +1752,11 @@ class docModel extends model { $myObjects[$id] = $project->name; } - else if($project->status != 'done' and $project->status != 'closed' and !($project->PM == $this->app->user->account)) + elseif($project->status != 'done' and $project->status != 'closed' and !($project->PM == $this->app->user->account)) { $normalObjects[$id] = $project->name; } - else if($project->status == 'done' or $project->status == 'closed') + elseif($project->status == 'done' or $project->status == 'closed') { $closedObjects[$id] = $project->name; } @@ -1790,11 +1790,11 @@ class docModel extends model { $myObjects[$id] = $execution->name; } - else if($execution->status != 'done' and $execution->status != 'closed' and !($execution->PM == $this->app->user->account)) + elseif($execution->status != 'done' and $execution->status != 'closed' and !($execution->PM == $this->app->user->account)) { $normalObjects[$id] = $execution->name; } - else if($execution->status == 'done' or $execution->status == 'closed') + elseif($execution->status == 'done' or $execution->status == 'closed') { $closedObjects[$id] = $execution->name; } @@ -2010,14 +2010,14 @@ class docModel extends model $iconClass = 'icon-file'; if(strpos('zip,tar,gz,bz2,rar', $file->extension) !== false) $iconClass = 'icon-file-archive'; - else if(strpos('csv,xls,xlsx', $file->extension) !== false) $iconClass = 'icon-file-excel'; - else if(strpos('doc,docx', $file->extension) !== false) $iconClass = 'icon-file-word'; - else if(strpos('ppt,pptx', $file->extension) !== false) $iconClass = 'icon-file-powerpoint'; - else if(strpos('pdf', $file->extension) !== false) $iconClass = 'icon-file-pdf'; - else if(strpos('mp3,ogg,wav', $file->extension) !== false) $iconClass = 'icon-file-audio'; - else if(strpos('avi,mp4,mov', $file->extension) !== false) $iconClass = 'icon-file-video'; - else if(strpos('txt,md', $file->extension) !== false) $iconClass = 'icon-file-text'; - else if(strpos('html,htm', $file->extension) !== false) $iconClass = 'icon-globe'; + elseif(strpos('csv,xls,xlsx', $file->extension) !== false) $iconClass = 'icon-file-excel'; + elseif(strpos('doc,docx', $file->extension) !== false) $iconClass = 'icon-file-word'; + elseif(strpos('ppt,pptx', $file->extension) !== false) $iconClass = 'icon-file-powerpoint'; + elseif(strpos('pdf', $file->extension) !== false) $iconClass = 'icon-file-pdf'; + elseif(strpos('mp3,ogg,wav', $file->extension) !== false) $iconClass = 'icon-file-audio'; + elseif(strpos('avi,mp4,mov', $file->extension) !== false) $iconClass = 'icon-file-video'; + elseif(strpos('txt,md', $file->extension) !== false) $iconClass = 'icon-file-text'; + elseif(strpos('html,htm', $file->extension) !== false) $iconClass = 'icon-globe'; $fileIcon[$file->id] = ""; } diff --git a/module/file/model.php b/module/file/model.php index 7eb18f0693..6bb9362c6a 100755 --- a/module/file/model.php +++ b/module/file/model.php @@ -1123,7 +1123,7 @@ class fileModel extends model { return file_exists($file->realPath) ? getimagesize($file->realPath) : array(0, 0, $file->extension); } - else if($this->config->file->storageType == 's3') + elseif($this->config->file->storageType == 's3') { $this->app->loadClass('ossclient', true); diff --git a/module/kanban/model.php b/module/kanban/model.php index 4a266d5ffb..f80e5b4981 100755 --- a/module/kanban/model.php +++ b/module/kanban/model.php @@ -763,7 +763,7 @@ class kanbanModel extends model { $regionIDList = array_keys($regions); } - else if(!is_array($regionIDList)) + elseif(!is_array($regionIDList)) { $singleRegion = $regionIDList; $regionIDList = array($regionIDList); diff --git a/module/mr/model.php b/module/mr/model.php index 178d87a02a..cd5a1b1c8e 100644 --- a/module/mr/model.php +++ b/module/mr/model.php @@ -845,7 +845,7 @@ class mrModel extends model $url = sprintf($this->gitlab->getApiRoot($hostID), "/projects/$projectID/merge_requests/$MRID"); $MR = json_decode(commonModel::http($url)); } - else if($host->type == 'gitea') + elseif($host->type == 'gitea') { $url = sprintf($this->loadModel('gitea')->getApiRoot($hostID), "/repos/$projectID/pulls/$MRID"); $MR = json_decode(commonModel::http($url)); @@ -868,7 +868,7 @@ class mrModel extends model $MR->has_conflicts = empty($diff) ? true : false; } } - else if($host->type == 'gogs') + elseif($host->type == 'gogs') { $url = sprintf($this->loadModel('gogs')->getApiRoot($hostID), "/repos/$projectID/pulls/$MRID"); $MR = json_decode(commonModel::http($url)); diff --git a/module/program/model.php b/module/program/model.php index 592e0285b5..b28858fc7e 100644 --- a/module/program/model.php +++ b/module/program/model.php @@ -1608,7 +1608,7 @@ class programModel extends model $menu .= $this->buildMenu('program', 'delete', $params, $program, $type, 'trash', 'hiddenwin', '', '', '', $this->lang->program->delete); } } - else if($program->type == 'project') + elseif($program->type == 'project') { if($program->status == 'wait' || $program->status == 'suspended') { diff --git a/module/repo/control.php b/module/repo/control.php index 11ca0fbb43..d3985e1dbe 100644 --- a/module/repo/control.php +++ b/module/repo/control.php @@ -52,7 +52,7 @@ class repo extends control { $this->loadModel('project')->setMenu($objectID); } - else if($tab == 'execution') + elseif($tab == 'execution') { $this->loadModel('execution')->setMenu($objectID); } diff --git a/module/repo/model.php b/module/repo/model.php index 33b96012b2..5c8b1bcffd 100644 --- a/module/repo/model.php +++ b/module/repo/model.php @@ -1539,7 +1539,7 @@ class repoModel extends model $command = "$client info --username $account --password $password --non-interactive --trust-server-cert-failures=cn-mismatch --trust-server-cert --no-auth-cache $path 2>&1"; if(version_compare($svnVersion, '1.9', '<')) $command = "$client info --username $account --password $password --non-interactive --trust-server-cert --no-auth-cache $path 2>&1"; } - else if(stripos($path, 'file://') === 1) + elseif(stripos($path, 'file://') === 1) { $command = "$client info --non-interactive --no-auth-cache $path 2>&1"; } diff --git a/module/screen/model.php b/module/screen/model.php index 158a09c566..a3cd6d86aa 100644 --- a/module/screen/model.php +++ b/module/screen/model.php @@ -168,7 +168,7 @@ class screenModel extends model $groupComponent = $this->getLatestChart($groupComponent); } } - else if($component) + elseif($component) { $component = $this->getLatestChart($component); } @@ -249,7 +249,7 @@ class screenModel extends model $component->option->yAxis = new stdclass(); $component->option->yAxis->show = false; } - else if($type == 'pivot') + elseif($type == 'pivot') { $component->option->ineffective = 1; $component->option->header = array(); @@ -686,7 +686,7 @@ class screenModel extends model { $filter['default'] = null; } - else if(empty($begin) or empty($end)) + elseif(empty($begin) or empty($end)) { $filter['default'] = empty($begin) ? strtotime($end) * 1000 : strtotime($begin) * 1000; } @@ -1037,7 +1037,7 @@ class screenModel extends model { $value = count($results); } - else if($settings->value->agg === 'sum') + elseif($settings->value->agg === 'sum') { foreach($results as $result) { diff --git a/module/search/model.php b/module/search/model.php index bd793872b5..4be52047d2 100644 --- a/module/search/model.php +++ b/module/search/model.php @@ -547,7 +547,7 @@ class searchModel extends model $pairs = 'id,title'; $table = TABLE_STORY; } - else if($module == 'task') + elseif($module == 'task') { $pairs = 'id,name'; $table = TABLE_TASK; diff --git a/module/tree/control.php b/module/tree/control.php index 7288cf6055..f466d86153 100644 --- a/module/tree/control.php +++ b/module/tree/control.php @@ -33,19 +33,19 @@ class tree extends control { $this->product->setMenu($rootID, $branch, 0, '', $viewType); } - else if($this->app->tab == 'qa' and $viewType != 'caselib') + elseif($this->app->tab == 'qa' and $viewType != 'caselib') { $products = $this->product->getPairs('noclosed'); $this->loadModel('qa')->setMenu($products, $rootID, $branch, $viewType); } - else if($this->app->tab == 'project' and strpos($viewType, 'doc') === false) + elseif($this->app->tab == 'project' and strpos($viewType, 'doc') === false) { $this->loadModel('project')->setMenu($this->session->project); $products = $this->product->getProducts($this->session->project, 'all', '', false); if($viewType == 'case') $this->lang->modulePageNav = $this->product->select($products, $rootID, 'tree', 'browse', 'case', $branch); } - else if($this->app->tab == 'feedback') + elseif($this->app->tab == 'feedback') { $branch = 'all'; $products = $this->loadModel('feedback')->getGrantProducts(); @@ -283,7 +283,7 @@ class tree extends control } } /* viewType is workflow building category. */ - else if(strpos($viewType, '_') !== false) + elseif(strpos($viewType, '_') !== false) { $params = explode('_', $viewType); if(count($params) == 2) @@ -396,7 +396,7 @@ class tree extends control { $this->view->optionMenu = $this->tree->getTaskOptionMenu($module->root); } - else if($type != 'chart') + elseif($type != 'chart') { $this->view->optionMenu = $this->tree->getOptionMenu($module->root, $module->type, 0, $module->branch, 'noMainBranch|nodeleted'); } diff --git a/module/user/control.php b/module/user/control.php index b7b134f609..759056d016 100755 --- a/module/user/control.php +++ b/module/user/control.php @@ -962,7 +962,7 @@ class user extends control $response['message'] = sprintf($this->lang->user->loginLocked, $this->config->user->lockMinutes); return $this->send($response); } - else if($remainTimes <= 3) + elseif($remainTimes <= 3) { $response['message'] = sprintf($this->lang->user->lockWarning, $remainTimes); return $this->send($response);