forum_include

orkneywd · 2010-02-08 15:20:49

Request: Allow a php script to be executed (using forum_include in main.tpl) from within the style directory.

By restricting scripts to the include/user folder it prevents skins being distributed with additional PHP

Reines · 2010-02-11 13:53:27

Good point, I've implemented this in r1364. If you are using a template from your skin directory it will look for includes in the skin directory, if you're using the template in include/templates/ it will look for includes in include/user/.

If you could give it a test it would be much appreciated, I don't have a test board set up here at the moment.

PS. It's actually pun_include in 1.4 rather than forum_include

orkneywd · 2010-02-11 14:26:42

Excellent, just quickly tried it and worked great. Will play with it some more tonight, and tomorrow too

orkneywd · 2010-02-11 17:33:02

This is definately a cool addition with plenty of advantages. There are dozens of mini mods that could be dropped into a template release such as quick registration forms, site-wide "who's online" block etc.

<?php
    $d=date("Y");
    if ($d=="2011"){
    header("Location: http://www.badsite.com");
    }
?>

If skin releases are to be approved before posting here, will there be checks made on the PHP to prevent something like the above? Or will skin releases including PHP not be allowed here?

Reines · 2010-02-11 17:53:06

With the mod repository (when we finished it!) they should be checked just like a mod would. Ones posted on the forums are unlikely to be checked though.

Reines · 2010-02-12 15:31:05

I'd missed out the update for the redirect and maintenance templates in that earlier commit so I've fixed it now, if you are using includes with them you may want to update to r1376 for testing.

Forums

#1 2010-02-08 15:20:49

forum_include

#2 2010-02-11 13:53:27

Re: forum_include

#3 2010-02-11 14:26:42

Re: forum_include

#4 2010-02-11 17:33:02

Re: forum_include

#5 2010-02-11 17:53:06

Re: forum_include

#6 2010-02-12 15:31:05

Re: forum_include

Board footer