mbirth/LuckyCoinkydink
1
0
Fork 0
Code Issues Pull Requests Activity

js ending delimiter S3 auto literal behaviour is quite bulletproof

Browse Source 
tests have proofed " } ;" and " };" and even "};" are interpreted correct!
This commit is contained in:
Ian 2012-12-07 18:00:52 +01:00
parent 7544348e08
commit 449367c7aa
6 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/admin/tpl/out_stack_loop.tpl
View File

@ -143,8 +143,9 @@
{$none}
{/if}
</fieldset>
{* Smarty 3 has a new auto literal option which is enabled by default.
When the { is surrounded by whitespace it is not interpreted as smarty delimiter but literal. e.g. ' } ;' *}
{*** Smarty 3 has a new auto literal option which is enabled by default.
When the { is surrounded by whitespace it is not interpreted as smarty delimiter but literal. e.g. ' } ;'
AS tests have proofed, ending javascript delimiters " } ;" and " };" and even "};" work fine as well. So back to origin. ***}
<script>
function sort_{$config_item}_Sequence() {
//var seq = DragDrop.serData('{$config_item}_group', null);
@ -173,7 +174,7 @@
DragDrop.makeListContainer(lst, '{$config_item}_group');
lst.onDragOut = function() {
sort_{$config_item}_Sequence();
} ;
};
}
addLoadEvent(init_{$config_item}_Sequence);
</script>

4
include/admin/tpl/show_plugins.fnc.tpl
View File

@ -15,8 +15,8 @@
{($is_first) ? 'var ' : ''}list = document.getElementById("{$sidebar}_col");
DragDrop.makeListContainer(list, 'g1');
list.onDragOver = function() { this.style["border"] = "1px solid #4d759b"; } ;
list.onDragOut = function() { this.style["border"] = "none"; } ;
list.onDragOver = function() { this.style["border"] = "1px solid #4d759b"; };
list.onDragOut = function() { this.style["border"] = "none"; };
{assign var="is_first" value=false}
{/foreach}

7
include/admin/tplold/out_stack_loop.tpl
View File

@ -165,8 +165,9 @@
{if (!is_array($items) || empty($order_id))}
{$none}
{/if}
{* Smarty 3 has a new auto literal option which is enabled by default.
When the { is surrounded by whitespace it is not interpreted as smarty delimiter but literal. e.g. ' } ;' *}
{*** Smarty 3 has a new auto literal option which is enabled by default.
When the { is surrounded by whitespace it is not interpreted as smarty delimiter but literal. e.g. ' } ;'
AS tests have proofed, ending javascript delimiters " } ;" and " };" and even "};" work fine as well. So back to origin. ***}
<script type="text/javascript">
function sort_{$config_item}_Sequence() {
//var seq = DragDrop.serData('{$config_item}_group', null);
@ -195,7 +196,7 @@
DragDrop.makeListContainer(lst, '{$config_item}_group');
lst.onDragOut = function() {
sort_{$config_item}_Sequence();
} ;
};
}
addLoadEvent(init_{$config_item}_Sequence);
</script>

4
include/admin/tplold/show_plugins.fnc.tpl
View File

@ -14,8 +14,8 @@ When the { is followed by a space it's not interpreted as smarty delimiter but l
{($is_first) ? 'var ' : ''}list = document.getElementById("{$sidebar}_col");
DragDrop.makeListContainer(list, 'g1');
list.onDragOver = function() { this.style["border"] = "1px solid #4d759b"; } ;
list.onDragOut = function() { this.style["border"] = "none"; } ;
list.onDragOver = function() { this.style["border"] = "1px solid #4d759b"; };
list.onDragOut = function() { this.style["border"] = "none"; };
{assign var="is_first" value=false}
{/foreach}

2
templates/2k11/admin/media_choose.tpl
View File

@ -296,7 +296,7 @@ if (parent.frames && parent.frames['tree']) {
tree = new YAHOO.widget.TreeView("treeDiv1");
tree.onExpand = function(node) {
document.getElementById('newdirlink').href = basenewdirurl + node.data.relpath;
} ;
};
coreNode = new YAHOO.widget.TextNode("{$CONST.MEDIA}", tree.getRoot(), false);
coreNode.href = baseurl;

2
templates/default/admin/media_choose.tpl
View File

@ -274,7 +274,7 @@
tree = new YAHOO.widget.TreeView("treeDiv1");
tree.onExpand = function(node) {
document.getElementById('newdirlink').href = basenewdirurl + node.data.relpath;
} ;
};
coreNode = new YAHOO.widget.TextNode("{$CONST.MEDIA}", tree.getRoot(), false);
coreNode.href = baseurl;