On Thu, Oct 25, 2012 at 10:50 AM, Tyler Romeo <[email protected]> wrote: > To be clear, I am not saying that any language functionality should be in > MWTimestamp, because that's not what it's for. On the other side of it, > there should be no timestamp functionality in the Language class either. We > need a fix so that if MWTimestamp::getHumanTimestamp is called, Language is > used to generate it properly.
I disagree. The human/formatting stuff absolutely should be in Language. The problem with having it in MWTimestamp is it requires a language or context to be passed to do it right. This introduces a new dependency there that I don't really like (MWTimestamp is nice and self-contained right now). -Chad _______________________________________________ Wikitech-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikitech-l
