search

inclusive-design / AChecker

Automated interactive Web content accessibility checker.
https://achecker.ca
GNU General Public License v2.0
69 stars 61 forks source link

ACHECKER-3: Fixing Issue with last_updated syntax #86

Closed VictorAlagwu closed 6 years ago

VictorAlagwu commented 6 years ago

Notice an issue with the use of 'last_updated' date NOT NULL DEFAULT CURRENT_TIMESTAMP , which is supposed to be 'last_updated' TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP

cindyli commented 6 years ago

@VictorAlagwu, good catch. I wonder if you can use DATETIME instead of TIMESTAMP for this field because:

  1. Be consistent with date definitions in other tables;
  2. From reading the mysql doc about the comparison between DATETIME and TIMESTAMP, TIMESTAMP has a much smaller range of '1970-01-01 00:00:01' UTC to '2038-01-19 03:14:07' UTC while DATETIME values can be '1000-01-01 00:00:00.000000' to '9999-12-31 23:59:59.999999'.