Note also that both of the suggested fixes are changes to a mailman file, not one of the one's we've previously declared to have systers-specific changes. So, at a minimum, fixing this bug requires adopting/checking-in another systers-specific file.
It might be better though, to use this wee fix as a stepping stone to our eventual mailman integration where, presumably, our dlist features will be optional. So rather than unilaterally plug in the +new, we could call a method that would return a 'dlistExtra' string that a) would be blank if the dlist feature is disabled, b) would be '+new' if dlist is enabled, and c) eventually could be localized if we decide to support globalization.
Note also that both of the suggested fixes are changes to a mailman file, not one of the one's we've previously declared to have systers-specific changes. So, at a minimum, fixing this bug requires adopting/ checking- in another systers-specific file.
It might be better though, to use this wee fix as a stepping stone to our eventual mailman integration where, presumably, our dlist features will be optional. So rather than unilaterally plug in the +new, we could call a method that would return a 'dlistExtra' string that a) would be blank if the dlist feature is disabled, b) would be '+new' if dlist is enabled, and c) eventually could be localized if we decide to support globalization.