@kovid,
I need to dowload a cover image to replace the default library and I still want the rest of the cover layout. I looked at the source of BasicRecipe to see if I could hook into the existing code, but could not find a way hence.
Reviewing now the current/resulting code, could indeed the _download_cover_img() be replaced with a call to _dowload_cover, renaming the file and doing the extra work.
I started out with modification more deeply in the code, but the resulting code can indeed be more elegant. Thanks for asking the question/making the remark. I'll make the adjustments later.
PS: if default_cover could take a variable for the default_image, that part of the code duplication could also be eliminated
Last edited by hansd; 04-12-2010 at 01:17 AM.
|