search

biow0lf / evedev-kb

Automatically exported from code.google.com/p/evedev-kb
1 stars 0 forks source link

Simplify image caching using one store #112

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Store all cached alliance/corp/pilot images using the same scheme of 
img/12/34/56/12345678.png and handle this with the cachehandler class. Pass 
in the image name and it returns appropriate directory, creates hierarchy 
automatically, etc.

Pros: simpler code.
Cons: None that I see, since the IDs don't clash and the code becomes 
simpler. (Might be a lot of code wasted with the tyrannis changes, might fit 
in well. Since I already wrote most of the code needed it's not much effort 
at this point, though.)

Original issue reported on code.google.com by kovellia on 23 May 2010 at 2:46

GoogleCodeExporter commented 9 years ago 
Would still be nice if CCP's new image service makes this redundant.

Original comment by kovellia on 23 May 2010 at 3:02

GoogleCodeExporter commented 9 years ago 
Finished in r683.

Original comment by kovellia on 28 May 2010 at 1:32