Re: Bug in squirrelmail-20111024_0200-SVN, addrbook_search_html.php

On 11/14/2011 08:49 PM, Paul Lesniewski wrote:
> Hi Dave, sorry for the delay.

No worries!

...snipped...

>> The following patch resolves this issue:
>
> This was fixed in September - looks like you're using a version from
> October.  The "better" fix is in templates/default/input.tpl, although
> even that fix isn't ideal in the long term.  Here's the commit you'd
> need if you don't want to install a new snapshot.
>
> http://squirrelmail.svn.sourceforge.net/viewvc/squirrelmail?view=revision&revision=14145

Ugh!  This is my fault.  I'm actually running a slightly older version 
in production and I had to fix a couple bugs along the way when we first 
put it into production.  I finally got the time to start going through 
and reporting some of the bugs I fixed, so I just grabbed the latest 
code and started to do diffs.  Since I saw that the code in 
addrbook_search_html.php hadn't changed, I assumed the same bug still 
existed.

I think I still have a couple others to report.  Shall I do so with the 
same disclaimer as above?  Or would you rather wait until I can actually 
get the latest code up and running to actually verify the bugs still 
exist before submitting my patches?

Sorry for the false alarm :(

Thanks!

Dave

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
-----
squirrelmail-devel mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: [email protected]
List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel