Incomplete code for handling current-scope loop in templates

[GuySartorelli]

Found while working on #11237

If you create a <% loop %> tag inside a template without including an argument (i.e. not <% loop $MyIterable %>), the template is generated with some very clearly incomplete code, which includes the placeholder "Foo".

Template:

<% loop %>
    // Inside the loop here
<% end_loop %>

Cached template (what actually gets executed):

$scope->obj('Up', null)->obj('Foo', null);
$scope->pushScope();
while (($key = $scope->next()) !== false) {
$val .= '    // Inside the loop here';
};
$scope->popScope();

This is handled by SSTemplateParser::ClosedBlock_Handle_Loop()

The relevant code was added in 91f4ba1 as part of #209 and it seems pretty clear the intention was to loop over whatever is currently in scope (basically a shortcut for <% loop $Me %>) but that for whatever reason the feature was never fully implemented.

PRs

• FIX Loop over current scope when no argument passed to loop block #11251
• DOC Document the current scope loop developer-docs#518

[sabina-talipova]

PRs merged. Task is completed. Close.