Details
-
Type:
Improvement
-
Status: Reopened
-
Priority:
Minor
-
Resolution: Unresolved
-
Affects Version/s: 4.7.9
-
Fix Version/s: Unscheduled
-
Component/s: None
-
Labels:
-
Versioning Impact:Patch (backwards-compatible bug fixes)
-
Documentation Required?:None
-
Funding Source:Contributed Code
Description
We enabled cron versionCheck to get rid of the huge lag when logging into CiviCRM but it doesn't seem to be working. I feel like the reason is a typo
if (!empty($this->cronJob['is_active']) &&
(!$this->isInfoAvailable || filemtime($this->cacheFile) < $expiryTime)
) {
// First try updating the files modification time, for 2 reasons:
// - if the file is not writeable, this saves the trouble of pinging back
// - if the remote server is down, this will prevent an immediate retry
if (touch($this->cacheFile) === FALSE) {
throw new Exception('File not writable');
}
$this->fetch();
}
My take on the code is the first line should be
if (empty($this->cronJob['is_active']) &&
Attachments
Issue Links
- supplements
-
CRM-17637 Address ping back performance issues
-
- Done/Fixed
-
- links to