The refresh-embargoed-reviews query updates reviews.embargopastdate. But there's a problem with that: reviews.moddate is defined like this:
CREATE TABLE `reviews` (
`moddate` timestamp NOT NULL DEFAULT current_timestamp() ON UPDATE current_timestamp(),
So when we touch reviews.embargopastdate, we're also automagically updating reviews.moddate.
reviews.moddate is the date shown on the /viewgame page, so that's making all previously embargoed reviews look like they were written on their embargopastdate, which, for the majority of them, is the day we rolled out the schema change (today).
(Once we fix this, we'll need to manually go in and revert erroneous moddate values on the reviews table.)
Apropos https://github.com/iftechfoundation/ifdb/pull/270
The
refresh-embargoed-reviewsquery updatesreviews.embargopastdate. But there's a problem with that:reviews.moddateis defined like this:So when we touch
reviews.embargopastdate, we're also automagically updatingreviews.moddate.reviews.moddateis the date shown on the/viewgamepage, so that's making all previously embargoed reviews look like they were written on theirembargopastdate, which, for the majority of them, is the day we rolled out the schema change (today).(Once we fix this, we'll need to manually go in and revert erroneous
moddatevalues on thereviewstable.)