diff options
author | Michael Muré <batolettre@gmail.com> | 2019-06-16 21:04:36 +0200 |
---|---|---|
committer | Michael Muré <batolettre@gmail.com> | 2019-06-16 21:04:36 +0200 |
commit | 08c0e18ade5241d124fc8a3424b7612174e82cef (patch) | |
tree | e7b1c0203d62e83ab8368d8a5974749fa97506dc | |
parent | d69dcce806d280ddbd6a4fb17700153bc03da90b (diff) | |
download | git-bug-08c0e18ade5241d124fc8a3424b7612174e82cef.tar.gz |
cache: expose the operation when creating a new bug
-rw-r--r-- | bridge/github/import.go | 4 | ||||
-rw-r--r-- | bridge/launchpad/import.go | 2 | ||||
-rw-r--r-- | cache/repo_cache.go | 18 | ||||
-rw-r--r-- | commands/add.go | 2 | ||||
-rw-r--r-- | termui/termui.go | 2 |
5 files changed, 14 insertions, 14 deletions
diff --git a/bridge/github/import.go b/bridge/github/import.go index edb97c4f..6cfd022e 100644 --- a/bridge/github/import.go +++ b/bridge/github/import.go @@ -106,7 +106,7 @@ func (gi *githubImporter) ensureIssue(repo *cache.RepoCache, issue issueTimeline } // create bug - b, err = repo.NewBugRaw( + b, _, err = repo.NewBugRaw( author, issue.CreatedAt.Unix(), issue.Title, @@ -140,7 +140,7 @@ func (gi *githubImporter) ensureIssue(repo *cache.RepoCache, issue issueTimeline // if the bug doesn't exist if b == nil { // we create the bug as soon as we have a legit first edition - b, err = repo.NewBugRaw( + b, _, err = repo.NewBugRaw( author, issue.CreatedAt.Unix(), issue.Title, diff --git a/bridge/launchpad/import.go b/bridge/launchpad/import.go index 177ff3fc..63101d9c 100644 --- a/bridge/launchpad/import.go +++ b/bridge/launchpad/import.go @@ -74,7 +74,7 @@ func (li *launchpadImporter) ImportAll(repo *cache.RepoCache, since time.Time) e if err == bug.ErrBugNotExist { createdAt, _ := time.Parse(time.RFC3339, lpBug.CreatedAt) - b, err = repo.NewBugRaw( + b, _, err = repo.NewBugRaw( owner, createdAt.Unix(), lpBug.Title, diff --git a/cache/repo_cache.go b/cache/repo_cache.go index b3b2ea1e..a80bc7c9 100644 --- a/cache/repo_cache.go +++ b/cache/repo_cache.go @@ -563,16 +563,16 @@ func (c *RepoCache) ValidLabels() []bug.Label { // NewBug create a new bug // The new bug is written in the repository (commit) -func (c *RepoCache) NewBug(title string, message string) (*BugCache, error) { +func (c *RepoCache) NewBug(title string, message string) (*BugCache, *bug.CreateOperation, error) { return c.NewBugWithFiles(title, message, nil) } // NewBugWithFiles create a new bug with attached files for the message // The new bug is written in the repository (commit) -func (c *RepoCache) NewBugWithFiles(title string, message string, files []git.Hash) (*BugCache, error) { +func (c *RepoCache) NewBugWithFiles(title string, message string, files []git.Hash) (*BugCache, *bug.CreateOperation, error) { author, err := c.GetUserIdentity() if err != nil { - return nil, err + return nil, nil, err } return c.NewBugRaw(author, time.Now().Unix(), title, message, files, nil) @@ -581,10 +581,10 @@ func (c *RepoCache) NewBugWithFiles(title string, message string, files []git.Ha // NewBugWithFilesMeta create a new bug with attached files for the message, as // well as metadata for the Create operation. // The new bug is written in the repository (commit) -func (c *RepoCache) NewBugRaw(author *IdentityCache, unixTime int64, title string, message string, files []git.Hash, metadata map[string]string) (*BugCache, error) { +func (c *RepoCache) NewBugRaw(author *IdentityCache, unixTime int64, title string, message string, files []git.Hash, metadata map[string]string) (*BugCache, *bug.CreateOperation, error) { b, op, err := bug.CreateWithFiles(author.Identity, unixTime, title, message, files) if err != nil { - return nil, err + return nil, nil, err } for key, value := range metadata { @@ -593,11 +593,11 @@ func (c *RepoCache) NewBugRaw(author *IdentityCache, unixTime int64, title strin err = b.Commit(c.repo) if err != nil { - return nil, err + return nil, nil, err } if _, has := c.bugs[b.Id()]; has { - return nil, fmt.Errorf("bug %s already exist in the cache", b.Id()) + return nil, nil, fmt.Errorf("bug %s already exist in the cache", b.Id()) } cached := NewBugCache(c, b) @@ -606,10 +606,10 @@ func (c *RepoCache) NewBugRaw(author *IdentityCache, unixTime int64, title strin // force the write of the excerpt err = c.bugUpdated(b.Id()) if err != nil { - return nil, err + return nil, nil, err } - return cached, nil + return cached, op, nil } // Fetch retrieve updates from a remote diff --git a/commands/add.go b/commands/add.go index ea40227c..d199b4c7 100644 --- a/commands/add.go +++ b/commands/add.go @@ -44,7 +44,7 @@ func runAddBug(cmd *cobra.Command, args []string) error { } } - b, err := backend.NewBug(addTitle, addMessage) + b, _, err := backend.NewBug(addTitle, addMessage) if err != nil { return err } diff --git a/termui/termui.go b/termui/termui.go index 54d9df69..46f9dd06 100644 --- a/termui/termui.go +++ b/termui/termui.go @@ -190,7 +190,7 @@ func newBugWithEditor(repo *cache.RepoCache) error { return errTerminateMainloop } else { - b, err = repo.NewBug(title, message) + b, _, err = repo.NewBug(title, message) if err != nil { return err } |