From 59b31b5b1e34b41a44efb22e59cb9c2999fdaadc Mon Sep 17 00:00:00 2001 From: hufangzhou Date: Wed, 4 Aug 2021 13:11:51 +0800 Subject: [PATCH] * Delete the unused code. --- module/project/model.php | 18 ------------------ module/stakeholder/control.php | 7 ------- module/stakeholder/model.php | 16 ---------------- 3 files changed, 41 deletions(-) diff --git a/module/project/model.php b/module/project/model.php index 6cff380fc7..28ae202f57 100644 --- a/module/project/model.php +++ b/module/project/model.php @@ -969,17 +969,6 @@ class projectModel extends model if($executions) $this->user->updateUserView($executions, 'sprint'); } - if($project->acl == 'private') - { - /* Update the viewers of the project main doc library. */ - $projectStakeHolders = $this->loadModel('stakeholder')->getStakeHolderPairs($projectID); - $projectStakeHolders = implode(',', $projectStakeHolders); - $authorizedUsers = $project->whitelist . ',' . $project->PM . ',' . $oldProject->openedBy . ',' . $projectStakeHolders; - $authorizedUsers = array_unique(explode(',', trim($authorizedUsers, ','))); - $authorizedUsers = ',' . implode(',', $authorizedUsers) . ','; - $this->dao->update(TABLE_DOCLIB)->set('users')->eq($authorizedUsers)->where('type')->eq('project')->andWhere('project')->eq($projectID)->andWhere('main')->eq(1)->exec(); - } - if($oldProject->parent != $project->parent) $this->loadModel('program')->processNode($projectID, $project->parent, $oldProject->path, $oldProject->grade); return common::createChanges($oldProject, $project); @@ -1307,13 +1296,6 @@ class projectModel extends model $this->dao->insert(TABLE_TEAM)->data($member)->exec(); } - /* Update the viewers of the project main doc library. */ - $authorizedUsers = $this->dao->select('users')->from(TABLE_DOCLIB)->where('type')->eq('project')->andWhere('project')->eq($projectID)->andWhere('main')->eq(1)->fetch(); - $authorizedUsers = empty($authorizedUsers) ? array() : explode(',', trim($authorizedUsers->users, ',')); - $authorizedUsers = array_merge($authorizedUsers, array_filter($accounts)); - $authorizedUsers = ',' . implode(',', array_unique($authorizedUsers)) . ','; - $this->dao->update(TABLE_DOCLIB)->set('users')->eq($authorizedUsers)->where('type')->eq('project')->andWhere('project')->eq($projectID)->andWhere('main')->eq(1)->exec(); - /* Only changed account update userview. */ $oldAccounts = array_keys($oldJoin); $changedAccounts = array_diff($accounts, $oldAccounts); diff --git a/module/stakeholder/control.php b/module/stakeholder/control.php index aca0ef7616..f60bcad5e0 100644 --- a/module/stakeholder/control.php +++ b/module/stakeholder/control.php @@ -315,13 +315,6 @@ class stakeholder extends control if($stakeholder->objectType == 'project' and $stakeholder->objectID) { $this->loadModel('project')->updateInvolvedUserView($stakeholder->objectID, $stakeholder->user); - - /* Update the viewers of the project main doc library. */ - $authorizedUsers = $this->dao->select('users')->from(TABLE_DOCLIB)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->fetch(); - $authorizedUsers = explode(',', trim($authorizedUsers->users, ',')); - unset($authorizedUsers[array_search($stakeholder->user, $authorizedUsers)]); - $authorizedUsers = ',' . implode(',', $authorizedUsers) . ','; - $this->dao->update(TABLE_DOCLIB)->set('users')->eq($authorizedUsers)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->exec(); } die(js::reload('parent')); } diff --git a/module/stakeholder/model.php b/module/stakeholder/model.php index a6c1137e30..781ad725f8 100644 --- a/module/stakeholder/model.php +++ b/module/stakeholder/model.php @@ -106,15 +106,6 @@ class stakeholderModel extends model if($stakeholder->objectType == 'project' and $stakeholder->objectID) { $this->loadModel('project')->updateInvolvedUserView($stakeholder->objectID, $stakeholder->user); - - /* Update the viewers of the project main doc library. */ - $authorizedUsers = $this->dao->select('users')->from(TABLE_DOCLIB)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->fetch(); - $authorizedUsers = empty($authorizedUsers) ? array() : explode(',', trim($authorizedUsers->users, ',')); - if(!in_array($stakeholder->user, $authorizedUsers)) - { - $authorizedUsers = ',' . implode(',', $authorizedUsers) . ',' . $stakeholder->user . ','; - $this->dao->update(TABLE_DOCLIB)->set('users')->eq($authorizedUsers)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->exec(); - } } if($stakeholder->objectType == 'program' and $stakeholder->objectID) @@ -182,13 +173,6 @@ class stakeholderModel extends model $this->loadModel('project')->updateInvolvedUserView($projectID, $changedAccounts); - /* Update the viewers of the project main doc library. */ - $authorizedUsers = $this->dao->select('users')->from(TABLE_DOCLIB)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->fetch(); - $authorizedUsers = empty($authorizedUsers) ? array() : explode(',', trim($authorizedUsers->users, ',')); - $authorizedUsers = array_merge($authorizedUsers, array_filter($accounts)); - $authorizedUsers = ',' . implode(',', array_unique($authorizedUsers)) . ','; - $this->dao->update(TABLE_DOCLIB)->set('users')->eq($authorizedUsers)->where('type')->eq('project')->andWhere('project')->eq($stakeholder->objectID)->andWhere('main')->eq(1)->exec(); - if($stakeholder->objectType == 'program' and $stakeholder->objectID) { $programID = $stakeholder->objectID;