On Wed, 30 May 2012, Theodoros Theodoropoulos wrote:
> The solution seems dead simple:
>
> In line #127 of bfe_fulltext.py,
>
> out += " ".join(url_list) + additional_str + versions_str + separator
> must move a few spaces, to be aligned with "for url, url_format in
> urls_and_format" or, else it gets executed only for the last filename
> in the dict.
>
> Can you verify this?

I checked and it seems that one indent to the right is indeed missing,
otherwise the previous iterations simply get forgotten.  Thanks for the
bug report and the analysis; I'll commit the change in your name.

But the code there is not very pretty WRT urls_dict and url_list
resetting inside the loop... I guess we'd better re-factor that part.
Sam, since it was you who touched this part the last (in ae82663e), 
can you please have a go?

Best regards
-- 
Tibor Simko

Reply via email to