Error when loading templates from Python eggs

[Juan Manuel Caicedo <juan.manuel.caicedo@…>]

The template loader for using templates included in Python eggs (django.template.loaders.eggs.load_template_source) has a bug that raises an exception when a template, thus the loader never finds an existent template and always raises the TemplateDoesNotExist exception.

The bug was introduced in changeset [5609] (see the ​diff), when the decode function is called on the tuple that should be returned, not on the template string that is returned by the resource_string function.

If the template exists in a egg, instead of return it, an exception is raised with the following message:

  'tuple' object has no attribute 'decode'

I attach here a patch that calls the function correctly.

[Juan Manuel Caicedo <juan.manuel.caicedo@…>]

Patch for loading template files from Python eggs

[James Bennett]

Juan, this patch looks right to me, but before it gets marked ready for checkin, could you add some tests to make sure we catch this sort of regression in the future?

[Juan Manuel Caicedo <juan.manuel.caicedo@…>]

I updated the patch and now includes a test case for this bug, it also includes a minor fix in the template loader.

For the tests I used some mock objects for creating eggs with resources within them. The mock objects deal with some implementation details of the pkg_resources module that could make the test code difficult to read, but I didn't found another way to do this programmatically and without needing to have an specific Python egg installed on the machine that runs the test code.

Feel free to change the code or ask questions about it.

[Juan Manuel Caicedo <juan.manuel.caicedo@…>]

Updated patch for this bug, now includes a test case.

[Warren Smith]

I emailed Juan and he replied with:

<snip>
I think the patch is done and I'm not working anymore on it. Last
time I checked it worked with the trunk version.

I submitted a test case, I'm not completely satisfied with it but it
still works.
</snip>

Therefore, I'm changing this to "Ready for checkin" in hopes that it really is.

[Russell Keith-Magee]

(In [7565]) Fixed #6430 -- Corrected the loading of templates from eggs, which was broken by the unicode merge. Thanks for the excellent report and patch, Juan Manuel Caicedo <juan.manuel.caicedo@…>.