From f6866f23a0215dd0d180ffb4f29999a338d4bdfc Mon Sep 17 00:00:00 2001
From: Deluan <github@deluan.com>
Date: Wed, 16 Mar 2016 20:45:08 -0400
Subject: [PATCH] Removed 'submit' parameter

---
 api/media_annotation.go  |  2 +-
 engine/scrobbler.go      | 10 ++++------
 engine/scrobbler_test.go |  4 ++--
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/api/media_annotation.go b/api/media_annotation.go
index 3bfdd4872..7e9c92a0d 100644
--- a/api/media_annotation.go
+++ b/api/media_annotation.go
@@ -24,7 +24,7 @@ func (c *MediaAnnotationController) Scrobble() {
 	time := c.ParamTime("time", time.Now())
 	submission := c.ParamBool("submission", false)
 	if submission {
-		mf, err := c.scrobbler.Register(id, time, true)
+		mf, err := c.scrobbler.Register(id, time)
 		if err != nil {
 			beego.Error("Error scrobbling:", err)
 			c.SendError(responses.ERROR_GENERIC, "Internal error")
diff --git a/engine/scrobbler.go b/engine/scrobbler.go
index fed7366f8..fb17d3a6b 100644
--- a/engine/scrobbler.go
+++ b/engine/scrobbler.go
@@ -10,7 +10,7 @@ import (
 )
 
 type Scrobbler interface {
-	Register(id string, playDate time.Time, submit bool) (*domain.MediaFile, error)
+	Register(id string, playDate time.Time) (*domain.MediaFile, error)
 }
 
 func NewScrobbler(itunes itunesbridge.ItunesControl, mr domain.MediaFileRepository, npr NowPlayingRepository) Scrobbler {
@@ -23,7 +23,7 @@ type scrobbler struct {
 	npRepo NowPlayingRepository
 }
 
-func (s scrobbler) Register(id string, playDate time.Time, submit bool) (*domain.MediaFile, error) {
+func (s scrobbler) Register(id string, playDate time.Time) (*domain.MediaFile, error) {
 	mf, err := s.mfRepo.Get(id)
 	if err != nil {
 		return nil, err
@@ -33,10 +33,8 @@ func (s scrobbler) Register(id string, playDate time.Time, submit bool) (*domain
 		return nil, errors.New(fmt.Sprintf(`Id "%s" not found`, id))
 	}
 
-	if submit {
-		if err := s.itunes.MarkAsPlayed(id, playDate); err != nil {
-			return nil, err
-		}
+	if err := s.itunes.MarkAsPlayed(id, playDate); err != nil {
+		return nil, err
 	}
 	return mf, nil
 }
diff --git a/engine/scrobbler_test.go b/engine/scrobbler_test.go
index 153c0b1f0..48017efe4 100644
--- a/engine/scrobbler_test.go
+++ b/engine/scrobbler_test.go
@@ -27,7 +27,7 @@ func TestScrobbler(t *testing.T) {
 
 		Convey("When I scrobble an existing song", func() {
 			now := time.Now()
-			mf, err := scrobbler.Register("2", now, true)
+			mf, err := scrobbler.Register("2", now)
 
 			Convey("Then I get the scrobbled song back", func() {
 				So(err, ShouldBeNil)
@@ -42,7 +42,7 @@ func TestScrobbler(t *testing.T) {
 		})
 
 		Convey("When the ID is not in the DB", func() {
-			_, err := scrobbler.Register("3", time.Now(), true)
+			_, err := scrobbler.Register("3", time.Now())
 
 			Convey("Then I receive an error", func() {
 				So(err, ShouldNotBeNil)