Edgewall Software

Opened 18 years ago

Closed 18 years ago

#36 closed defect (fixed)

[PATCH] Context name set to None should override builtins

Reported by: cboos Owned by: cmlenz
Priority: minor Milestone: 0.3
Component: Expression evaluation Version: 0.2
Keywords: Cc:

Description

I discovered that while trying to define a macro using an id parameter, e.g. 

<py:def function="check(name, id=None)">
Hello ${name} (${id})
</py:def>

Patch follows.

Attachments (1)

data_always_override_builtin-r239.diff​ (2.2 KB) - added by cboos 18 years ago.
Test case plus fix, on r239. I used the same logic for the _locals and for the data. I'm not 100% sure how the _locals dict gets populated, but I think it should work that way.

Download all attachments as: .zip

Change History (2)

Changed 18 years ago by cboos

Test case plus fix, on r239. I used the same logic for the _locals and for the data. I'm not 100% sure how the _locals dict gets populated, but I think it should work that way.

comment:1 Changed 18 years ago by cmlenz

  • Resolution set to fixed
  • Status changed from new to closed

This has been fixed in [242]. Also the the ​discussion on the mailing list.

Note: See TracTickets for help on using tickets.