summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLars-Dominik Braun <lars@6xq.net>2012-06-26 18:17:54 +0200
committerLars-Dominik Braun <lars@6xq.net>2012-06-26 18:20:18 +0200
commit845cf4c1bdae57e8ed59bc3bb6fcaa7464c8904e (patch)
treec2e5d52bbd8899ed001585abae2c336fd06d14da
parentf2d6dc360a7385dd490832bf9fb391addcb250a0 (diff)
downloadpianobar-845cf4c1bdae57e8ed59bc3bb6fcaa7464c8904e.tar.gz
pianobar-845cf4c1bdae57e8ed59bc3bb6fcaa7464c8904e.tar.bz2
pianobar-845cf4c1bdae57e8ed59bc3bb6fcaa7464c8904e.zip
piano: Remove move song
Doesn’t work with JSON API. See #280.
-rw-r--r--src/libpiano/piano.h8
-rw-r--r--src/libpiano/request.c36
-rw-r--r--src/libpiano/response.c15
-rw-r--r--src/settings.h29
-rw-r--r--src/ui_act.c37
-rw-r--r--src/ui_act.h1
-rw-r--r--src/ui_dispatch.h2
7 files changed, 14 insertions, 114 deletions
diff --git a/src/libpiano/piano.h b/src/libpiano/piano.h
index 167f0f9..e881621 100644
--- a/src/libpiano/piano.h
+++ b/src/libpiano/piano.h
@@ -147,7 +147,6 @@ typedef enum {
PIANO_REQUEST_GET_PLAYLIST = 3,
PIANO_REQUEST_RATE_SONG = 4,
PIANO_REQUEST_ADD_FEEDBACK = 5,
- PIANO_REQUEST_MOVE_SONG = 6,
PIANO_REQUEST_RENAME_STATION = 7,
PIANO_REQUEST_DELETE_STATION = 8,
PIANO_REQUEST_SEARCH = 9,
@@ -199,13 +198,6 @@ typedef struct {
} PianoRequestDataAddFeedback_t;
typedef struct {
- PianoSong_t *song;
- PianoStation_t *from;
- PianoStation_t *to;
- unsigned short step;
-} PianoRequestDataMoveSong_t;
-
-typedef struct {
PianoStation_t *station;
char *newName;
} PianoRequestDataRenameStation_t;
diff --git a/src/libpiano/request.c b/src/libpiano/request.c
index 5619e1b..cb1b30a 100644
--- a/src/libpiano/request.c
+++ b/src/libpiano/request.c
@@ -403,42 +403,6 @@ PianoReturn_t PianoRequest (PianoHandle_t *ph, PianoRequest_t *req,
goto cleanup;
break;
}
-
- case PIANO_REQUEST_MOVE_SONG: {
- /* move song to a different station, needs two requests */
- PianoRequestDataMoveSong_t *reqData = req->data;
- PianoRequestDataAddFeedback_t transformedReqData;
-
- assert (reqData != NULL);
- assert (reqData->song != NULL);
- assert (reqData->from != NULL);
- assert (reqData->to != NULL);
- assert (reqData->step < 2);
-
- transformedReqData.trackToken = reqData->song->trackToken;
- req->data = &transformedReqData;
-
- switch (reqData->step) {
- case 0:
- transformedReqData.stationId = reqData->from->id;
- transformedReqData.rating = PIANO_RATE_BAN;
- break;
-
- case 1:
- transformedReqData.stationId = reqData->to->id;
- transformedReqData.rating = PIANO_RATE_LOVE;
- break;
- }
-
- /* create request data (url, post data) */
- ret = PianoRequest (ph, req, PIANO_REQUEST_ADD_FEEDBACK);
- /* and reset request type/data */
- req->type = PIANO_REQUEST_MOVE_SONG;
- req->data = reqData;
-
- goto cleanup;
- break;
- }
}
/* standard parameter */
diff --git a/src/libpiano/response.c b/src/libpiano/response.c
index 659ed2d..1b63bee 100644
--- a/src/libpiano/response.c
+++ b/src/libpiano/response.c
@@ -334,21 +334,6 @@ PianoReturn_t PianoResponse (PianoHandle_t *ph, PianoRequest_t *req) {
break;
}
- case PIANO_REQUEST_MOVE_SONG: {
- /* move song to different station */
- PianoRequestDataMoveSong_t *reqData = req->data;
-
- assert (req->responseData != NULL);
- assert (reqData != NULL);
- assert (reqData->step < 2);
-
- if (reqData->step == 0) {
- ret = PIANO_RET_CONTINUE_REQUEST;
- ++reqData->step;
- }
- break;
- }
-
case PIANO_REQUEST_DELETE_STATION: {
/* delete station from server and station list */
PianoStation_t *station = req->data;
diff --git a/src/settings.h b/src/settings.h
index a466b72..59dd7b6 100644
--- a/src/settings.h
+++ b/src/settings.h
@@ -42,22 +42,21 @@ typedef enum {
BAR_KS_HISTORY = 8,
BAR_KS_INFO = 9,
BAR_KS_ADDSHARED = 10,
- BAR_KS_MOVESONG = 11,
- BAR_KS_SKIP = 12,
- BAR_KS_PLAYPAUSE = 13,
- BAR_KS_QUIT = 14,
- BAR_KS_RENAMESTATION = 15,
- BAR_KS_SELECTSTATION = 16,
- BAR_KS_TIRED = 17,
- BAR_KS_UPCOMING = 18,
- BAR_KS_SELECTQUICKMIX = 19,
- BAR_KS_DEBUG = 20,
- BAR_KS_BOOKMARK = 21,
- BAR_KS_VOLDOWN = 22,
- BAR_KS_VOLUP = 23,
- BAR_KS_MANAGESTATION = 24,
+ BAR_KS_SKIP = 11,
+ BAR_KS_PLAYPAUSE = 12,
+ BAR_KS_QUIT = 13,
+ BAR_KS_RENAMESTATION = 14,
+ BAR_KS_SELECTSTATION = 15,
+ BAR_KS_TIRED = 16,
+ BAR_KS_UPCOMING = 17,
+ BAR_KS_SELECTQUICKMIX = 18,
+ BAR_KS_DEBUG = 19,
+ BAR_KS_BOOKMARK = 20,
+ BAR_KS_VOLDOWN = 21,
+ BAR_KS_VOLUP = 22,
+ BAR_KS_MANAGESTATION = 23,
/* insert new shortcuts _before_ this element and increase its value */
- BAR_KS_COUNT = 25,
+ BAR_KS_COUNT = 24,
} BarKeyShortcutId_t;
#define BAR_KS_DISABLED '\x00'
diff --git a/src/ui_act.c b/src/ui_act.c
index ea50ec7..298180c 100644
--- a/src/ui_act.c
+++ b/src/ui_act.c
@@ -300,43 +300,6 @@ BarUiActCallback(BarUiActSkipSong) {
BarUiDoSkipSong (&app->player);
}
-/* move song to different station
- */
-BarUiActCallback(BarUiActMoveSong) {
- PianoReturn_t pRet;
- WaitressReturn_t wRet;
- PianoRequestDataMoveSong_t reqData;
-
- assert (selSong != NULL);
-
- reqData.step = 0;
-
- reqData.to = BarUiSelectStation (app, app->ph.stations,
- "Move song to station: ", NULL, false);
- if (reqData.to != NULL) {
- /* find original station (just is case we're playing a quickmix
- * station) */
- reqData.from = PianoFindStationById (app->ph.stations,
- selSong->stationId);
- if (reqData.from == NULL) {
- BarUiMsg (&app->settings, MSG_ERR, "Station not found\n");
- return;
- }
-
- if (!BarTransformIfShared (app, reqData.from) ||
- !BarTransformIfShared (app, reqData.to)) {
- return;
- }
- BarUiMsg (&app->settings, MSG_INFO, "Moving song to \"%s\"... ", reqData.to->name);
- reqData.song = selSong;
- if (BarUiActDefaultPianoCall (PIANO_REQUEST_MOVE_SONG, &reqData) &&
- selSong == app->playlist) {
- BarUiDoSkipSong (&app->player);
- }
- BarUiActDefaultEventcmd ("songmove");
- }
-}
-
/* pause
*/
BarUiActCallback(BarUiActPause) {
diff --git a/src/ui_act.h b/src/ui_act.h
index 4173b75..02cb54f 100644
--- a/src/ui_act.h
+++ b/src/ui_act.h
@@ -44,7 +44,6 @@ BarUiActCallback(BarUiActStationFromGenre);
BarUiActCallback(BarUiActSongInfo);
BarUiActCallback(BarUiActLoveSong);
BarUiActCallback(BarUiActSkipSong);
-BarUiActCallback(BarUiActMoveSong);
BarUiActCallback(BarUiActPause);
BarUiActCallback(BarUiActRenameStation);
BarUiActCallback(BarUiActSelectStation);
diff --git a/src/ui_dispatch.h b/src/ui_dispatch.h
index e8c9944..acd0f67 100644
--- a/src/ui_dispatch.h
+++ b/src/ui_dispatch.h
@@ -69,8 +69,6 @@ static const BarUiDispatchAction_t dispatchActions[BAR_KS_COUNT] = {
"print information about song/station", "act_songinfo"},
{'j', BAR_DC_GLOBAL, BarUiActAddSharedStation, "add shared station",
"act_addshared"},
- {'m', BAR_DC_SONG, BarUiActMoveSong, "move song to different station",
- "act_songmove"},
{'n', BAR_DC_GLOBAL | BAR_DC_STATION, BarUiActSkipSong, "next song",
"act_songnext"},
{'p', BAR_DC_GLOBAL | BAR_DC_STATION, BarUiActPause, "pause/continue",