Re: [Ids-devel] More patches (last of the albumIconName)

[John M. <mo...@mu...>]

Thanks.

I'm not sure about using the image cache for album preview images. I like
the fact that nothing valuable is kept in there so I can clear it out from
time to time. If a custom album icon had been created, it would be lost.

I've been wanting to move everything that isn't an original image (album
previews and comments) out of the albums directory. Many people have
complained about IDS requiring write access to the directories that contain
their original images. I've been putting the change off because it would
break people's current sites.

I think I'll get started on this. Unless someone has a better suggestion,
I'll call the new directory "album-data."

I wonder how hard it would be to write a program to convert from the old
layout to the new...

John


on 2/17/01 3:37 PM, Randy Maas at ra...@ac... wrote:

> Attacheed are three little patches that make the albumIconName be relative
> to the imageCache. This patch is separate for two reasons; one I clipped
> that part out of the previous patch to John*.  Two, this patch will
> probably break current settings for album icon images.  Album icon images,
> to the best of my understanding, are stored in the albums source
> directory, but don't risk overwriting a legitimate album picture because
> the icon's file name is prefixed with a '.'
> 
> I don't use files that start with a '.', so there is a risk that the
> icon filename may overwrite a real image.  The solution is to place it in
> the imagecache where there won't be a name collision.  (Images in the
> image-cache, have suffixes that distinguish them).
> 
> 
> (*) I edit patches by hand since my diffs are otherwise kinda big.
> 
> Now to download 6a4,
> Randy Maas
> 
>