Add Snapshot::appendTimelineItem methode

Author: GlancingMind

bug/op_add_comment.go

@@ -30,22 +30,17 @@ func (op *AddCommentOperation) Id() entity.Id {
 func (op *AddCommentOperation) Apply(snapshot *Snapshot) {
 	snapshot.addActor(op.Author_)
 	snapshot.addParticipant(op.Author_)
-
 	comment := Comment{
 		id:       entity.CombineIds(snapshot.Id(), op.Id()),
 		Message:  op.Message,
 		Author:   op.Author_,
 		Files:    op.Files,
 		UnixTime: timestamp.Timestamp(op.UnixTime),
 	}
-
 	snapshot.appendComment(comment)
-
-	item := &AddCommentTimelineItem{
+	snapshot.appendTimelineItem(&AddCommentTimelineItem{
 		CommentTimelineItem: NewCommentTimelineItem(comment),
-	}
-
-	snapshot.timeline = append(snapshot.timeline, item)
+	})
 }
 
 func (op *AddCommentOperation) GetFiles() []repository.Hash {

bug/op_label_change.go

@@ -38,15 +38,13 @@ func (op *LabelChangeOperation) Apply(snapshot *Snapshot) {
 		snapshot.removeLabel(label)
 	}
 
-	item := &LabelChangeTimelineItem{
+	snapshot.appendTimelineItem(&LabelChangeTimelineItem{
 		id:       op.Id(),
 		Author:   op.Author_,
 		UnixTime: timestamp.Timestamp(op.UnixTime),
 		Added:    op.Added,
 		Removed:  op.Removed,
-	}
-
-	snapshot.timeline = append(snapshot.timeline, item)
+	})
 }
 
 func (op *LabelChangeOperation) Validate() error {

bug/op_set_status.go

@@ -25,15 +25,12 @@ func (op *SetStatusOperation) Id() entity.Id {
 func (op *SetStatusOperation) Apply(snapshot *Snapshot) {
 	snapshot.setStatusTo(op.Status)
 	snapshot.addActor(op.Author_)
-
-	item := &SetStatusTimelineItem{
+	snapshot.appendTimelineItem(&SetStatusTimelineItem{
 		id:       op.Id(),
 		Author:   op.Author_,
 		UnixTime: timestamp.Timestamp(op.UnixTime),
 		Status:   op.Status,
-	}
-
-	snapshot.timeline = append(snapshot.timeline, item)
+	})
 }
 
 func (op *SetStatusOperation) Validate() error {

bug/op_set_title.go

@@ -27,16 +27,13 @@ func (op *SetTitleOperation) Id() entity.Id {
 func (op *SetTitleOperation) Apply(snapshot *Snapshot) {
 	snapshot.changeTitleTo(op.Title)
 	snapshot.addActor(op.Author_)
-
-	item := &SetTitleTimelineItem{
+	snapshot.appendTimelineItem(&SetTitleTimelineItem{
 		id:       op.Id(),
 		Author:   op.Author_,
 		UnixTime: timestamp.Timestamp(op.UnixTime),
 		Title:    op.Title,
 		Was:      op.Was,
-	}
-
-	snapshot.timeline = append(snapshot.timeline, item)
+	})
 }
 
 func (op *SetTitleOperation) Validate() error {

bug/snapshot.go

@@ -165,6 +165,11 @@ func (snap *Snapshot) addParticipant(participant identity.Interface) {
 	snap.participants = append(snap.participants, participant)
 }
 
+// Append the supplied timelineitem to the snapshots timeline
+func (snap *Snapshot) appendTimelineItem(item TimelineItem) {
+	snap.timeline = append(snap.timeline, item)
+}
+
 // HasParticipant return true if the id is a participant
 func (snap *Snapshot) HasParticipant(id entity.Id) bool {
 	for _, p := range snap.participants {