Hi,
On line 327 'name' => $list['info']['name'] in context_addassets.module file, i find it not reliable as the theme machine name or folder name is not always the same as the arbitrary theme name. So, I change it to 'name' => $list['name']. Is it going to break anything? Thanks!
Comments
Comment #1
jstollerDev version has same issue.
Comment #2
jstollerNow that I think about it, why is this code running repeatedly on every page load? From what I can tell, this function is scanning my theme directory eight times on every page load—four times for css and four times for js files. I can understand the module scanning the directory for all files when I'm setting up a new context, but when a page loads, shouldn't the module already know which files its activating and where they are? Forgive me if this is just my ignorance in how context works.
And just because its in the same section of code, I noticed you define the $list variable twice, the same way, for no apparent reason. See my comments in ALL CAPS bellow.