On Mar 17, 2014, at 07:59 PM, Aurelien Bompard wrote: >On Mailman3, every "message held" notification has "XXX" as the reason. This >comes from the HoldChain class in mailman/chains/hold.py, where "XXX" is >hardcoded. I'd like to fix that. I think there could be a list in the >message metadata to which each matching rule would add a short description of >why it matched, like we currently have a list of matching rules in the >msgdata['rule_hits'] variable. I'll make sure to mark these descriptions as >i18n strings. > >Are you OK with that? Any trap I'm not seeing?
Yes, it would be pretty cool to implement this. The main reason I never did was because I couldn't quite work out a nice way to display multiple reasons given the rules for template interpolation. You'll notice the postheld.txt template has a single *indented* line containing $reason and that has implications for exactly what kind of string can go in there (e.g. if it's multiple lines, who's responsible for getting everything to line up correctly). Maybe that shouldn't be indented, but then the wrapping rules might interfere. If you can work out the display issues, keeping in mind an i18n'd postheld.txt template, and the fact that those templates can be replaced by a site or list admin, then I think your basic plan is on the right track. Cheers, -Barry _______________________________________________ Mailman-Developers mailing list Mailman-Developers@python.org https://mail.python.org/mailman/listinfo/mailman-developers Mailman FAQ: http://wiki.list.org/x/AgA3 Searchable Archives: http://www.mail-archive.com/mailman-developers%40python.org/ Unsubscribe: https://mail.python.org/mailman/options/mailman-developers/archive%40jab.org Security Policy: http://wiki.list.org/x/QIA9