Browse Source

Don't add __render_variables to custom tag vars.

It makes ChicagoBoss choke. Thanks @entronic for reporting.
The same effect can be had by passing a copy of the Variables with the RenderOptions.
Thanks @evanmiller for tip.

See pull request #83 for discussion.

The same effect can be achieved
Andreas Stenius 12 years ago
parent
commit
13875e1965
1 changed files with 1 additions and 2 deletions
  1. 1 2
      src/erlydtl_compiler.erl

+ 1 - 2
src/erlydtl_compiler.erl

@@ -1427,8 +1427,7 @@ tag_ast(Name, Args, Context, TreeWalker) ->
                           {{[ExtAst|ArgsAcc], merge_info(ExtInfo, AstInfoAcc)}, ExtTreeWalker}
                           {{[ExtAst|ArgsAcc], merge_info(ExtInfo, AstInfoAcc)}, ExtTreeWalker}
                   end
                   end
           end, {{[], #ast_info{}}, TreeWalker}, Args),
           end, {{[], #ast_info{}}, TreeWalker}, Args),
-    TagArgs = [erl_syntax:tuple([erl_syntax:atom('__render_variables'), erl_syntax:variable("_Variables")])|InterpretedArgs],
-    {RenderAst, RenderInfo} = custom_tags_modules_ast(Name, TagArgs, Context),
+    {RenderAst, RenderInfo} = custom_tags_modules_ast(Name, InterpretedArgs, Context),
     {{RenderAst, merge_info(AstInfo1, RenderInfo)}, TreeWalker1}.
     {{RenderAst, merge_info(AstInfo1, RenderInfo)}, TreeWalker1}.
 
 
 custom_tags_modules_ast(Name, InterpretedArgs, #dtl_context{ custom_tags_modules = [], is_compiling_dir = false }) ->
 custom_tags_modules_ast(Name, InterpretedArgs, #dtl_context{ custom_tags_modules = [], is_compiling_dir = false }) ->