00978: Charset bug in page text variables.

Summary: Charset bug in page text variables.
Created: 2007-09-20 15:51
Status: Closed, fixed for 2.2.44
Category: Bug
From: swestrup
Assigned:
Priority: 3
Version: 2.2.0-beta63
OS: Apache/Linux/4.x

Description: Page text variables, as indicated in the $PageTextPatterns regexes, are allowed to contain '-' in their names, thusly:

$PageTextVarPatterns = array
( 'var:'        => '/^(:*\\s*(\\w[-\\w]*)\\s*:[ \\t]?)(.*)($)/m'
, '(:var:...:)' => '/(\\(: *(\\w[-\\w]*) *:(?!\\))\\s?)(.*?)(:\\))/s'
);

But the markup that detects variable's doesn't allow that:

# {$var} substitutions
Markup('{$var}', '>$[phrase]',
   '/\\{(\\*|!?[-\\w.\\/\\x80-\\xff]*)(\\$:?\\w+)\\}/e',

I recommend that the latter markup be extended to cover the same character set as the former. At the very least, they should both accept the same set of chars.

Should be fixed at the same time as 01272. --Petko November 23, 2011, at 07:04 PM