adjusted uri used by inheriting templates, caused lookup error. #607

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
@aodag
Contributor

aodag commented May 22, 2012

File "/home/aodag/.envs/pyramid/local/lib/python2.7/site-packages/Mako-0.7.0-py2.7.egg/mako/runtime.py", line 732, in _lookup_template
raise exceptions.TemplateLookupException(str(e))
TemplateLookupException: Cant locate template for uri u'demo$templates/layout.mak'

Template Uri shourd be original uri, when loading that.

@aodag

This comment has been minimized.

Show comment
Hide comment
@aodag

aodag May 22, 2012

Contributor

I found that bug in pyramid 1.3.2.
On pyramid 1.3, template inheritance works fine.

Contributor

aodag commented May 22, 2012

I found that bug in pyramid 1.3.2.
On pyramid 1.3, template inheritance works fine.

@aodag

This comment has been minimized.

Show comment
Hide comment
@aodag

aodag May 22, 2012

Contributor

related issues #606, #512

Contributor

aodag commented May 22, 2012

related issues #606, #512

@mcdonc

This comment has been minimized.

Show comment
Hide comment
@mcdonc

mcdonc Jun 13, 2012

Member

Is there any way to provide a test which fails without the fix?

Member

mcdonc commented Jun 13, 2012

Is there any way to provide a test which fails without the fix?

@aodag

This comment has been minimized.

Show comment
Hide comment
@aodag

aodag Jun 14, 2012

Contributor

I gonna add tests at this week end.

Contributor

aodag commented Jun 14, 2012

I gonna add tests at this week end.

@aodag

This comment has been minimized.

Show comment
Hide comment
@aodag

aodag Jun 23, 2012

Contributor

I tested on windows, and fix that.

tested and passed

  • Ubuntu 12.04 64bit, python2.7.3
  • Ubuntu 12.04 64bit, python3.2.3
  • Windows7 64bit, python 2.7.2 64bit
Contributor

aodag commented Jun 23, 2012

I tested on windows, and fix that.

tested and passed

  • Ubuntu 12.04 64bit, python2.7.3
  • Ubuntu 12.04 64bit, python3.2.3
  • Windows7 64bit, python 2.7.2 64bit
@mcdonc

This comment has been minimized.

Show comment
Hide comment
@mcdonc

mcdonc Sep 15, 2012

Member

This is fixed in master.

Member

mcdonc commented Sep 15, 2012

This is fixed in master.

@mcdonc mcdonc closed this Sep 15, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment