Opened 14 years ago

Closed 14 years ago

Last modified 11 years ago

#3981 closed defect (fixed)

dojox.layout.ContentPane script tag match requires empty script tag or a src attribute

Reported by: guest Owned by: mumme
Priority: high Milestone: 0.9
Component: DojoX Layout Version: 0.9
Keywords: contentpane script tag Cc:
Blocked By: Blocking:


Since [9850] the script regex will match:

<script> correct <script src='xxx'> correct <script type='text/javascript' src='xxx'> correct

and skips <script type='dojo/method' > correct and <script type='text/javascript'> incorrect

the following regex fixes the problem (I think).




Change History (6)

comment:1 Changed 14 years ago by guest

F! a D!

try this:


comment:2 Changed 14 years ago by guest

BTW I have a CLA on file [email protected]

comment:3 Changed 14 years ago by bill

Owner: changed from Tom Trenka to mumme

comment:4 Changed 14 years ago by guest

This will really do it as a drop in replacement for the pattern. (capture groups are the same as they used are in the existing source).


comment:5 Changed 14 years ago by mumme

Resolution: fixed
Status: newclosed

(In [9965]) Make script match regex a bit smarter, didn't match type='text/javascript' correctly, added a test for that. Credits to reporter of ticket 3981,

Fixes #3981

comment:6 Changed 11 years ago by bill

Component: DojoxDojoX Layout
Note: See TracTickets for help on using tickets.