Remove unused variables

src/components/syncPlay/core/Manager.js

@@ -130,7 +130,7 @@ class Manager {
     /**
      * Called when the player changes.
      */
-    onPlayerChange(newPlayer, newTarget, oldPlayer) {
+    onPlayerChange(newPlayer) {
         this.bindToPlayer(newPlayer);
     }
 
@@ -228,9 +228,8 @@ class Manager {
     /**
      * Handles a playback command from the server.
      * @param {Object} cmd The playback command.
-     * @param {Object} apiClient The ApiClient.
      */
-    processCommand(cmd, apiClient) {
+    processCommand(cmd) {
         if (cmd === null) return;
 
         if (typeof cmd.When === 'string') {
@@ -277,9 +276,8 @@ class Manager {
     /**
      * Handles a group state change.
      * @param {Object} update The group state update.
-     * @param {Object} apiClient The ApiClient.
      */
-    processStateChange(update, apiClient) {
+    processStateChange(update) {
         if (update === null || update.State === null || update.Reason === null) return;
 
         if (!this.isSyncPlayEnabled()) {

src/components/syncPlay/core/players/GenericPlayer.js

@@ -158,6 +158,7 @@ class GenericPlayer {
      * Sets the playback rate, if supported.
      * @param {number} value The playback rate.
      */
+    // eslint-disable-next-line no-unused-vars
     setPlaybackRate(value) {
         // Do nothing.
     }
@@ -196,6 +197,7 @@ class GenericPlayer {
      * Seeks the player to the specified position.
      * @param {number} positionTicks The new position.
      */
+    // eslint-disable-next-line no-unused-vars
     localSeek(positionTicks) {
 
     }
@@ -211,6 +213,7 @@ class GenericPlayer {
      * Sends a command to the player.
      * @param {Object} command The command.
      */
+    // eslint-disable-next-line no-unused-vars
     localSendCommand(command) {
 
     }
@@ -219,6 +222,7 @@ class GenericPlayer {
      * Starts playback.
      * @param {Object} options Playback data.
      */
+    // eslint-disable-next-line no-unused-vars
     localPlay(options) {
 
     }
@@ -227,6 +231,7 @@ class GenericPlayer {
      * Sets playing item from playlist.
      * @param {string} playlistItemId The item to play.
      */
+    // eslint-disable-next-line no-unused-vars
     localSetCurrentPlaylistItem(playlistItemId) {
 
     }
@@ -235,6 +240,7 @@ class GenericPlayer {
      * Removes items from playlist.
      * @param {Array} playlistItemIds The items to remove.
      */
+    // eslint-disable-next-line no-unused-vars
     localRemoveFromPlaylist(playlistItemIds) {
 
     }
@@ -244,6 +250,7 @@ class GenericPlayer {
      * @param {string} playlistItemId The item to move.
      * @param {number} newIndex The new position.
      */
+    // eslint-disable-next-line no-unused-vars
     localMovePlaylistItem(playlistItemId, newIndex) {
 
     }
@@ -252,6 +259,7 @@ class GenericPlayer {
      * Queues in the playlist.
      * @param {Object} options Queue data.
      */
+    // eslint-disable-next-line no-unused-vars
     localQueue(options) {
 
     }
@@ -260,6 +268,7 @@ class GenericPlayer {
      * Queues after the playing item in the playlist.
      * @param {Object} options Queue data.
      */
+    // eslint-disable-next-line no-unused-vars
     localQueueNext(options) {
 
     }
@@ -282,6 +291,7 @@ class GenericPlayer {
      * Sets repeat mode.
      * @param {string} value The repeat mode.
      */
+    // eslint-disable-next-line no-unused-vars
     localSetRepeatMode(value) {
 
     }
@@ -290,6 +300,7 @@ class GenericPlayer {
      * Sets shuffle mode.
      * @param {string} value The shuffle mode.
      */
+    // eslint-disable-next-line no-unused-vars
     localSetQueueShuffleMode(value) {
 
     }

src/components/syncPlay/ui/players/NoActivePlayer.js

@@ -135,7 +135,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's seek method.
      */
-    seekRequest(positionTicks, player) {
+    seekRequest(positionTicks) {
         const controller = syncPlayManager.getController();
         controller.seek(positionTicks);
     }
@@ -148,19 +148,19 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
         const controller = syncPlayManager.getController();
         const playerWrapper = syncPlayManager.getPlayerWrapper();
 
-        const defaultAction = (_command, _player) => {
+        const defaultAction = (_command) => {
             playerWrapper.localSendCommand(_command);
         };
 
-        const ignoreCallback = (_command, _player) => {
+        const ignoreCallback = () => {
             // Do nothing.
         };
 
-        const SetRepeatModeCallback = (_command, _player) => {
+        const SetRepeatModeCallback = (_command) => {
             controller.setRepeatMode(_command.Arguments.RepeatMode);
         };
 
-        const SetShuffleQueueCallback = (_command, _player) => {
+        const SetShuffleQueueCallback = (_command) => {
             controller.setShuffleMode(_command.Arguments.ShuffleMode);
         };
 
@@ -242,7 +242,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's setCurrentPlaylistItem method.
      */
-    setCurrentPlaylistItemRequest(playlistItemId, player) {
+    setCurrentPlaylistItemRequest(playlistItemId) {
         const controller = syncPlayManager.getController();
         controller.setCurrentPlaylistItem(playlistItemId);
     }
@@ -250,7 +250,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's removeFromPlaylist method.
      */
-    removeFromPlaylistRequest(playlistItemIds, player) {
+    removeFromPlaylistRequest(playlistItemIds) {
         const controller = syncPlayManager.getController();
         controller.removeFromPlaylist(playlistItemIds);
     }
@@ -258,7 +258,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's movePlaylistItem method.
      */
-    movePlaylistItemRequest(playlistItemId, newIndex, player) {
+    movePlaylistItemRequest(playlistItemId, newIndex) {
         const controller = syncPlayManager.getController();
         controller.movePlaylistItem(playlistItemId, newIndex);
     }
@@ -266,7 +266,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's queue method.
      */
-    queueRequest(options, player) {
+    queueRequest(options) {
         const controller = syncPlayManager.getController();
         controller.queue(options);
     }
@@ -274,7 +274,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's queueNext method.
      */
-    queueNextRequest(options, player) {
+    queueNextRequest(options) {
         const controller = syncPlayManager.getController();
         controller.queueNext(options);
     }
@@ -282,7 +282,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's nextTrack method.
      */
-    nextTrackRequest(player) {
+    nextTrackRequest() {
         const controller = syncPlayManager.getController();
         controller.nextItem();
     }
@@ -290,7 +290,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's previousTrack method.
      */
-    previousTrackRequest(player) {
+    previousTrackRequest() {
         const controller = syncPlayManager.getController();
         controller.previousItem();
     }
@@ -298,7 +298,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's setRepeatMode method.
      */
-    setRepeatModeRequest(mode, player) {
+    setRepeatModeRequest(mode) {
         const controller = syncPlayManager.getController();
         controller.setRepeatMode(mode);
     }
@@ -306,7 +306,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's setQueueShuffleMode method.
      */
-    setQueueShuffleModeRequest(mode, player) {
+    setQueueShuffleModeRequest(mode) {
         const controller = syncPlayManager.getController();
         controller.setShuffleMode(mode);
     }
@@ -314,7 +314,7 @@ class NoActivePlayer extends SyncPlay.Players.GenericPlayer {
     /**
      * Overrides PlaybackManager's toggleQueueShuffleMode method.
      */
-    toggleQueueShuffleModeRequest(player) {
+    toggleQueueShuffleModeRequest() {
         const controller = syncPlayManager.getController();
         controller.toggleShuffleMode();
     }

src/components/syncPlay/ui/players/QueueManager.js

@@ -21,14 +21,14 @@ class QueueManager {
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    setPlaylist(items) {
+    setPlaylist() {
         // Do nothing.
     }
 
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    queue(items) {
+    queue() {
         // Do nothing.
     }
 
@@ -49,14 +49,14 @@ class QueueManager {
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    clearPlaylist(clearCurrentItem = false) {
+    clearPlaylist() {
         // Do nothing.
     }
 
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    queueNext(items) {
+    queueNext() {
         // Do nothing.
     }
 
@@ -90,28 +90,28 @@ class QueueManager {
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    setPlaylistState(playlistItemId, playlistIndex) {
+    setPlaylistState() {
         // Do nothing.
     }
 
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    setPlaylistIndex(playlistIndex) {
+    setPlaylistIndex() {
         // Do nothing.
     }
 
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    removeFromPlaylist(playlistItemIds) {
+    removeFromPlaylist() {
         // Do nothing.
     }
 
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    movePlaylistItem(playlistItemId, newIndex) {
+    movePlaylistItem() {
         // Do nothing.
         return {
             result: 'noop'
@@ -128,7 +128,7 @@ class QueueManager {
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    setRepeatMode(value) {
+    setRepeatMode() {
         // Do nothing.
     }
 
@@ -142,7 +142,7 @@ class QueueManager {
     /**
      * Placeholder for original PlayQueueManager method.
      */
-    setShuffleMode(value) {
+    setShuffleMode() {
         // Do nothing.
     }