fix: wrong non-multi-array caching of getMovement caused a lot of error

#377
This commit is contained in:
Martin Ambrus
2017-11-25 00:45:33 +01:00
parent 7bc582e674
commit f599e08586
+7 -7
View File
@@ -5275,7 +5275,7 @@ References: User ID/Message ID, Mode
// first of all, check if we should be using cache and whether the field // first of all, check if we should be using cache and whether the field
// required is already cached // required is already cached
if ($use_cache && !$array_passed && isset(self::$marketMovementCache[$type.$village[0].$mode]) && is_array(self::$marketMovementCache[$type.$village[0].$mode]) && !count(self::$marketMovementCache[$type.$village[0].$mode])) { if ($use_cache && !$array_passed && isset(self::$marketMovementCache[$type.$village[0].$mode]) && is_array(self::$marketMovementCache[$type.$village[0].$mode]) && !count(self::$marketMovementCache[$type.$village[0].$mode])) {
return []; return self::$marketMovementCache[$type.$village[0].$mode];
} else if ($use_cache && $array_passed) { } else if ($use_cache && $array_passed) {
// check what we can return from cache // check what we can return from cache
$newIDs = []; $newIDs = [];
@@ -5294,7 +5294,7 @@ References: User ID/Message ID, Mode
} }
} else if ($use_cache && !$array_passed && ($cachedValue = self::returnCachedContent(self::$marketMovementCache, $type.$village[0].$mode)) && !is_null($cachedValue)) { } else if ($use_cache && !$array_passed && ($cachedValue = self::returnCachedContent(self::$marketMovementCache, $type.$village[0].$mode)) && !is_null($cachedValue)) {
// special case when we have empty arrays cached for this cache only // special case when we have empty arrays cached for this cache only
return ($array_passed ? self::$marketMovementCache : [$cachedValue]); return ($array_passed ? self::$marketMovementCache: $cachedValue);
} }
$time = time(); $time = time();
@@ -5347,7 +5347,7 @@ References: User ID/Message ID, Mode
} else { } else {
if ($result && count($result)) { if ($result && count($result)) {
foreach ( $result as $record ) { foreach ( $result as $record ) {
self::$marketMovementCache[ $type . $record[ $where ] . $mode ] = $record; self::$marketMovementCache[ $type . $record[ $where ] . $mode ][] = $record;
} }
} }
@@ -5802,7 +5802,7 @@ References: User ID/Message ID, Mode
// first of all, check if we should be using cache and whether the field // first of all, check if we should be using cache and whether the field
// required is already cached // required is already cached
if ($use_cache && !$array_passed && isset(self::$abTechCache[$vid[0]]) && is_array(self::$abTechCache[$vid[0]]) && !count(self::$abTechCache[$vid[0]])) { if ($use_cache && !$array_passed && isset(self::$abTechCache[$vid[0]]) && is_array(self::$abTechCache[$vid[0]]) && !count(self::$abTechCache[$vid[0]])) {
return []; return self::$abTechCache[$vid[0]];
} else if ($use_cache && $array_passed) { } else if ($use_cache && $array_passed) {
// check what we can return from cache // check what we can return from cache
$newVIDs = []; $newVIDs = [];
@@ -6127,7 +6127,7 @@ References: User ID/Message ID, Mode
// first of all, check if we should be using cache and whether the field // first of all, check if we should be using cache and whether the field
// required is already cached // required is already cached
if ($use_cache && !$array_passed && isset(self::$oasisReinforcementsCache[$ref[0].$mode]) && is_array(self::$oasisReinforcementsCache[$ref[0].$mode]) && !count(self::$oasisReinforcementsCache[$ref[0].$mode])) { if ($use_cache && !$array_passed && isset(self::$oasisReinforcementsCache[$ref[0].$mode]) && is_array(self::$oasisReinforcementsCache[$ref[0].$mode]) && !count(self::$oasisReinforcementsCache[$ref[0].$mode])) {
return []; return self::$oasisReinforcementsCache[$ref[0].$mode];
} else if ($use_cache && $array_passed) { } else if ($use_cache && $array_passed) {
// check what we can return from cache // check what we can return from cache
$newREFs = []; $newREFs = [];
@@ -6346,7 +6346,7 @@ References: User ID/Message ID, Mode
// first of all, check if we should be using cache and whether the field // first of all, check if we should be using cache and whether the field
// required is already cached // required is already cached
if ($use_cache && !$array_passed && isset(self::$villageReinforcementsCache[$id[0].$mode]) && is_array(self::$villageReinforcementsCache[$id[0].$mode]) && !count(self::$villageReinforcementsCache[$id[0].$mode])) { if ($use_cache && !$array_passed && isset(self::$villageReinforcementsCache[$id[0].$mode]) && is_array(self::$villageReinforcementsCache[$id[0].$mode]) && !count(self::$villageReinforcementsCache[$id[0].$mode])) {
return []; return self::$villageReinforcementsCache[$id[0].$mode];
} else if ($use_cache && $array_passed) { } else if ($use_cache && $array_passed) {
// check what we can return from cache // check what we can return from cache
$newIDs = []; $newIDs = [];
@@ -7359,7 +7359,7 @@ References: User ID/Message ID, Mode
// first of all, check if we should be using cache and whether the field // first of all, check if we should be using cache and whether the field
// required is already cached // required is already cached
if ($use_cache && !$array_passed && isset(self::$prisonersCache[$wid[0].$mode]) && is_array(self::$prisonersCache[$wid[0].$mode]) && !count(self::$prisonersCache[$wid[0].$mode])) { if ($use_cache && !$array_passed && isset(self::$prisonersCache[$wid[0].$mode]) && is_array(self::$prisonersCache[$wid[0].$mode]) && !count(self::$prisonersCache[$wid[0].$mode])) {
return []; return self::$prisonersCache[$wid[0].$mode];
} else if ($use_cache && $array_passed) { } else if ($use_cache && $array_passed) {
// check what we can return from cache // check what we can return from cache
$newWIDs = []; $newWIDs = [];