Not sure what is included here, just pasting from the local localize.drupal.org diff, asking killes to explain it.

CommentFileSizeAuthor
l10n-packager.patch1.57 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

killes@www.drop.org’s picture

The only change is in the where condition

pr.status IS NULL OR (pr.status = %d AND (pr.checked < %d OR pr.updated < %d))

pr.status IS NULL OR ((pr.status = %d AND (pr.checked < %d OR (pr.updated < %d AND (pr.checked < %d OR pr.checked IS NULL)))))

Essentially, I also check if a release hasn't be checked before.

Gábor Hojtsy’s picture

Breaking that down, "pr.updated < %d AND pr.checked < %d" looks like just a subset of "pr.checked < %d". The added condition seems to only filter out items where pr.checked is either equivalent to the current timestamp or is in the future. The later one could seem like a bug in itself, hopefully not to be fixed here. The current timestamp would only kick in if we are checking the same thing again in the cycle. Did you see that happening?

Gábor Hojtsy’s picture

Status: Needs review » Needs work

@killes rolled it back on live for now, leaving this open for possible future inclusion, once we figure out how would this help at all.