mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 05:58:34 +01:00
Code Issues Projects Releases Wiki Activity

MDL-68697 core_h5p: Store missing translations as null

Browse Source
This commit is contained in:
Andrew Nicols 2020-05-13 09:18:59 +08:00
parent 206e179df5
commit 0e809762d4
1 changed files with 19 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
h5p/classes/editor_framework.php
View File

@ -62,7 +62,14 @@ class editor_framework implements H5peditorStorage {
$librarykey = helper::get_cache_librarykey(core::record_to_string($library));
$cachekey = "{$librarykey}/{$lang}";
$translation = $langcache->get($cachekey);
if ($translation) {
if ($translation !== false) {
// When there is no translation we store it in the cache as `null`.
// This API requires it be returned as `false`.
if ($translation === null) {
return false;
}
return $translation;
}
@ -88,14 +95,19 @@ class editor_framework implements H5peditorStorage {
$result = $DB->get_record_sql($sql, $params);
if (!empty($result)) {
// If the JS language file exists, its content should be returned.
$fs = get_file_storage();
$file = $fs->get_file_by_hash($result->pathnamehash);
$translation = $file->get_content();
if (empty($result)) {
// Save the fact that there is no translation into the cache.
// The cache API cannot handle setting a literal `false` value so conver to `null` instead.
$langcache->set($cachekey, null);
return false;
}
// Save translation into the cache (even if there is no translation for this language).
// Save translation into the cache, and return its content.
$fs = get_file_storage();
$file = $fs->get_file_by_hash($result->pathnamehash);
$translation = $file->get_content();
$langcache->set($cachekey, $translation);
return $translation;