Thanks.
I did test it some more and it seemed to work for everything I could
throw at it.
-Jon
Paul J Stevens wrote:
Jonathan,
I've verified the validity of your patch and have merged it in my tree.
Thanks a lot.
regards,
Jonathan Feally wrote:
While playing around with 2.2.0 rc2 - this is broken in 2.1.x as well -
i found out that my netscape mail 7.2 will not show the folders as being
able to be subscribed to.
So - how to duplicate
Create a folder structure that is 3 levels deep like Lists/DBMAIL/Boring
with no other folders in DBMAIL
Unsubscribe from both Boring and its parent DBMAIL - Hit ok
Go back to add them both back in and you can't becuase DBMAIL just
doesn't show up.
Netscape is doing: 666 LIST "" "%/%"
DBMAIL returns nothing - the listex_match function never moves on
past the first % thus it fails as their is still more to be looked at on
either the pattern or the string being examined. In this case we have
both % in the pattern and DBMAIL in the string. This simple 1 line patch
says that if we still have both more string to look at and more pattern
to look at then call the whole thing again from where we left off.
Pretty simple and something that should be tested by others and put into
2.2.0
Attached is a diff -u for misc.c
-Jon
------------------------------------------------------------------------
--- ../1/dbmail-2.2.0rc2/misc.c Fri Nov 10 01:32:18 2006
+++ misc.c Tue Nov 14 15:33:14 2006
@@ -664,6 +664,7 @@
/* %. */
for (i = 0; x[i] && x[i] == p[i]; i++);
if (! x[i]) p += i;
+ if (*s && *p) return listex_match(p,s,x,flags); // We
have more to look at - lets look again.
if (*s || *p) return 0;
return 1;
------------------------------------------------------------------------
_______________________________________________
DBmail mailing list
[email protected]
https://mailman.fastxs.nl/mailman/listinfo/dbmail
