diff options
author | diosmosis <benaka@piwik.pro> | 2015-08-21 19:28:21 +0300 |
---|---|---|
committer | diosmosis <benaka@piwik.pro> | 2015-08-21 19:28:21 +0300 |
commit | adb5773c7fdc63e846b0d815cf8f0c89ea841967 (patch) | |
tree | de810136a32147ec0e384196df1ff712df71b54f /core/CronArchive | |
parent | 7dc49ccb9767b601f01c1fe0d1a5fdb3476c3d69 (diff) |
Small refactor to remove indentation in SegmentArchivingRequestUrlProvider.
Diffstat (limited to 'core/CronArchive')
-rw-r--r-- | core/CronArchive/SegmentArchivingRequestUrlProvider.php | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/core/CronArchive/SegmentArchivingRequestUrlProvider.php b/core/CronArchive/SegmentArchivingRequestUrlProvider.php index 65abe86c32..708e57ae77 100644 --- a/core/CronArchive/SegmentArchivingRequestUrlProvider.php +++ b/core/CronArchive/SegmentArchivingRequestUrlProvider.php @@ -154,18 +154,23 @@ class SegmentArchivingRequestUrlProvider if ($this->isSegmentForSite($segment, $idSite) && $segment['definition'] == $segmentDefinition ) { + // check for an earlier ts_created timestamp $createdTime = Date::factory($segment['ts_created']); if ($createdTime->getTimestamp() < $earliestCreatedTime->getTimestamp()) { $earliestCreatedTime = $createdTime; } - if (!empty($segment['ts_last_edit'])) { - $lastEditTime = Date::factory($segment['ts_last_edit']); - if ($latestEditTime === null - || $latestEditTime->getTimestamp() < $lastEditTime->getTimestamp() - ) { - $latestEditTime = $lastEditTime; - } + // if there is no ts_last_edit timestamp, initialize it to ts_created + if (empty($segment['ts_last_edit'])) { + $segment['ts_last_edit'] = $segment['ts_created']; + } + + // check for a later ts_last_edit timestamp + $lastEditTime = Date::factory($segment['ts_last_edit']); + if ($latestEditTime === null + || $latestEditTime->getTimestamp() < $lastEditTime->getTimestamp() + ) { + $latestEditTime = $lastEditTime; } } } |