#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: |
Description
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).
<code>
<scripts*(?![>]*type=['"]?dojo)(?:[>]*?(src=(['"]?)([>]*?)1[>]*)?)?>([sS]*?)</script>
</code>
Change History (6)
comment:1 Changed 14 years ago by
comment:3 Changed 14 years ago by
Owner: | changed from Tom Trenka to mumme |
---|
comment:4 Changed 14 years ago by
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).
<scripts*(?![^>]*type=['"]?dojo)(?:[^>]*?(?:src=(['"]?)([^>]*?)1[^>]*)?)*>([sS]*?)</script>
comment:5 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:6 Changed 11 years ago by
Component: | Dojox → DojoX Layout |
---|
Note: See
TracTickets for help on using
tickets.
F! a D!
try this: