Not sure what you mean "mixed in with this commit"? Yes I checked it into stable, the original reporter of the issue was having problems in stable, seemed stupid to test provide patch against development.
I've got to do this for devel next.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I said "mixed" because that commit includes two other unrelated fixes. Out of all the files touched in that commit, the fixes needed for this issue are only in a couple of them.
Whether or not the OP saw the problem in STABLE or DEVEL is irrelevant as far as I can see -- the responsibility on our side is to then see if the issue exists in either or both products. If new IMAP library code is added, for instance, it should certainly be added to DEVEL so we don't lose it going forward.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This bug has been resolved.
Please pull the latest version from the appropriate development
tree repository to fix your bug.
Thank you for your help in resolving this issue.
I'm reopening it because I don't think this issue has been tested in our development branch (1.5.2 currently).
The STABLE branch fix was mixed in with this commit:
http://squirrelmail.svn.sourceforge.net/viewvc/squirrelmail?view=rev&revision=13789
Not sure what you mean "mixed in with this commit"? Yes I checked it into stable, the original reporter of the issue was having problems in stable, seemed stupid to test provide patch against development.
I've got to do this for devel next.
I said "mixed" because that commit includes two other unrelated fixes. Out of all the files touched in that commit, the fixes needed for this issue are only in a couple of them.
Whether or not the OP saw the problem in STABLE or DEVEL is irrelevant as far as I can see -- the responsibility on our side is to then see if the issue exists in either or both products. If new IMAP library code is added, for instance, it should certainly be added to DEVEL so we don't lose it going forward.