[GH-ISSUE #40] How does Koel get album / artist covers? #28

New issue

Closed

opened 2026-02-26 02:31:51 +03:00 by kerem · 4 comments

kerem commented

2026-02-26 02:31:51 +03:00

Owner

Originally created by @askoxyz on GitHub (Dec 14, 2015).
Original GitHub issue: https://github.com/koel/koel/issues/40

So far it has managed to get no covers for any albums and/or artists I have, while they do show up in any other media player, so hence my question, how does Koel get the album covers, or how does it fetch them? from the song meta data or from a 3rd party (e.g last.fm or something)?

Originally created by @askoxyz on GitHub (Dec 14, 2015). Original GitHub issue: https://github.com/koel/koel/issues/40 So far it has managed to get no covers for any albums and/or artists I have, while they do show up in any other media player, so hence my question, how does Koel get the album covers, or how does it fetch them? from the song meta data or from a 3rd party (e.g last.fm or something)?

kerem closed this issue

2026-02-26 02:31:51 +03:00

kerem commented

2026-02-26 02:31:51 +03:00

Author

Owner

@askoxyz commented on GitHub (Dec 14, 2015):

Ok so I see it does all of this in Album.php#L68, but doesn't seem to actually work tough. Any thoughts?

@askoxyz commented on GitHub (Dec 14, 2015): Ok so I see it does all of this in [Album.php#L68](https://github.com/phanan/koel/blob/master/app/Models/Album.php#L68), but doesn't seem to actually work tough. Any thoughts?

kerem commented

2026-02-26 02:31:51 +03:00

Author

Owner

@cgrossde commented on GitHub (Dec 14, 2015):

I also had some issues here: When importing Koel tries to put the images in /img/covers, a folder that does not exist at the time. When accessing the covers it looks at /public/img/covers. The problem is, that this Laravel project is setup to have the index.php and .htaccess in / instead of /public. Therefore Laravel thinks the public_path() is / and not /public.

Quick fix: Change link 72 in Album.php to

$coverPath = public_path().'/public/img/covers/'.$fileName;

Clear the DB and reimport your music.

Better solution:

Move index.php and .htaccess to /public
Adapt templates and remove "public" path segment since public is the new root
Adapt your web servers docroot to the public folder and add the root folder to your include_path and open_basedir
Reimport your music

@cgrossde commented on GitHub (Dec 14, 2015): I also had some issues here: When importing Koel tries to put the images in `/img/covers`, a folder that does not exist at the time. When accessing the covers it looks at `/public/img/covers`. The problem is, that this Laravel project is setup to have the `index.php` and `.htaccess` in `/` instead of `/public`. Therefore Laravel thinks the `public_path()` is `/` and not `/public`. **Quick fix:** Change link [72](https://github.com/phanan/koel/blob/master/app/Models/Album.php#L72) in [Album.php](https://github.com/phanan/koel/blob/master/app/Models/Album.php#L72) to ``` PHP $coverPath = public_path().'/public/img/covers/'.$fileName; ``` Clear the DB and reimport your music. **Better solution:** - Move `index.php` and `.htaccess` to `/public` - Adapt templates and remove "public" path segment since public is the new root - Adapt your web servers docroot to the public folder and add the root folder to your include_path and open_basedir - Reimport your music

kerem commented

2026-02-26 02:31:51 +03:00

Author

Owner

@askoxyz commented on GitHub (Dec 14, 2015):

Yup. Went with the quick fix on this one and worked brilliantly. I do understand the use case for storing index and .htaccess in / instead of /public, for example shared hosts which do not allow anything up from public_html, but then public_path() should also be changed accordingly from the get-go. Either way, thank you @cgrossde .

@askoxyz commented on GitHub (Dec 14, 2015): Yup. Went with the quick fix on this one and worked brilliantly. I do understand the use case for storing index and .htaccess in `/` instead of `/public`, for example shared hosts which do not allow anything up from `public_html`, but then public_path() should also be changed accordingly from the get-go. Either way, thank you @cgrossde .

kerem commented

2026-02-26 02:31:52 +03:00

Author

Owner

@phanan commented on GitHub (Dec 14, 2015):

Thanks. Let's discuss in the PR.

@phanan commented on GitHub (Dec 14, 2015): Thanks. Let's discuss in the PR.

kerem referenced this issue

2026-02-26 02:31:52 +03:00

[GH-ISSUE #39] Importing artists with a & in their name #30

kerem referenced this issue

2026-02-26 02:33:03 +03:00