[Nms-cgi-devel] adjustment to wwwadmin.pl

[Neil W. <li...@co...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Line 465 in the parse_message_list subroutine:

    if (/<!--top: (\d+)--><li><a href="[^"]+">(.*)<\/a> - 
<b>(.*)<\/b>\s+<i>(.*)<\/i>/) {

(line break added by mail client) 

The original line above doesn't catch all messages. There needs to be an 
optional match for a preceding </li> tag after the top comment:
<!--top: (\d+)--><li>
needs to be:
<!--top: (\d+)-->[<\/li>]+<li>

The full line then becomes:

    if (/<!--top: (\d+)-->[<\/li>]+<li><a href="[^"]+">(.*)<\/a> - 
<b>(.*)<\/b>\s+<i>(.*)<\/i>/) {

I had a number of messages that could not be removed by wwwadmin because the 
message never showed up. All messages shown by wwwadmin must parse this 
statement but an additional </li> for some messages was causing the match to 
fail.

- -- 

Neil Williams
=============
http://www.codehelp.co.uk
http://www.dclug.org.uk

http://www.wewantbroadband.co.uk/

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE+x1QyiAEJSii8s+MRAsX0AKC9NyiiF0auFYSvNrga6tVemrshOQCdGZkj
pIIxWj8+Ny17eDP/Lxovc/4=
=X5bT
-----END PGP SIGNATURE-----