Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.4
-
Fix Version/s: 2.4
-
Component/s: Caching
-
Labels:
-
Testing Instructions:
- Run unit tests
-
Affected Branches:MOODLE_24_STABLE
-
Fixed Branches:MOODLE_24_STABLE
-
Pull from Repository:
-
Pull Master Branch:wip-
MDL-36111-m24 -
Pull Master Diff URL:
Description
A few small obvious things to fix up. Big thanks to Fred for looking over the code to find these!
Gliffy Diagrams
Attachments
Issue Links
- has been marked as being related by
-
-
- Closed
-
Activity
- All
- Comments
- Work Log
- History
- Activity
- Links Hierarchy
Putting this up for integration now.
Integrated, thanks Sam
Looks good.
What about adding an extra commit to remove ;; in cache/?
# find cache/ -type f -name '*.php' -exec grep -H -n -E ';;$' {} \; | sort
|
cache/classes/dummystore.php:267: return $cache;;
|
cache/stores/mongodb/lib.php:242: return false;;
|
cache/stores/static/lib.php:363: return $cache;;
|
(More in MDL-36357)
Matteo, thanks! I have done a quick commit to remove those new ones, since you pointed them out. 
Many, many thanks for your effort!
Millions of people will enjoy the results of your work, yay!
Closing as fixed. Ciao
Noticed this as well. Just an argument which is never used because not set:
diff --git a/cache/classes/factory.php b/cache/classes/factory.phpindex 729a46b..32415fc 100644--- a/cache/classes/factory.php+++ b/cache/classes/factory.php@@ -135,7 +135,7 @@ class cache_factory {}$definition = $this->create_definition($component, $area, $aggregate);$definition->set_identifiers($identifiers);- $cache = $this->create_cache($definition, $identifiers);+ $cache = $this->create_cache($definition);if ($definition->should_be_persistent()) {$this->cachesfromdefinitions[$definitionname] = $cache;}