A question about executeScripts

Lines 208 to 213 in c26c223

    
           // execute scripts when DOM have been completely updated 
        
           this.options.selectors.forEach(function(selector) { 
        
             forEachEls(document.querySelectorAll(selector), function(el) { 
        
               executeScripts(el); 
        
             }); 
        
           });

pjax/lib/execute-scripts.js

Lines 10 to 17 in c26c223

    
           forEachEls(el.querySelectorAll("script"), function(script) { 
        
             if (!script.type || script.type.toLowerCase() === "text/javascript") { 
        
               if (script.parentNode) { 
        
                 script.parentNode.removeChild(script); 
        
               } 
        
               evalScript(script); 
        
             } 
        
           });

After script.parentNode.removeChild(script); is executed, script.parentNode will be null

pjax/lib/eval-script.js

Lines 4 to 5 in c26c223

    
           var parent = 
        
             el.parentNode || document.querySelector("head") || document.documentElement;

This means that the return value of the short-circuit evaluation here is document.querySelector("head")

In other words, all the scripts that are refreshed by pjax will be placed in the <head>. Is this expected behavior?

	// execute scripts when DOM have been completely updated
	this.options.selectors.forEach(function(selector) {
	forEachEls(document.querySelectorAll(selector), function(el) {
	executeScripts(el);
	});
	});

	forEachEls(el.querySelectorAll("script"), function(script) {
	if (!script.type \|\| script.type.toLowerCase() === "text/javascript") {
	if (script.parentNode) {
	script.parentNode.removeChild(script);
	}
	evalScript(script);
	}
	});

	var parent =
	el.parentNode \|\| document.querySelector("head") \|\| document.documentElement;