cache administration: prepopulate memcached settings and fix message upon...

Closes #1536 and #1124

Merge request studip/studip!962

app/controllers/admin/cache.php

@@ -112,8 +112,6 @@ class Admin_CacheController extends AuthenticatedController
         if (Config::get()->store('SYSTEMCACHE', $settings)) {
             PageLayout::postSuccess(_('Die Einstellungen wurden gespeichert.'));
             StudipCacheFactory::unconfigure();
-        } else {
-            PageLayout::postError(_('Die Einstellungen konnten nicht gespeichert werden.'));
         }
 
         $this->relocate('admin/cache/settings');

lib/classes/StudipMemcachedCache.php

@@ -46,7 +46,7 @@ class StudipMemcachedCache implements StudipSystemCache
 
         if (count($this->memcache->getServerList()) === 0) {
             foreach ($servers as $server) {
-                $status = $this->memcache->addServer($server['hostname'], $server['port']);
+                $status = $this->memcache->addServer($server['hostname'], (int) $server['port']);
 
                 if (!$status) {
                     throw new Exception("Could not add server: {$server['hostname']} @ port {$server['port']}");

resources/vue/components/MemcachedCacheConfig.vue

@@ -46,13 +46,17 @@ export default {
     },
     methods: {
         addServer () {
-            this.serverConfig.push({ server: '', port: null })
+            this.serverConfig.push({ hostname: 'localhost', port: 11211 })
         },
         removeServer (event, index) {
             this.serverConfig.splice(index, 1)
         },
         isValid () {
-            return this.serverConfig.length > 0;
+            return this.serverConfig.length > 0
+                && this.serverConfig.every(server => {
+                    return server.hostname.length > 0
+                        && server.port > 0;
+                });
         }
     },
     watch: {