Codespilot / vocadb

Automatically exported from code.google.com/p/vocadb
2 stars 1 forks source link

Users are able to upload multiple pictures for albums and artists #58

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Should probably move the pictures out of the database

Original issue reported on code.google.com by rii...@gmail.com on 18 Apr 2012 at 11:04

GoogleCodeExporter commented 8 years ago
It would probably be useful to support adding pictures to songs as well, at 
least so that the default thumbnail provided by the video service could be 
replaced.

I've decided to save the additional pictures out of the database instead of 
saving them as BLOBs as pictures currently are. This is mainly to reduce the 
size of the database. I think that for now it's best to leave the main picture 
(cover, artist picture) management as is, perhaps move it to the same page 
where the additional pictures are. Later on it's easy to move main pictures out 
of the database as well, if it makes sense.

I'm unsure what to do about versioning of additional pictures. For now it seems 
that it's better to ignore that, meaning additional pictures won't be saved 
with archived entry versions. Saving all pictures every time one of them is 
changed wastes too much space. It would be possible to handle each picture 
individually so that they'd only be saved when new pictures are added, but 
that's more tricky to implement, and considering the pictures aren't 
informational data, versioning them isn't as important anyway. I'll add 
versioning to additional pictures later if needed. Main picture will be 
versioned as it is now.

Original comment by rii...@gmail.com on 3 Jul 2012 at 8:41

GoogleCodeExporter commented 8 years ago
Implemented for artists as well.

Original comment by rii...@gmail.com on 18 Jul 2012 at 6:03

GoogleCodeExporter commented 8 years ago

Original comment by rii...@gmail.com on 18 Jul 2012 at 9:13