From mboxrd@z Thu Jan 1 00:00:00 1970 From: marcus-sup@bar-coded.net (Marcus Williams) Date: Wed, 07 Nov 2007 16:34:32 +0000 Subject: [sup-talk] [PATCH] Fix for before-add-message hook In-Reply-To: <1194451347-sup-1828@tomsk> References: <1194344471-sup-8807@tomsk> <1194400555-sup-7015@south> <1194451347-sup-1828@tomsk> Message-ID: <1194452873-sup-8208@tomsk> On 7.11.2007, Marcus Williams wrote: > It might not be related to this change, but I'd not noticed it until > just now. I'll see if I can reproduce it somehow. Got it - in is_relevant you changed the patch to use "&&" instead of "and" and I think this broke it because of the precedence of "&&" makes it do m.labels.include? (:inbox && ......) whereas mine does (m.labels.include? :inbox) and (......) reverting back to "and" fixes it in my version. Marcus