Dont copy blocktemplates to theme. Just having them in module will work

thomasbnielsen · Nov 18, 2016 · c19ba52 · c19ba52
1 parent ed4be72
commit c19ba52
Showing 1 changed file with 7 additions and 23 deletions.
diff --git a/code/dataobjects/Block.php b/code/dataobjects/Block.php
@@ -159,9 +159,13 @@ public function getCMSFields()
 
         // Template tab
         $optionset = array();
-        $theme = SSViewer::current_theme();
-        $src = BASE_PATH . "/themes/" . $theme . "/templates/BlockTemplates/";
-        $imgsrc = "/themes/" . $theme . "/templates/BlockTemplates/";
+        $theme = Config::inst()->get('SSViewer', 'theme');
+        $src = BASE_PATH . "/themes/" . $theme . "/templates/" . CONTENTBLOCKS_MODULE_DIR;
+        $imgsrc = "/themes/" . $theme . "/templates/" . CONTENTBLOCKS_MODULE_DIR;
+        if(!file_exists($src)){
+            $src = BASE_PATH . '/' . CONTENTBLOCKS_MODULE_DIR . '/templates/' . CONTENTBLOCKS_TEMPLATE_DIR . '/';
+            $imgsrc = '/' . CONTENTBLOCKS_MODULE_DIR . '/templates/' . CONTENTBLOCKS_TEMPLATE_DIR . '/';
+        }
 
         // TODO: If ClassName == Block, return the templates of the folder.
         // If ClassName is something else (extension of block) then see if there is a folder with that name and only return templates from this folder
@@ -285,26 +289,6 @@ public function onBeforeDelete()
         $this->Images()->removeAll();
     }
 
-    function requireDefaultRecords()
-    {
-        parent::requireDefaultRecords();
-        // Run on dev build	- move to module file or why is it here?
-
-        // If templates does not exist on current theme, copy from module
-        $theme = SSViewer::current_theme();
-        $copyto = "../themes/" . $theme . "/templates/" . CONTENTBLOCKS_TEMPLATE_DIR . "/";
-
-        if (!file_exists($copyto)) {
-            $copyfrom = BASE_PATH . "/" . CONTENTBLOCKS_MODULE_DIR . "/templates/" . CONTENTBLOCKS_TEMPLATE_DIR . "/";
-            if (file_exists($copyfrom)) {
-                $this->recurse_copy($copyfrom, $copyto);
-                echo '<li style="color: green">BlockTemplates copied to: ' . $copyto . '</li>';
-            } else {
-                echo "The default template archive was not found: " . $copyfrom;
-            }
-        }
-    }
-
     // Should only unlink if a block is on more than one page
     public function canDelete($member = null)
     {