Assembla home | Assembla project page
 

Ticket #49 (closed defect: fixed)

Opened 3 months ago

Last modified 4 weeks ago

Sed plugin issues

Reported by: tobias382 Assigned to: tobias382
Priority: normal Component: plugins
Version: 1.5 Severity: normal
Keywords: Cc:

Description

[11:10:04] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phpwomen :sorry about what is about to happen [11:10:15] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phpwomen :phergie: test [11:10:18] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phpwomen :s/test/hest No response was received from Phergie for the above replacement.

[11:11:25] <- :Dynom!n=dynom@eden.dynom.nl PRIVMSG #phpc :config Phergie; delete * from beer [11:11:28] <- :Dynom!n=dynom@eden.dynom.nl PRIVMSG #phpc :s/* // The above replacement resulted in this error -- Warning: preg_replace(): Compilation failed: nothing to repeat at offset 0 in /home/elazar/Phergie/Plugin/Sed.php on line 121. However, it doesn't appear to do so consistently, as the replacement below also resulted in no response. Perhaps this is due to anti-flood measures? (plugins -v was used to confirm Sed was active in the channels at the time.)

[11:15:13] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phergie :config phergie; delete * from beer [11:15:20] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phergie :s/* // [11:15:32] <- :ijansch!n=ijansch@cable-175-208.zeelandnet.nl PRIVMSG #phergie :nothing, NOTHING!

Attachments

Change History

08/01/08 09:01:48 changed by Seldaek

#1 : It needs s/foo/bar/ with a trailing slash, so that's expected #2 : also expected, and very hard to prevent (just use @preg_replace I guess) since the possibilities to fuck up a regex statement are endless.. if you want it to work, s/\*// should do the trick, but I'm not sure #3 : no clue

Anyway all those seem a bit bogus to me, but I'll leave you be the judge of that.

09/17/08 00:25:26 changed by tobias382

As of r284, I don't experience the issue from the first test case. Points 2 and 3 appear to address the same issue, which is use of invalid regular expressions. Error suppression for this was added in r286.

09/17/08 00:25:40 changed by tobias382

  • status changed from new to closed.
  • resolution set to fixed.

Add/Change #49 (Sed plugin issues)




Action