peter/go-gitea
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fixed creating a new dependency when the other issue didn't exists

Browse Source 
Signed-off-by: Konrad <konrad@kola-entertainments.de>
This commit is contained in:
Konrad Langenberg 2017-08-27 20:32:57 +02:00 committed by Jonas Franz
parent b6ecf50342
commit 0661b57134
2 changed files with 31 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
models/issue_dependency_add.go
View File

@ -42,42 +42,48 @@ func (iw *IssueDependency) BeforeUpdate() {
iw.UpdatedUnix = u
}
// CreateOrUpdateIssueDependency creates a new dependency for an issue
func CreateOrUpdateIssueDependency(userID, issueID int64, depID int64) (err error, exists bool) {
// CreateIssueDependency creates a new dependency for an issue
func CreateIssueDependency(userID, issueID int64, depID int64) (err error, exists bool, depExists bool) {
err = x.Sync(new(IssueDependency))
if err != nil {
return err, exists
return err, exists, false
}
// Check if it aleready exists
exists, err = issueDepExists(x, issueID, depID)
if err != nil {
return err, exists
return err, exists, false
}
// If it not exists, create it, otherwise show an error message
if !exists {
newId := new(IssueDependency)
newId.UserID = userID
newId.IssueID = issueID
newId.DependencyID = depID
// Check if the other issue exists
var issue = Issue{}
issueExists, err := x.Id(depID).Get(&issue)
if issueExists {
newId := new(IssueDependency)
newId.UserID = userID
newId.IssueID = issueID
newId.DependencyID = depID
if _, err := x.Insert(newId); err != nil {
return err, exists
}
if _, err := x.Insert(newId); err != nil {
return err, exists, false
}
// Add comment referencing to the stopwatch
comment := &Comment{
IssueID: issueID,
PosterID: userID,
Type: CommentTypeAddedDependency,
}
// Add comment referencing the new dependency
comment := &Comment{
IssueID: issueID,
PosterID: userID,
Type: CommentTypeAddedDependency,
}
if _, err := x.Insert(comment); err != nil {
return err, exists
if _, err := x.Insert(comment); err != nil {
return err, exists, false
}
}
return err, exists, true
}
return nil, exists
return nil, exists, false
}
// Check if the dependency already exists

6
routers/repo/issue_dependency_add.go
View File

@ -28,12 +28,16 @@ func AddDependency(c *context.Context) {
return
}
err, exists := models.CreateOrUpdateIssueDependency(c.User.ID, issue.ID, dep);
err, exists, depExists := models.CreateIssueDependency(c.User.ID, issue.ID, dep);
if err != nil {
c.Handle(http.StatusInternalServerError, "CreateOrUpdateIssueDependency", err)
return
}
if depExists {
c.Flash.Error("Dependend issue does not exist!")
}
if exists {
c.Flash.Error("Dependency already exists!")
}