Hide Snapshot::Status from public access

Author: GlancingMind

api/graphql/models/lazy_bug.go

@@ -167,7 +167,7 @@ func (l *loadedBug) LastEdit() time.Time {
 }
 
 func (l *loadedBug) Status() bug.Status {
-	return l.Snapshot.Status
+	return l.Snapshot.Status()
 }
 
 func (l *loadedBug) Title() string {

bug/bug.go

@@ -147,7 +147,7 @@ func (bug *Bug) Operations() []Operation {
 func (bug *Bug) Compile() Snapshot {
 	snap := Snapshot{
 		id:     bug.Id(),
-		Status: OpenStatus,
+		status: OpenStatus,
 	}
 
 	for _, op := range bug.Operations() {

bug/op_set_status.go

@@ -23,7 +23,7 @@ func (op *SetStatusOperation) Id() entity.Id {
 }
 
 func (op *SetStatusOperation) Apply(snapshot *Snapshot) {
-	snapshot.Status = op.Status
+	snapshot.setStatusTo(op.Status)
 	snapshot.addActor(op.Author_)
 
 	item := &SetStatusTimelineItem{

bug/snapshot.go

@@ -12,7 +12,7 @@ import (
 type Snapshot struct {
 	id entity.Id
 
-	Status       Status
+	status       Status
 	title        string
 	Comments     []Comment
 	Labels       []Label
@@ -35,6 +35,11 @@ func (snap *Snapshot) Id() entity.Id {
 	return snap.id
 }
 
+// Return the bugs status
+func (snap *Snapshot) Status() Status {
+	return snap.status
+}
+
 // Return the bugs title
 func (snap *Snapshot) Title() string {
 	return snap.title
@@ -76,6 +81,11 @@ func (snap *Snapshot) SearchComment(id entity.Id) (*Comment, error) {
 	return nil, fmt.Errorf("comment item not found")
 }
 
+// Change current status to the new status
+func (snap *Snapshot) setStatusTo(newStatus Status) {
+	snap.status = newStatus
+}
+
 // Change current title to the new title
 func (snap *Snapshot) changeTitleTo(newTitle string) {
 	snap.title = newTitle

cache/bug_excerpt.go

@@ -73,7 +73,7 @@ func NewBugExcerpt(b bug.Interface, snap *bug.Snapshot) *BugExcerpt {
 		EditLamportTime:   b.EditLamportTime(),
 		CreateUnixTime:    b.FirstOp().Time().Unix(),
 		EditUnixTime:      snap.EditTime().Unix(),
-		Status:            snap.Status,
+		Status:            snap.Status(),
 		Labels:            snap.Labels,
 		Actors:            actorsIds,
 		Participants:      participantsIds,

commands/show.go

@@ -83,7 +83,7 @@ func runShow(env *Env, opts showOptions, args []string) error {
 		case "shortId":
 			env.out.Printf("%s\n", snap.Id().Human())
 		case "status":
-			env.out.Printf("%s\n", snap.Status)
+			env.out.Printf("%s\n", snap.Status())
 		case "title":
 			env.out.Printf("%s\n", snap.Title())
 		default:
@@ -109,7 +109,7 @@ func showDefaultFormatter(env *Env, snapshot *bug.Snapshot) error {
 	// Header
 	env.out.Printf("%s [%s] %s\n\n",
 		colors.Cyan(snapshot.Id().Human()),
-		colors.Yellow(snapshot.Status),
+		colors.Yellow(snapshot.Status()),
 		snapshot.Title(),
 	)
 
@@ -216,7 +216,7 @@ func showJsonFormatter(env *Env, snapshot *bug.Snapshot) error {
 		HumanId:    snapshot.Id().Human(),
 		CreateTime: NewJSONTime(snapshot.CreateTime, 0),
 		EditTime:   NewJSONTime(snapshot.EditTime(), 0),
-		Status:     snapshot.Status.String(),
+		Status:     snapshot.Status().String(),
 		Labels:     snapshot.Labels,
 		Title:      snapshot.Title(),
 		Author:     NewJSONIdentity(snapshot.Author),
@@ -247,7 +247,7 @@ func showOrgModeFormatter(env *Env, snapshot *bug.Snapshot) error {
 	// Header
 	env.out.Printf("%s [%s] %s\n",
 		snapshot.Id().Human(),
-		snapshot.Status,
+		snapshot.Status(),
 		snapshot.Title(),
 	)
 

commands/status.go

@@ -31,7 +31,7 @@ func runStatus(env *Env, args []string) error {
 
 	snap := b.Snapshot()
 
-	env.out.Println(snap.Status)
+	env.out.Println(snap.Status())
 
 	return nil
 }

termui/show_bug.go

@@ -226,7 +226,7 @@ func (sb *showBug) renderMain(g *gocui.Gui, mainView *gocui.View) error {
 	bugHeader := fmt.Sprintf("[%s] %s\n\n[%s] %s opened this bug on %s%s",
 		colors.Cyan(snap.Id().Human()),
 		colors.Bold(snap.Title()),
-		colors.Yellow(snap.Status),
+		colors.Yellow(snap.Status()),
 		colors.Magenta(snap.Author.DisplayName()),
 		snap.CreateTime.Format(timeLayout),
 		edited,
@@ -622,7 +622,7 @@ func (sb *showBug) setTitle(g *gocui.Gui, v *gocui.View) error {
 }
 
 func (sb *showBug) toggleOpenClose(g *gocui.Gui, v *gocui.View) error {
-	switch sb.bug.Snapshot().Status {
+	switch sb.bug.Snapshot().Status() {
 	case bug.OpenStatus:
 		_, err := sb.bug.Close()
 		return err