[patch] More robust resolve_variable() in Template()

[anonymous]

After having issues with stuff not raising the conventional errors when the templates tries to access them as dictionaries et c; I modified the try/except nest in Template() resolve_variable() to try and make it more robust.

[Adrian Holovaty]

What do you mean by "more robust"? Faster?

[anonymous]

The idea was that the code should check if the object is a dictionary before trying to access it as one, or check if an attribute existed before trying to acccess it. This because I was having problems with certain objects returning errors that were not being handled.

I later realized that stuff like:

"elif bit in dir(current): # attribute lookup" 

won't be good anyway since you might want to access attributes via some object's getattr.

[Chris Beaven]

Anonymous author admitted patch wasn't good enough. Let's just close it for now since there haven't been many complaints about this.