fix(playlist): implement api according to new standard

pkg/api/api.go

@@ -174,12 +174,12 @@ func Register(r *macaron.Macaron) {
 
 
 		// Playlist
 		// Playlist
 		r.Group("/playlists", func() {
 		r.Group("/playlists", func() {
-			r.Get("/", SearchPlaylists)
-			r.Get("/:id", ValidateOrgPlaylist, GetPlaylist)
-			r.Get("/:id/dashboards", ValidateOrgPlaylist, GetPlaylistDashboards)
-			r.Delete("/:id", reqEditorRole, ValidateOrgPlaylist, DeletePlaylist)
-			r.Put("/:id", reqEditorRole, bind(m.UpdatePlaylistQuery{}), ValidateOrgPlaylist, UpdatePlaylist)
-			r.Post("/", reqEditorRole, bind(m.CreatePlaylistQuery{}), CreatePlaylist)
+			r.Get("/", wrap(SearchPlaylists))
+			r.Get("/:id", ValidateOrgPlaylist, wrap(GetPlaylist))
+			r.Get("/:id/dashboards", ValidateOrgPlaylist, wrap(GetPlaylistDashboards))
+			r.Delete("/:id", reqEditorRole, ValidateOrgPlaylist, wrap(DeletePlaylist))
+			r.Put("/:id", reqEditorRole, bind(m.UpdatePlaylistQuery{}), ValidateOrgPlaylist, wrap(UpdatePlaylist))
+			r.Post("/", reqEditorRole, bind(m.CreatePlaylistQuery{}), wrap(CreatePlaylist))
 		})
 		})
 
 
 		// Search
 		// Search

pkg/api/playlist.go

@@ -22,7 +22,7 @@ func ValidateOrgPlaylist(c *middleware.Context) {
 	}
 	}
 }
 }
 
 
-func SearchPlaylists(c *middleware.Context) {
+func SearchPlaylists(c *middleware.Context) Response {
 	query := c.Query("query")
 	query := c.Query("query")
 	limit := c.QueryInt("limit")
 	limit := c.QueryInt("limit")
 
 
@@ -38,66 +38,60 @@ func SearchPlaylists(c *middleware.Context) {
 
 
 	err := bus.Dispatch(&searchQuery)
 	err := bus.Dispatch(&searchQuery)
 	if err != nil {
 	if err != nil {
-		c.JsonApiErr(500, "Search failed", err)
-		return
+		return ApiError(500, "Search failed", err)
 	}
 	}
 
 
-	c.JSON(200, searchQuery.Result)
+	return Json(200, searchQuery.Result)
 }
 }
 
 
-func GetPlaylist(c *middleware.Context) {
+func GetPlaylist(c *middleware.Context) Response {
 	id := c.ParamsInt64(":id")
 	id := c.ParamsInt64(":id")
 	cmd := m.GetPlaylistByIdQuery{Id: id}
 	cmd := m.GetPlaylistByIdQuery{Id: id}
 
 
 	if err := bus.Dispatch(&cmd); err != nil {
 	if err := bus.Dispatch(&cmd); err != nil {
-		c.JsonApiErr(500, "Playlist not found", err)
-		return
+		return ApiError(500, "Playlist not found", err)
 	}
 	}
 
 
-	c.JSON(200, cmd.Result)
+	return Json(200, cmd.Result)
 }
 }
 
 
-func GetPlaylistDashboards(c *middleware.Context) {
+func GetPlaylistDashboards(c *middleware.Context) Response {
 	id := c.ParamsInt64(":id")
 	id := c.ParamsInt64(":id")
 
 
 	query := m.GetPlaylistDashboardsQuery{Id: id}
 	query := m.GetPlaylistDashboardsQuery{Id: id}
 	if err := bus.Dispatch(&query); err != nil {
 	if err := bus.Dispatch(&query); err != nil {
-		c.JsonApiErr(500, "Playlist not found", err)
-		return
+		return ApiError(500, "Playlist not found", err)
 	}
 	}
 
 
-	c.JSON(200, query.Result)
+	return Json(200, query.Result)
 }
 }
 
 
-func DeletePlaylist(c *middleware.Context) {
+func DeletePlaylist(c *middleware.Context) Response {
 	id := c.ParamsInt64(":id")
 	id := c.ParamsInt64(":id")
 
 
 	cmd := m.DeletePlaylistQuery{Id: id}
 	cmd := m.DeletePlaylistQuery{Id: id}
 	if err := bus.Dispatch(&cmd); err != nil {
 	if err := bus.Dispatch(&cmd); err != nil {
-		c.JsonApiErr(500, "Failed to delete playlist", err)
-		return
+		return ApiError(500, "Failed to delete playlist", err)
 	}
 	}
 
 
-	c.JSON(200, "")
+	return Json(200, "")
 }
 }
 
 
-func CreatePlaylist(c *middleware.Context, query m.CreatePlaylistQuery) {
+func CreatePlaylist(c *middleware.Context, query m.CreatePlaylistQuery) Response {
 	query.OrgId = c.OrgId
 	query.OrgId = c.OrgId
 	err := bus.Dispatch(&query)
 	err := bus.Dispatch(&query)
 	if err != nil {
 	if err != nil {
-		c.JsonApiErr(500, "Failed to create playlist", err)
-		return
+		return ApiError(500, "Failed to create playlist", err)
 	}
 	}
 
 
-	c.JSON(200, query.Result)
+	return Json(200, query.Result)
 }
 }
 
 
-func UpdatePlaylist(c *middleware.Context, query m.UpdatePlaylistQuery) {
+func UpdatePlaylist(c *middleware.Context, query m.UpdatePlaylistQuery) Response {
 	err := bus.Dispatch(&query)
 	err := bus.Dispatch(&query)
 	if err != nil {
 	if err != nil {
-		c.JsonApiErr(500, "Failed to save playlist", err)
-		return
+		return ApiError(500, "Failed to save playlist", err)
 	}
 	}
 
 
-	c.JSON(200, query.Result)
+	return Json(200, query.Result)
 }
 }