diff options
| author | Michael Hamann <michael@content-space.de> | 2010-11-26 16:54:51 +0100 |
|---|---|---|
| committer | Michael Hamann <michael@content-space.de> | 2010-11-26 17:02:33 +0100 |
| commit | 4871414204799044c31aa2764c4b4ca020e2331d (patch) | |
| tree | 45f1da8ce8c6a99678a395c576e0f0d6cb9ded11 /inc | |
| parent | d1d99bb9528b5a7ca62b74e60314288602cbd5a0 (diff) | |
| download | rpg-4871414204799044c31aa2764c4b4ca020e2331d.tar.gz rpg-4871414204799044c31aa2764c4b4ca020e2331d.tar.bz2 | |
Fix for $conf['breadcrumbs'] < 0, FS#2107
This fixes an infinite loop in breadcrumbs() and makes the behaviors in
all places where breadcrumbs are used consistent so that non-numeric
values, values < 0 and 0 are treated the same way.
Diffstat (limited to 'inc')
| -rw-r--r-- | inc/common.php | 3 | ||||
| -rw-r--r-- | inc/template.php | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/inc/common.php b/inc/common.php index 18f782788..667846804 100644 --- a/inc/common.php +++ b/inc/common.php @@ -268,6 +268,9 @@ function breadcrumbs(){ global $ACT; global $conf; + // Prevent infinite loop later in this function + if (!is_numeric($conf['breadcrumbs']) || $conf['breadcrumbs'] <= 0) return array(); + //first visit? $crumbs = isset($_SESSION[DOKU_COOKIE]['bc']) ? $_SESSION[DOKU_COOKIE]['bc'] : array(); //we only save on show and existing wiki documents diff --git a/inc/template.php b/inc/template.php index e2ea6e386..b89f7abbc 100644 --- a/inc/template.php +++ b/inc/template.php @@ -690,7 +690,7 @@ function tpl_breadcrumbs($sep='»'){ global $conf; //check if enabled - if(!$conf['breadcrumbs']) return false; + if(!is_numeric($conf['breadcrumbs']) || $conf['breadcrumbs'] <= 0) return false; $crumbs = breadcrumbs(); //setup crumb trace |