Can I suggest a RowTest here...same boiler plate each time. Also I'm curious to know, did you guys just add in the data scheme today?
Did my changes make sense? Tim On Thu, Dec 17, 2009 at 4:07 PM, John Simons <[email protected]>wrote: > Not needed, it already handles it :) > > Here are the set of tests I'm using: > > https://svn.castleproject.org/svn/castle/MonoRail/trunk/src/Castle.MonoRail.Framework.Tests/ViewComponents/CombineJSViewComponentTestCase.cs > > Cheers > John > > > On Dec 18, 9:02 am, James Curran <[email protected]> wrote: > > You missed something. Fully-qualified and absolute URLs, whether > > quoted or not, should not match. With yours, they still match if > > quoted. (I'll work on getting the "data:" tag into mine) > > > > 2009/12/17 Tim Barcz <[email protected]>: > > > > > > > > > I took a look at the script and it's actually pretty close. > > > > > Here is the updated Regex... > > > > > (?:.*?)(?:url)\((?!http|/|data:)(?<Url>.*?)\)(?:.*?) > > > > > The updates to the regex posted can be summarized as follows: > > > > > Added non-capturing instructions to all unnecessary groups (only > capturing > > > group now is "Url") > > > added the "data" uri scheme (a less common/known option) > > > > > I'm not sure on the quoting vs non-quoting issue if you want the regex > to > > > "swallow" it. In other words when you get the url do you want it > unquoted > > > (if it was quoted). > > > > > Here is my testing set: > > > > > background-image: url(../images/default/shadow-c.png); // non quoted > > > relative > > > background-image: url('../images/default/shadow-c.png'); // quotes > relative > > > background-image: url(images/icons/file-xslx1.gif) // non-quoted > relative > > > (no "../") > > > background-image: url('images/icons/file-xslx1.gif') // non-quoted > relative > > > (no "../") > > > background-image:url(data:image/gif;base64,AAAA) // non-quoted data > > > background-image:url('data:image/gif;base64,AAAA') // quoted data > > > background-image:url(http://www.domain.com/images/image.gif) // > non-quoted > > > full URL > > > background-image:url('http://www.domain.com/images/image.gif') // > quoted > > > full URL > > > background-image:url(/image/icons/file-xlsx.gif) // non-quoted absolute > > > background-image:url('/image/icons/file-xlsx.gif') // quoted absolute > > > > -- > > Truth, > > James > > -- > > You received this message because you are subscribed to the Google Groups > "Castle Project Development List" group. > To post to this group, send email to [email protected] > . > To unsubscribe from this group, send email to > [email protected]<castle-project-devel%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/castle-project-devel?hl=en. > > > -- Tim Barcz Microsoft C# MVP Microsoft ASPInsider http://timbarcz.devlicio.us http://www.twitter.com/timbarcz -- You received this message because you are subscribed to the Google Groups "Castle Project Development List" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.
