in reply to Re^2: Regex to match non image urls
in thread Regex to match non image urls

Basically this doesn't work: 
$text =~ s~(\[img\]\s*.*?\.(?:jpe?g|png|svg|gif|bmp)\s*\[/img\])~Inval
+id~isg;

[download]

Replies are listed 'Best First'.
Re^4: Regex to match non image urls (Parse::BBCode)
by Anonymous Monk on Jan 18, 2013 at 10:01 UTC
    You want to use Parse::BBCode, but you could try 
    $text =~ s{
 \[(\w+)\]   # $1 tagname
 ( .*? )     # $2 stuff
 \[\/(\w+)\] # $3 tagname
}{
  TagFudge( $1, $2, $3 );
}gsex;

sub TagFudge {
    my( $open, $content, $close ) = @_;
    if( $content =~ m{\.(jpe?g|gif|xbm|png|bmp)$} ){
        return "Ok";
    } else {
        return "Invalid";
    }
}

    [download]
    But you should use Parse::BBCode
[reply]
[d/l]
      Could't get that to work, it just says everything is invalid.
[reply]
        Ah ha it was missing brackets on the tag parts. Works great now, thanks :)
[reply]
Re^4: Regex to match non image urls
by muba (Priest) on Jan 18, 2013 at 11:19 UTC

    "Doesn't work" is not a problem description.

    Are you sure $text is what you think it is? You can try to do a print "\n>$text<\n"; and see if it matches your expectations.

    If $text is what you think it is, then how doesn't it work? What is your input, what is your expected output, what is your actual output? And with "actual output" I also mean warnings and errors. On that topic, is your script running with use strict; use warnings;?

[reply]
[d/l]
[select]
Re^4: Regex to match non image urls
by userdefinable (Initiate) on Jan 18, 2013 at 10:06 UTC
    Right now it's doing the exact opposite of what I want it to!
[reply]
      Yes, and?
[reply]

In Section Seekers of Perl Wisdom