fix new semclass entries closes #1290

Closes #1290

Merge request studip/studip!808

1 files changed, 13 insertions, 2 deletions

diff --git a/lib/classes/SemClass.class.php b/lib/classes/SemClass.class.php
index 027d29c..5ab5a2d 100644
--- a/lib/classes/SemClass.class.php
+++ b/lib/classes/SemClass.class.php
@@ -52,7 +52,7 @@ class SemClass implements ArrayAccess
     static public function getDefaultSemClass() {
         $data = [
             'name' => "Fehlerhafte Seminarklasse!",
-            'modules' => '{"CoreOverview":{"activated":1,"sticky":1},"CoreAdmin":{"activated":1,"sticky":1}, "CoreResources":{"activated":1,"sticky":0}}',
+            'modules' => '{"CoreOverview":{"activated":1,"sticky":1},"CoreAdmin":{"activated":1,"sticky":1}}',
             'visible' => 1,
             'is_group' => false
         ];
@@ -115,11 +115,21 @@ class SemClass implements ArrayAccess
         } else {
             $this->data = $data;
         }
-        if ($this->data['modules']) {
+        if (!empty($this->data['modules'])) {
             $this->data['modules'] = self::object2array(json_decode($this->data['modules']));
+
         } else {
             $this->data['modules'] = [];
         }
+        if (!empty($this->data['studygroup_mode'])) {
+            if (!isset($this->data['modules']['CoreStudygroupAdmin'])) {
+                $this->data['modules']['CoreStudygroupAdmin'] = ['activated' => 1, 'sticky' => 1];
+            }
+        } else {
+            if (!isset($this->data['modules']['CoreAdmin'])) {
+                $this->data['modules']['CoreAdmin'] = ['activated' => 1, 'sticky' => 1];
+            }
+        }
         foreach (array_keys($this->data['modules']) as $modulename) {
             if ($this->isModuleForbidden($modulename)) {
                 unset($this->data['modules'][$modulename]);
@@ -592,6 +602,7 @@ class SemClass implements ArrayAccess
      */
     static public function refreshClasses()
     {
+        StudipCacheFactory::getCache()->expire('DB_SEM_CLASSES_ARRAY');
         self::$sem_classes = null;
         return self::getClasses();
     }