FIX: fix registering service providers in templates

app/CMS/Helpers/PageRenderer.php

@@ -1,37 +0,0 @@
-<?php
-namespace App\CMS\Helpers;
-
-use App\CMS\Handlers\Contracts\AppHandler;
-use App\Models\Content\Content;
-use Webwizo\Shortcodes\Facades\Shortcode;
-
-/**
- * Class PageRenderer
- *
- * Simple class for rendering page for visitor
- *
- * @package App\CMS\Helpers
- */
-class PageRenderer
-{
-    /**
-     * Render page
-     *
-     * @param Content $page
-     *
-     * @return mixed
-     */
-    public static function render(Content $page)
-    {
-        // do something template-specific
-        $parameters = call_user_func_array([CurrentTemplate::handler(), 'beforeRender'], [&$page]);
-
-        /**
-         * @var $handler AppHandler
-         */
-        $handler = new $page->moduleHandler;
-
-        // render page
-        return $handler->render($page, $parameters);
-    }
-}

app/CMS/ModuleManager/Providers/ModuleManager.php

@@ -16,7 +16,7 @@ class ModuleManager extends ServiceProvider
     {
         // this service provider needs published config - config is merged right after this service provider
         try {
-            $modules = app(ModuleRepository::class)->getOnlyModules();
+            $modules = app(ModuleRepository::class)->all();
         } catch (QueryException $e) {
             return;
         }
@@ -32,7 +32,13 @@ class ModuleManager extends ServiceProvider
             // include files (if any)
             if (isset($module->module_settings['files']) && !empty($module->module_settings['files'])) {
                 foreach ($module->module_settings['files'] as $file) {
-                    include config('modules.paths.modules') . '/' . $module->static_name . '/' . $file;
+                    $path = config('modules.paths.modules') . '/' . $module->static_name . '/' . $file;
+
+                    if (!file_exists($path)) {
+                        $path = base_path($file);
+                    }
+
+                    @include_once $path;
                 }
             }
         }

app/Http/Controllers/Controller.php

@@ -14,7 +14,6 @@ class Controller extends BaseController
 
     /**
      * Custom wrapper for generating view.
-     *
      * This method will be used for switching among templates.
      *
      * @param       $template
@@ -25,8 +24,23 @@ class Controller extends BaseController
      */
     public static function visitorsView($template, $data = [], $mergeData = [])
     {
-        $view = Template::name($template);
+        $view = self::getView($template, $data, $mergeData);
 
-        return response(view($view, $data, $mergeData))->header('X-Powered-By', strrev('zc.ytiranib.www//:sptth | .o.r.s ytiranib | SMCiniM'));
+        return response($view)->header('X-Powered-By', strrev('zc.ytiranib.www//:sptth | .o.r.s ytiranib | SMCiniM'));
+    }
+
+    /**
+     * Get instance of view for passing into response
+     *
+     * @param $template
+     * @param $data
+     * @param $mergeData
+     *
+     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
+     */
+    public static function getView($template, $data = [], $mergeData = [])
+    {
+        $view = Template::name($template);
+        return view($view, $data, $mergeData);
     }
 }

app/Providers/TemplateServiceProvider.php

@@ -27,12 +27,7 @@ class TemplateServiceProvider extends ServiceProvider
             return;
         }
 
-        // include files
+       // service providers and files already registered in ModuleManager
-        $files = isset($template->moduleSettings->files) ? $template->moduleSettings->files : [];
-
-        foreach ($files as $file) {
-            include_once $file;
-        }
 
         // register templates
         $publicTemplates = CurrentTemplate::getPublicTemplatePaths();