cleanup

models/list_create_update.go

@@ -0,0 +1,22 @@
+package models
+
+// CreateOrUpdateList updates a list or creates it if it doesn't exist
+func CreateOrUpdateList(list *List) (err error) {
+	// Check if it exists
+	_, err = GetListByID(list.ID)
+	if err != nil {
+		return
+	}
+
+	list.OwnerID = list.Owner.ID
+
+	if list.ID == 0 {
+		_, err = x.Insert(list)
+	} else {
+		_, err = x.ID(list.ID).Update(list)
+		return
+	}
+
+	return
+
+}

models/lists.go

@@ -34,7 +34,7 @@ func GetListByID(id int64) (list List, err error) {
 	return list, nil
 }
 
-//
+// GetListsByUser gets all lists a user owns
 func GetListsByUser(user *User) (lists []*List, err error) {
 	fullUser, _, err := GetUserByID(user.ID)
 	if err != nil {
@@ -52,24 +52,3 @@ func GetListsByUser(user *User) (lists []*List, err error) {
 
 	return
 }
-
-// CreateOrUpdateList updates a list or creates it if it doesn't exist
-func CreateOrUpdateList(list *List) (err error) {
-	// Check if it exists
-	_, err = GetListByID(list.ID)
-	if err != nil {
-		return
-	}
-
-	list.OwnerID = list.Owner.ID
-
-	if list.ID == 0 {
-		_, err = x.Insert(list)
-	} else {
-		_, err = x.ID(list.ID).Update(list)
-		return
-	}
-
-	return
-
-}

routes/api/v1/lists_list.go

@@ -1,11 +1,12 @@
 package v1
 
 import (
-	"github.com/labstack/echo"
 	"git.kolaente.de/konrad/list/models"
+	"github.com/labstack/echo"
 	"net/http"
 )
 
+// GetListsByUser gets all lists a user owns
 func GetListsByUser(c echo.Context) error {
 
 	currentUser, err := models.GetCurrentUser(c)
@@ -19,4 +20,4 @@ func GetListsByUser(c echo.Context) error {
 	}
 
 	return c.JSON(http.StatusOK, allLists)
-}
+}