Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embedded lyrics missing when using "shuffle all" #175

Closed
jixunmoe opened this issue Jul 8, 2023 · 2 comments
Closed

Embedded lyrics missing when using "shuffle all" #175

jixunmoe opened this issue Jul 8, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@jixunmoe
Copy link

jixunmoe commented Jul 8, 2023

Expected Behavior

Lyrics been fetched and shown.

Current Behavior

Lyrics missing when using "shuffle all".

Steps to Reproduce

  1. Create a navidrome instance where all songs contained LRC lyrics embedded to the media and indexed.
  2. "Shuffle all" from the interface.
  3. No lyrics available when switched to the Lyrics tab.

Possible Solution (Not obligatory)

Workaround: play from tracks section and manually shuffle from there.

Context

The old client (sonixd) is able to display the lyrics text content as it is.

When requesting the random api (/rest/getRandomSongs.view), the response does not include lyrics:

{
  "subsonic-response": {
    "status": "ok",
    "version": "1.16.1",
    "type": "navidrome",
    "serverVersion": "0.49.3 (8b93962f)",
    "randomSongs": {
      "song": [
        {
          "id": "5bbb499925c655b4147298e40785dc93",
          "parent": "61c01b1ddfe2f4872fc39888a1793e0c",
          "isDir": false,
          "title": "玫瑰少年",
          "album": "UGLY BEAUTY",
          "artist": "蔡依林",
          "track": 2,
          "year": 2018,
          "coverArt": "mf-5bbb499925c655b4147298e40785dc93_64921e97",
          "size": 4879472,
          "contentType": "audio/mp4",
          "suffix": "m4a",
          "duration": 191,
          "bitRate": 200,
          "path": "蔡依林/UGLY BEAUTY/02 - 玫瑰少年.m4a",
          "playCount": 3,
          "played": "2023-07-04T22:28:05Z",
          "discNumber": 1,
          "created": "2023-06-20T22:06:02.855677599Z",
          "albumId": "61c01b1ddfe2f4872fc39888a1793e0c",
          "artistId": "060545170130c81c5505fda49b58d018",
          "type": "music",
          "isVideo": false
        }
      ]
    }
  }
}

When playing directly from the tracks interface, it makes a request to song /api/song which had lyrics populated.

The same issue can also be observed (not always!) from the album or search page, as those filtering endpoints does not provide lyrics field neither.

Your Environment

@jixunmoe jixunmoe added the bug Something isn't working label Jul 8, 2023
@kgarner7
Copy link
Collaborator

kgarner7 commented Feb 8, 2024

When you upgrade to Navidrome 0.51 (or later releases), this will be resolved by #484. This issue (as you noted) comes up because the getRandomSongs doesn't return a full Navidrome track object.

@kgarner7
Copy link
Collaborator

I'm going to close this, as it is resolved using Navidrome 0.51 and development Feishin 83d5fee (no longer relying on the song lyrics property)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants