Hi,

The Image shown for an album on the album rearrange page is not the cover image.
This is not a major issue but it is harder te recognize the album.

Regards,

Hans

Comments

jhmnieuwenhuis created an issue. See original summary.

jhmnieuwenhuis’s picture

jhmnieuwenhuis commented

I checked again.
It works as it should.
Probably i did not clear the cache after changing the cover image for an album.

Issue can be closed.

Regards,
Hans

nathaniel’s picture

nathaniel commented
Status: Active » Fixed

Sounds good. If you notice it doesn't update consistently somewhere, please let me know. It should clear automatically.

jhmnieuwenhuis’s picture

jhmnieuwenhuis commented

Whenever i change the cover image of an album i have to rebuild cache when i show the list of albums.
Otherwise the old cover is shown.

Regards,
Hans

nathaniel’s picture

nathaniel commented
Status: Fixed » Active

This has something to do with cache tags and views. Adding "photos_image_list" to Cache::invalidateTags when set to cover is clicked should do the trick for now. I would like something more efficient than that, but I don't see how to get around it at the moment.

  • Nathaniel committed 8910d0b on 8.x-5.x 
    Issue #3129052 by Nathaniel: Image show for album on rearrange page is...
jhmnieuwenhuis’s picture

jhmnieuwenhuis commented

That works !

Better then a complete cash rebuild.

I have the same issue when reordering the albums :

When i go to : photos/user/2/album-rearrange and change the order and then go to /photos/user/2/album
the order is still the old order, until i rebuild cache.
Same if i go to my own album list view.

And during the testing i somehow ended up with two albums who both had a weight of 0.
I think that two albums with the same weight should not be possible. If that occurs, a complete reorder should be done.

Regards,
Hans

nathaniel’s picture

nathaniel commented
Title: Image show for album on rearrange page is not the cover » Cache tags for album rearrange, cover and views

  • Nathaniel committed 5c8b180 on 8.x-5.x 
    Issue #3129052 by Nathaniel: Cache tags for album rearrange, cover and...
nathaniel’s picture

nathaniel commented
Category: Support request » Bug report
Status: Active » Fixed

I think this is fixed now, please re-open or start a new issue if you notice any others. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.