Skip to content
Permalink
Browse files

MDL-56602 themes: Use version number as base themerev

  • Loading branch information...
andrewnicols committed Nov 8, 2016
1 parent 7eb3467 commit 9d75a519f38fa2d0f1988be40fe18978ba4d35e7
Showing with 58 additions and 1 deletion.
  1. +29 −0 lib/moodlelib.php
  2. +7 −1 lib/outputlib.php
  3. +22 −0 lib/tests/moodlelib_test.php
@@ -9620,6 +9620,35 @@ function get_course_display_name_for_list($course) {
}
}
/**
* Convert a Moodle Version formatted number into a guaranteed incrementing timestamp.
*
* @param float $version The version to convert
* @return int The calculated timestamp
*/
function version_to_timestamp($version) {
// Moodle version numbers are in stored in one of the following formats:
// * YYYYMMDDRR.II
// * YYYYMMDDRR
//
// To convert these into a value akin to a Unix Time Stamp, we take the timestamp of the YYYYMMDD component. and add
// a calculation of the RR.II component.
// The hours/minutes/seconds must be specified otherwise the current time is used.
$date = DateTime::createFromFormat('YmdHis', substr($version, 0, 8) . '000000');
$revision = $date->format('U');
// To ensure a unique incrementing timestamp, we multiple the $rr value by a value larger than the max of $ii,
// and then add $ii.
// As a two-digit number, the max value of $ii is 99, so we multiply by 100.
$revision += (100 * (int) substr($version, 8, 2));
// Note: If there is no .II value, or it is 00, then the substr will return false, but be calculated as 0 when cast to int.
$revision += (int) substr($version, 11, 2);
return $revision;
}
/**
* The lang_string class
*
@@ -86,7 +86,13 @@ function theme_get_revision() {
if (empty($CFG->themedesignermode)) {
if (empty($CFG->themerev)) {
return -1;
// If theme designer mode is not set, and there is no themerev, this is almost certainly part of the installation.
// We attempt to set a themerev based on the Moodle version number to avoid costly rebuilds of the dynamic
// theme files between each page load.
$version = null;
require("{$CFG->dirroot}/version.php");
return version_to_timestamp($version);
} else {
return $CFG->themerev;
}
@@ -3282,4 +3282,26 @@ public function data_can_send_from_real_email_address() {
'samecourse' => false, 'result' => false],
];
}
/**
* Unit tests for the version_to_timestamp function.
*/
public function test_version_to_timestamp() {
$baseversion = 2016110100.00;
$basestamp = version_to_timestamp($baseversion);
$this->assertEquals(1477929600, $basestamp);
// Adding 00.99 to the version should increase the timestamp.
$stamp = version_to_timestamp($baseversion + .99);
$this->assertGreaterThan($basestamp, $stamp);
// Adding 01.00 to the base version should increase the stamp higher than the 00.99 version.
$newstamp = version_to_timestamp($baseversion + 01.00);
$this->assertGreaterThan($basestamp, $newstamp);
$this->assertGreaterThan($stamp, $newstamp);
// The previous day's timestamp at it's highest increment should be lower than the base version.
$stamp = version_to_timestamp($baseversion - 00.01);
$this->assertLessThan($basestamp, $stamp);
}
}

0 comments on commit 9d75a51

Please sign in to comment.
You can’t perform that action at this time.