fixed lint
Signed-off-by: Konrad <konrad@kola-entertainments.de>
This commit is contained in:
parent
c1ee648998
commit
cf3988ef52
|
|
@ -43,47 +43,47 @@ func (iw *IssueDependency) BeforeUpdate() {
|
||||||
}
|
}
|
||||||
|
|
||||||
// CreateIssueDependency creates a new dependency for an issue
|
// CreateIssueDependency creates a new dependency for an issue
|
||||||
func CreateIssueDependency(user *User, issue, dep *Issue) (err error, exists bool) {
|
func CreateIssueDependency(user *User, issue, dep *Issue) (exists bool, err error) {
|
||||||
sess := x.NewSession()
|
sess := x.NewSession()
|
||||||
|
|
||||||
// TODO: Move this to the appropriate place
|
// TODO: Move this to the appropriate place
|
||||||
err = x.Sync(new(IssueDependency))
|
err = x.Sync(new(IssueDependency))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err, exists
|
return exists, err
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check if it aleready exists
|
// Check if it aleready exists
|
||||||
exists, err = issueDepExists(x, issue.ID, dep.ID)
|
exists, err = issueDepExists(x, issue.ID, dep.ID)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err, exists
|
return exists, err
|
||||||
}
|
}
|
||||||
|
|
||||||
// If it not exists, create it, otherwise show an error message
|
// If it not exists, create it, otherwise show an error message
|
||||||
if !exists {
|
if !exists {
|
||||||
newId := new(IssueDependency)
|
newID := new(IssueDependency)
|
||||||
newId.UserID = user.ID
|
newID.UserID = user.ID
|
||||||
newId.IssueID = issue.ID
|
newID.IssueID = issue.ID
|
||||||
newId.DependencyID = dep.ID
|
newID.DependencyID = dep.ID
|
||||||
|
|
||||||
if _, err := x.Insert(newId); err != nil {
|
if _, err := x.Insert(newID); err != nil {
|
||||||
return err, exists
|
return exists, err
|
||||||
}
|
}
|
||||||
|
|
||||||
// Add comment referencing the new dependency
|
// Add comment referencing the new dependency
|
||||||
_, err = createIssueDependencyComment(sess, user, issue, dep, true)
|
_, err = createIssueDependencyComment(sess, user, issue, dep, true)
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err, exists
|
return exists, err
|
||||||
}
|
}
|
||||||
|
|
||||||
// Create a new comment for the dependent issue
|
// Create a new comment for the dependent issue
|
||||||
_, err = createIssueDependencyComment(sess, user, dep, issue, true)
|
_, err = createIssueDependencyComment(sess, user, dep, issue, true)
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err, exists
|
return exists, err
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return nil, exists
|
return exists, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// RemoveIssueDependency removes a dependency from an issue
|
// RemoveIssueDependency removes a dependency from an issue
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user