Something I have just noticed in testing...

Is COM_CJLIB_DATE_ONE_WEEK_AGO meant to be COM_CJLIB_DATE_ONE_WEEK? Currently failing to find COM_CJLIB_DATE_ONE_WEEK_AGO.

When

return JText::_('COM_CJLIB_DATE_ONE_WEEK_AGO', $suffix);

changed to

return JText::_('COM_CJLIB_DATE_ONE_WEEK', $suffix);

then resultant One week %1$s. Changed to

return JText::sprintf('COM_CJLIB_DATE_ONE_WEEK', $suffix);

and resultant correct One week ago.

Looking at the surrounding code there are also

return JText::_('COM_CJLIB_DATE_ONE_HOUR', $suffix);

and

return JText::_('COM_CJLIB_DATE_ONE_MONTH', $suffix);

Should these be changed too? I don't happen to have been watching so far when either of these messages would have been displayed.