Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

General Questions /

General questions about getting started with SilverStripe that don't fit in any of the categories above.

Moderators: martimiz, Sean, biapar, Willr, Ingo, swaiba, simon_w

Possible to use Requirements::combine_files in theme files?


Reply


2 Posts   475 Views

Avatar
svandragt

Community Member, 44 Posts

13 July 2012 at 3:15am

I'm just starting my first SS site and I am adapting the simple theme that comes with SS3. I want to replace the 4 require themecss calls with a single Requirements::combine_files call, to minimize the amount of requests on my live site.

It works when I add the following code to the Page_Controller init() method:

      // combine css files
      Requirements::combine_files(
         'all.css',
         array(
            THEMES_DIR . '/simple/css/reset.css',
            THEMES_DIR . '/simple/css/typography.css',
            THEMES_DIR . '/simple/css/form.css',
            THEMES_DIR . '/simple/css/layout.css',
         )
      );

However there is a note there that says:
      // Note: you should use SS template require tags inside your templates
      // instead of putting Requirements calls here. However these are
      // included so that our older themes still work

When I put the call directly in the Page.ss theme file however the site won't load.
As I want to make sure I'm future proofing my learning is there any guidance on how I have to change the call when putting it in Page.ss if this is possible? Thanks.

Avatar
svandragt

Community Member, 44 Posts

21 July 2014 at 9:29pm

I noticed this thread hasn't had any replies. For the uninitiated: you are better off to use the controller's init() method to add-in the code.