fix(admin): use correct params for memcache and redis server information

Elgg · Jul 17, 2023 · 6ee26b9 · 6ee26b9
1 parent 1a285b4
commit 6ee26b9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/docs/admin/performance.rst b/docs/admin/performance.rst
@@ -188,8 +188,14 @@ Uncomment and populate the following sections in ``settings.php``
     $CONFIG->memcache = true;
     
     $CONFIG->memcache_servers = array (
-        array('server1', 11211),
-        array('server2', 11211)
+        array (
+            'host' => 'server1',
+            'port' => 11211
+        ),
+        array (
+            'host' => 'server2',
+            'port' => 11211
+        )
     );
 
 Optionaly if you run multiple Elgg installations but use ony one Memcache server, you may want 

diff --git a/views/default/admin/server/memcache.php b/views/default/admin/server/memcache.php
@@ -15,7 +15,7 @@
 }
 
 foreach ($servers as $server) {
-	$memcache->addserver($server[0], $server[1]);
+	$memcache->addserver($server['host'], $server['port']);
 }
 
 if ($memcache instanceof Memcache) {

diff --git a/views/default/admin/server/redis.php b/views/default/admin/server/redis.php
@@ -12,7 +12,7 @@
 $redis = new Redis();
 
 foreach ($servers as $server) {
-	$redis->connect($server[0], $server[1]);
+	$redis->connect($server['host'], $server['port']);
 }
 
 $password = elgg_extract('password', elgg_get_config('redis_options'));