URL rewritting for the build scripts removes bgiframe.js from the head when it shouldn't

Description

URL rewritting for the build scripts removes bgiframe.js from the head when it shouldn't

It should only remove references to files for which are contained within the InfusionAll.js file

Environment

None

Activity

Show:

Justin Obara November 13, 2009 at 2:15 PM

Closed as per previous comment

Justin Obara June 11, 2009 at 4:57 PM

Instead of replacing Fluid.js with a reference to the single concatenated js file, the first occurence of a file that is contained within the single js file, is replaced. This will prevent the issue of the single js file not being added to pages where Fluid.js does not occur but other files from the single js file do.

Justin Obara June 11, 2009 at 4:44 PM

Actually looks now as though it is removing jQuery, which is the only js file on the outer page that is in the concatenated js file

Justin Obara June 2, 2009 at 6:52 PM

For the bSpace example, the outer page has jQuery but no Fluid in the js references.

Since the InfusionAll.js replaces the reference to Fluid.js it is removing the jquery reference but not putting InfusionAll.js in.

Should probably check for Fluid.js before doing an URL rewriting in a file. Or replace Fluid.js with infustionAll.js and not do any more removing of js references unless InfusionAll.js is present.

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created June 2, 2009 at 6:37 PM
Updated November 13, 2009 at 2:15 PM
Resolved June 11, 2009 at 4:57 PM

Flag notifications