hyung-hwan/codepot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

changed the issue model to delete rows from issue_coderev when an issue is deleted.

Browse Source 
removed a few unused functions from the issue model
This commit is contained in:
hyung-hwan 2016-01-23 15:33:05 +00:00
parent 8664ed78f3
commit 947a878869
2 changed files with 78 additions and 205 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
codepot/etc/codepot.mysql
View File

@ -186,7 +186,7 @@ CREATE TABLE issue_coderev (
CONSTRAINT issue_coderev_codeproid FOREIGN KEY (codeproid) REFERENCES project(id) CONSTRAINT issue_coderev_codeproid FOREIGN KEY (codeproid) REFERENCES project(id)
ON DELETE RESTRICT ON UPDATE CASCADE ON DELETE RESTRICT ON UPDATE CASCADE
-- Commit message is typypicall free text. Its issue reference could be error-prone. -- Commit message is typically a free text. Its issue reference could be error-prone.
-- So i won't have this constraint enforced. -- So i won't have this constraint enforced.
-- CONSTRAINT issue_coderev_issueid FOREIGN KEY (projectid,issueid) REFERENCES issue(projectid,id) -- CONSTRAINT issue_coderev_issueid FOREIGN KEY (projectid,issueid) REFERENCES issue(projectid,id)
-- ON DELETE RESTRICT ON UPDATE CASCADE -- ON DELETE RESTRICT ON UPDATE CASCADE

283
codepot/src/codepot/models/issuemodel.php
View File

@ -29,6 +29,7 @@ class IssueModel extends Model
$query = $this->db->get ('issue'); $query = $this->db->get ('issue');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback (); $this->db->trans_rollback ();
return FALSE; return FALSE;
} }
@ -36,6 +37,7 @@ class IssueModel extends Model
$result = $query->result (); $result = $query->result ();
if (empty($result)) if (empty($result))
{ {
$this->errmsg = $this->db->_error_message();
$this->db->trans_commit (); $this->db->trans_commit ();
return NULL; return NULL;
} }
@ -45,6 +47,7 @@ class IssueModel extends Model
$query = $this->db->get ('issue_file_list'); $query = $this->db->get ('issue_file_list');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback (); $this->db->trans_rollback ();
return FALSE; return FALSE;
} }
@ -56,6 +59,7 @@ class IssueModel extends Model
$query = $this->db->get ('issue_change'); $query = $this->db->get ('issue_change');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback (); $this->db->trans_rollback ();
return FALSE; return FALSE;
} }
@ -70,7 +74,7 @@ class IssueModel extends Model
function getNumEntries ($userid, $project, $search) function getNumEntries ($userid, $project, $search)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
if ($search->type != '') $this->db->where ('type', $search->type); if ($search->type != '') $this->db->where ('type', $search->type);
@ -82,24 +86,22 @@ class IssueModel extends Model
$query = $this->db->get ('issue'); $query = $this->db->get ('issue');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->db->trans_complete (); $this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE; return FALSE;
} }
$result = $query->result(); $result = $query->result();
$num = empty($result)? 0: $num = empty($result)? 0:
isset($result[0]->COUNT)? $result[0]->COUNT: $result[0]->count; isset($result[0]->COUNT)? $result[0]->COUNT: $result[0]->count;
$this->db->trans_complete (); $this->db->trans_commit();
if ($this->db->trans_status() === FALSE) return FALSE;
return $num; return $num;
} }
function getEntries ($userid, $offset, $limit, $project, $search) function getEntries ($userid, $offset, $limit, $project, $search)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
if ($search->type != '') $this->db->where ('type', $search->type); if ($search->type != '') $this->db->where ('type', $search->type);
@ -109,27 +111,37 @@ class IssueModel extends Model
if ($search->summary != '') $this->db->like ('summary', $search->summary); if ($search->summary != '') $this->db->like ('summary', $search->summary);
$this->db->order_by ('id', 'desc'); $this->db->order_by ('id', 'desc');
$query = $this->db->get ('issue', $limit, $offset); $query = $this->db->get ('issue', $limit, $offset);
$this->db->trans_complete (); if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
if ($this->db->trans_status() === FALSE) return FALSE; $this->db->trans_commit ();
return $query->result (); return $query->result ();
} }
function getAll ($userid, $project) function getAll ($userid, $project)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
$this->db->order_by ('id', 'desc'); $this->db->order_by ('id', 'desc');
$query = $this->db->get ('issue'); $query = $this->db->get ('issue');
$this->db->trans_complete (); if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
if ($this->db->trans_status() === FALSE) return FALSE; $this->db->trans_commit ();
return $query->result (); return $query->result ();
} }
function getMyIssues ($userid, $status_filter, $hour_limit = 0) function getMyIssues ($userid, $status_filter, $hour_limit = 0)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
if (strlen($userid) > 0) $this->db->where ('owner', $userid); if (strlen($userid) > 0) $this->db->where ('owner', $userid);
if (is_array($status_filter)) if (is_array($status_filter))
@ -153,10 +165,15 @@ class IssueModel extends Model
} }
$query = $this->db->get ('issue'); $query = $this->db->get ('issue');
$this->db->trans_complete (); if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
if ($this->db->trans_status() === FALSE) return FALSE; $this->db->trans_commit ();
return $query->result (); return $query->result ();
} }
@ -230,7 +247,7 @@ class IssueModel extends Model
function getFile ($userid, $project, $issueid, $filename) function getFile ($userid, $project, $issueid, $filename)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
$this->db->select ('filename,encname,md5sum,description,createdon,createdby'); $this->db->select ('filename,encname,md5sum,description,createdon,createdby');
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
@ -238,166 +255,20 @@ class IssueModel extends Model
$this->db->where ('filename', $filename); $this->db->where ('filename', $filename);
$query = $this->db->get ('issue_file_list'); $query = $this->db->get ('issue_file_list');
$this->db->trans_complete (); if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->trans_commit();
if ($this->db->trans_status() === FALSE) return FALSE;
$result = $query->result (); $result = $query->result ();
if (empty($result)) return NULL; if (empty($result)) return NULL;
return $result[0]; return $result[0];
} }
function create ($userid, $issue)
{
$now = codepot_nowtodbdate();
// TODO: check if userid can do this..
$this->db->trans_begin ();
$this->db->where ('projectid', $issue->projectid);
$this->db->select ('MAX(id) as maxid');
$query = $this->db->get ('issue');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$result = $query->result();
$maxid = (empty($result) || $result[0] == NULL)? 0: $result[0]->maxid;
$newid = $maxid + 1;
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('id', $newid);
$this->db->set ('summary', $issue->summary);
$this->db->set ('description', $issue->description);
$this->db->set ('type', $issue->type);
$this->db->set ('status', $issue->status);
$this->db->set ('owner', $issue->owner);
$this->db->set ('priority', $issue->priority);
$this->db->set ('createdon', $now);
$this->db->set ('updatedon', $now);
$this->db->set ('createdby', $userid);
$this->db->set ('updatedby', $userid);
$this->db->insert ('issue');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('id', $newid);
$this->db->set ('sno', 1);
$this->db->set ('type', $issue->type);
$this->db->set ('status', $issue->status);
$this->db->set ('owner', $issue->owner);
$this->db->set ('comment', '');
$this->db->set ('priority', $issue->priority);
$this->db->set ('createdon', $now);
$this->db->set ('createdby', $userid);
$this->db->set ('updatedon', $now);
$this->db->set ('updatedby', $userid);
$this->db->insert ('issue_change');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->set ('createdon', $now);
$this->db->set ('type', 'issue');
$this->db->set ('action', 'create');
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('userid', $userid);
$this->db->set ('message', $newid);
$this->db->insert ('log');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->trans_commit ();
return $newid;
}
function update_partial ($userid, $issue)
{
$now = codepot_nowtodbdate();
$this->db->trans_start ();
$this->db->where ('projectid', $issue->projectid);
$this->db->where ('id', $issue->id);
$this->db->set ('summary', $issue->summary);
$this->db->set ('description', $issue->description);
$this->db->set ('updatedon', $now);
$this->db->set ('updatedby', $userid);
$this->db->update ('issue');
$this->db->set ('createdon', $now);
$this->db->set ('type', 'issue');
$this->db->set ('action', 'update');
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('userid', $userid);
$this->db->set ('message', $issue->id);
$this->db->insert ('log');
$this->db->trans_complete ();
if ($this->db->trans_status() === FALSE) return FALSE;
return $issue->id;
}
function update ($userid, $issue)
{
$now = codepot_nowtodbdate();
// TODO: check if userid can do this..
$this->db->trans_start ();
$this->db->where ('projectid', $issue->projectid);
$this->db->where ('id', $issue->id);
$this->db->set ('summary', $issue->summary);
$this->db->set ('description', $issue->description);
$this->db->set ('type', $issue->type);
$this->db->set ('status', $issue->status);
$this->db->set ('owner', $issue->owner);
$this->db->set ('priority', $issue->priority);
$this->db->set ('updatedon', $now);
$this->db->set ('updatedby', $userid);
$this->db->update ('issue');
$this->db->where ('projectid', $issue->projectid);
$this->db->where ('id', $issue->id);
$this->db->where ('sno', 1);
$this->db->set ('type', $issue->type);
$this->db->set ('status', $issue->status);
$this->db->set ('owner', $issue->owner);
$this->db->set ('priority', $issue->priority);
$this->db->set ('comment', '');
$this->db->set ('updatedon', $now);
$this->db->set ('updatedby', $userid);
$this->db->update ('issue_change');
$this->db->set ('createdon', $now);
$this->db->set ('type', 'issue');
$this->db->set ('action', 'update');
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('userid', $userid);
$this->db->set ('message', $issue->id);
$this->db->insert ('log');
$this->db->trans_complete ();
if ($this->db->trans_status() === FALSE) return FALSE;
return $issue->id;
}
function change ($userid, $project, $id, $change, $disallow_state_change) function change ($userid, $project, $id, $change, $disallow_state_change)
{ {
$now = codepot_nowtodbdate(); $now = codepot_nowtodbdate();
@ -514,7 +385,7 @@ class IssueModel extends Model
function undo_last_change ($userid, $project, $id) function undo_last_change ($userid, $project, $id)
{ {
$this->db->trans_start (); $this->db->trans_begin ();
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
$this->db->where ('id', $id); $this->db->where ('id', $id);
@ -522,9 +393,11 @@ class IssueModel extends Model
$query = $this->db->get ('issue_change'); $query = $this->db->get ('issue_change');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->db->trans_complete (); $this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE; return FALSE;
} }
$result = $query->result(); $result = $query->result();
if (!empty($result)) if (!empty($result))
{ {
@ -535,6 +408,12 @@ class IssueModel extends Model
$this->db->where ('id', $id); $this->db->where ('id', $id);
$this->db->where ('sno', $maxsno); $this->db->where ('sno', $maxsno);
$this->db->delete ('issue_change'); $this->db->delete ('issue_change');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->where ('projectid', $project->id); $this->db->where ('projectid', $project->id);
$this->db->where ('id', $id); $this->db->where ('id', $id);
@ -542,9 +421,11 @@ class IssueModel extends Model
$query = $this->db->get ('issue_change'); $query = $this->db->get ('issue_change');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->db->trans_complete (); $this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE; return FALSE;
} }
$result = $query->result(); $result = $query->result();
if (!empty($result)) if (!empty($result))
{ {
@ -555,9 +436,11 @@ class IssueModel extends Model
$query = $this->db->get ('issue_change'); $query = $this->db->get ('issue_change');
if ($this->db->trans_status() === FALSE) if ($this->db->trans_status() === FALSE)
{ {
$this->db->trans_complete (); $this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE; return FALSE;
} }
$result = $query->result(); $result = $query->result();
if (!empty($result)) if (!empty($result))
{ {
@ -571,40 +454,20 @@ class IssueModel extends Model
$this->db->set ('updatedon', $change->updatedon); $this->db->set ('updatedon', $change->updatedon);
$this->db->set ('updatedby', $change->updatedby); $this->db->set ('updatedby', $change->updatedby);
$this->db->update ('issue'); $this->db->update ('issue');
} if ($this->db->trans_status() === FALSE)
}
}
}
$this->db->trans_complete ();
return $this->db->trans_status();
}
function delete ($userid, $issue)
{ {
// TODO: check if userid can do this.. $this->errmsg = $this->db->_error_message();
$this->db->trans_start (); $this->db->trans_rollback ();
return FALSE;
$this->db->where ('projectid', $issue->projectid); }
$this->db->where ('id', $issue->id); }
$this->db->delete ('issue_change'); }
}
$this->db->where ('projectid', $issue->projectid);
$this->db->where ('id', $issue->id);
$this->db->delete ('issue');
$this->db->set ('createdon', codepot_nowtodbdate());
$this->db->set ('type', 'issue');
$this->db->set ('action', 'delete');
$this->db->set ('projectid', $issue->projectid);
$this->db->set ('userid', $userid);
$this->db->set ('message', $issue->id);
$this->db->insert ('log');
$this->db->trans_complete ();
return $this->db->trans_status();
} }
$this->db->trans_commit ();
return TRUE;
}
private function delete_all_files ($files) private function delete_all_files ($files)
{ {
@ -820,6 +683,16 @@ class IssueModel extends Model
return FALSE; return FALSE;
} }
$this->db->where ('projectid', $projectid);
$this->db->where ('issueid', $issueid);
$this->db->delete ('issue_coderev');
if ($this->db->trans_status() === FALSE)
{
$this->errmsg = $this->db->_error_message();
$this->db->trans_rollback ();
return FALSE;
}
$this->db->where ('projectid', $projectid); $this->db->where ('projectid', $projectid);
$this->db->where ('id', $issueid); $this->db->where ('id', $issueid);
$this->db->delete ('issue_change'); $this->db->delete ('issue_change');