{% block %} inheritance not working after a few levels of {% extends %} (includes minimal demo app)

Why can't I attach my file to the ticket?

Oh well, it's definetly a buggy day for Django, when I try to attach a file I get a pgsql error. Well, until I can attach my file, you can get it there:

http://s3.amazonaws.com:80/goombah/extends_bug.zip

You cannot attach .zip or .tar.gz files here; the preferred format is a single .diff.

Found the problem, it's related to the following code in source:django/trunk/django/template/loader_tags.py#L64:

class ExtendsNode(Node):
    ...
    def render(self, context):
        compiled_parent = self.get_parent(context)
        parent_is_child = isinstance(compiled_parent.nodelist[0], ExtendsNode)
        parent_blocks = dict([(n.name, n) for n in compiled_parent.nodelist.get_nodes_by_type(BlockNode)])

parent_is_child is set to False, when nodelist[0] node is {% load "..." %} block, and mystics occurred. I think correct solution should search for existance of ExtendsNode? in nodelist, instead of picking up nodelist[0]. Continuing investigations to be sure my proposed change won't break anything.

Core devs, please decide what to do:

• throw error that ExtendsNode? should always be the first node (i think there was one)

or

• search ExtendsNode? through blocks

Patches for both decisions are very small.

Decision by Malcolm: throw error that ExtendsNode?? should always be the first node (i think there was one) Problem: I can't find good enough place to put it.

adds support for a new Node property "must_be_first"

This patch adds support for a new Node property "must_be_first", which when set in a Node subclass (like ExtendsNode?) means that that node should be the first one on the NodeList?, if it isnt' it raises a TemplateSyntaxError? when trying to add that node into the nodelist.

Thanks SmileyChris? for your help on this =)

Thanks for the patch, k0001, but it's not fixed until it is applied.

I will however promote it to ready for checkin, because it does everything that Malcolm wants.

(In [7084]) Fixed #5124 -- Added a reasonable error when "extends" is not the first template tag. Patch from k0001.

Refs #6274.

In 7089, a 'contains_nontext' attribute was added to the NodeList? class as a class attribute, it needs to be an attribute of a NodeList? instance.

Whoops! Sorry, I knocked that first diff out without thinking, of course when I ran my server, it failed. The correct diff is attached as contains_nontext_2.diff. Again, my appologies.

Thanks AmanKow? for the patch, but in the future, please open a new ticket for a new bug rather than re-opening this ticket (you can always reference it). In fact, I'll close this one again -- could you please open a new ticket for this and reattach your patch to that one?

Also, you haven't actually explained the problem. It may help to add some tests showing the failing behavior.