Tickets :: Comment on [#4345] sieve: empty allof generates 'false' instead of 'true'

* Your Email Address
* Spam protection	Enter the letters below: . ,. .. .\ /.__ \./ \| \|\| \| >< [ __ \| \|__\|\|__\|/ \[_./
Comment	>> You wrote: > >> > >>> That is anyof, which evaluates to false. > >> > >> But with your patch it always evaluates to true if no conditions are > >> specified. > > > > I told you already, that it is not the case. Please. Apply my patch > to a test directory and test it if you do not believe, but I think it > is clear from the source also. Since it doesn't patch > Sieve_Test_Anyof class, but Sieve_Test_Allof only. > > > >> But this is an academical discussion anyway, because "if > >> false" doesn't make any sense. "if true" doesn't make much sense > >> either, so I'm tending to disallow any rules that don't have > >> conditions specified. > > > > It will be a big mistake, since I see no other way to create an > always running custom rule, if there is, please enlighten me. Anyway > this was the reason, why I bothered patching it. > > > > gergely
Attachment
Watch this ticket