?
avatar Bakual
Bakual
30 May 2017

Steps to reproduce the issue

  • Create a new article, contact or newsfeed (maybe others as well) and save once intially
  • Check the "Modified Date" and "Modified By"

Expected result

  • Either both are not set like in J3 where they are only set after the next save (edit)
  • or both are set

Actual result

Only the modified date is set, but the user remains empty until the next save (edit)

Additional comments

I can see where that comes from, since with MySQL strict mode the modified column can't be an empty string. It has to be a valid date or the "nulldate".
However apparently it somehow works for com_fields where creating a field has the same behavior as in J3: The modified date is only populated after the first successive save/edit. I think it's because com_fields, the modified date has filter="unset" in the XML which means it gets nulled during validation.

We should choose a way how we want to handle that date and be consistent across all extensions, either always setting it (but then also the user) or only after an edit (like com_fields).

avatar Bakual Bakual - open - 30 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 30 May 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 30 May 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 May 2017
Category com_content
avatar tonypartridge
tonypartridge - comment - 31 May 2017

This PR should solve the issue:

#11530

As it should be merged into 4.0 too.

avatar franz-wohlkoenig franz-wohlkoenig - change - 31 May 2017
Status New Information Required
avatar Bakual
Bakual - comment - 31 May 2017

I don't think it does fix the underlying issue, because I think it would still not accept an empty string or null.

avatar Bakual Bakual - change - 2 Jun 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-06-02 08:00:15
Closed_By Bakual
avatar Bakual Bakual - close - 2 Jun 2017
avatar Bakual
Bakual - comment - 2 Jun 2017

Closing as we have a PR #16444

Add a Comment

Login with GitHub to post a comment