Today's 2.6.11 zip is at 2.6.11_20150422.zip

The newest bug fixes (I hope.)

Dandello wrote on Mar 24^th, 2015 at 5:06pm:

That's what I usually do with longer spans of text... but fixing my own typos on text that is attached as a file takes a lot longer than fixing inlined text

Nothing wrong with putting them in a text file and attaching them.

Dandello wrote on Mar 24^th, 2015 at 4:13pm:


YaBB trashes what's inside the code div even before converting to e-mail text
I'm just going to not post anymore regex patterns because it just don't come out right Too many slashes for YaBB to grok

Okay according to http://www.regexplanet.com/advanced/perl/index.html

</?([A-Za-z](?>[^s>/]*))(?>(??>[^>"']+)|"[^"]*"|'[^']*')*)>  

and

<(([a-z]+|[A-Z]+)(\s?/?|[^a-zA-Z<>][^<>]*[^/<>]/?)|/([a-z]+|[A-Z]+))>
  

behave the same even on some fairly tricky (and wrong) html.

And YaBB thoroughly trashes what's inside the code div when converting to e-mail text.

Tomorrow or so I'll look at true html e-mails again.

Dandello wrote on Mar 24^th, 2015 at 2:47pm:


if you leave out ? after \s, it fails on tags like <br/>, but works for <br />...

on the other hand... YaBB chokes on \s when replying, strips off \, it's tricky because it has to match either 1 character or 2 characters because the next rule only matches 3 characters or more as it has to check the start of tag isn't mixed-case.

Okay. This one works according to http://www.regexplanet.com/advanced/perl/index.html

<(([a-z]+|[A-Z]+)(\s/?|[^a-zA-Z<>][^<>]*[^/<>]/?)|/([a-z]+|[A-Z]+))> 

and so does the one from Regular Expressions Cookbook page 426.

Dandello wrote on Mar 24^th, 2015 at 1:35pm:


So it doesn't like the literal space character... that can be fixed by using \s instead.

And so long as

</?([A-Za-z](?>[^s>/]*))(?>(??>[^>"']+)|"[^"]*"|'[^']*')*)> 

is set to global, it does work to remove just tags from code like this:

<p class="class">>>>test<<<</p><br /> 

Sequence (?/...) not recognized in regex; marked by <-- HERE in m/(?isx)<(([a-z]+|[A-Z]+)( ?/ <-- HERE ?|[^a-zA-Z<>][^<>]*[^/<>]/?)|/([a-z]+|[A-Z]+))>/ 

Sequence (?\...) not recognized in regex; marked by <-- HERE in m/(?isx)<(([a-z]+|[A-Z]+)( ?\ <-- HERE /?|[^a-zA-Z<>][^<>]*[^\/<>]\/?)|\/([a-z]+|[A-Z]+))>/
  

It's the '?/?' that chokes.

You can test it out for Perl here: http://www.regexplanet.com/advanced/perl/index.html

Dandello wrote on Mar 24^th, 2015 at 12:11am:


It's still wrong... Initial / can't be optional... it's either initial / or final / but not both...

It is possible that it chokes on you because it still thinks / is special character even though it uses ~ as separator...

Alternative version with escaped / :

 s/<(([a-z]+|[A-Z]+)( ?\/?|[^a-zA-Z<>][^<>]*[^\/<>]\/?)|\/([a-z]+|[A-Z]+))>//g
  

Um, this doesn't test out as a good regex in Perl.

~<(([a-z]+|[A-Z]+)( ?/?|[^a-zA-Z<>][^<>]*[^/<>]/?)|/([a-z]+|[A-Z]+))>~ 

However this does:

~</?([A-Za-z](?>[^\s>/]*))(?>(??>[^>"']+)|"[^"]*"|'[^']*')*)>~ 

and it checks out at http://www.regexplanet.com/advanced/perl/index.html (The code is from O'Reilly's Regular Expressions Cookbook.)

Dandello wrote on Mar 23^rd, 2015 at 9:01pm:


It is not big trouble to fix both YaBB and HTML tag detection as long as we know which characters can appear at start of tag and which characters can appear at end of tag... This is how I changed the HTML tag detection to detect everything else except comments, because those have very specific restrictions that make the regex very long...

Which means that the issue with autolink urls will break it. Looking at how that section of code evolved I think we can probably remove the

$thismessage =~ s/<.*?>//g; 

lines in while leaving the

$thismessage =~ s/\[.*?\]//g; 

  that's a few lines above it because the chances of an errant ']' in YaBB is smaller than an errant '>'. (Plus that's what's been used, supposedy successfully for ages, in PM notifications.)

The supposed best solution is to use something like HTML:: Parser to remove the HTML tags.

Dandello wrote on Mar 23^rd, 2015 at 7:41pm:

It works when there is none stray < or >... But if there is even one stray >, it doesn't... it also fails miserably if there is no other characters between < and >, which is alternative way to say !=.

http://regexr.com/ is the tool I use to check regex patterns for bugs...