arno/isso
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #414 from vincentbernat/fix/no-eval

Browse Source 
jade: avoid using eval once compiled
This commit is contained in:
Benoît Latinier 2018-04-25 22:32:13 +02:00 committed by GitHub
commit 3754ba7596
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
isso/js/app/jade.js
View File

@ -7,6 +7,9 @@ define(["libjs-jade-runtime", "app/utils", "jade!app/text/postbox", "jade!app/te
var load = function(name, js) { var load = function(name, js) {
templates[name] = (function(jade) { templates[name] = (function(jade) {
var fn; var fn;
if (js.compiled) {
return js(jade);
}
eval("fn = " + js); eval("fn = " + js);
return fn; return fn;
})(runtime); })(runtime);

4
isso/js/lib/requirejs-jade/jade.js
View File

@ -49,8 +49,12 @@ define(function() {
write: function(plugin, name, write) { write: function(plugin, name, write) {
if (builds.hasOwnProperty(name)) { if (builds.hasOwnProperty(name)) {
write("define('" + plugin + "!" + name +"', function () {" + write("define('" + plugin + "!" + name +"', function () {" +
" var wfn = function (jade) {" +
" var fn = " + builds[name] + ";" + " var fn = " + builds[name] + ";" +
" return fn;" + " return fn;" +
" };" +
"wfn.compiled = true;" +
"return wfn;" +
"});\n"); "});\n");
} }
} }