Skip to content

Commit

Permalink
MDL-18165 filterlib - Use one exclusive separator, instead of "." (do…
Browse files Browse the repository at this point in the history
…t) to avoid saved tags to match with phrases being processed ; merged from 19_STABLE
  • Loading branch information
stronk7 committed Feb 7, 2009
1 parent 7cde122 commit 60ff6ad
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions lib/filterlib.php
Expand Up @@ -53,6 +53,11 @@ public function hash() {
abstract function filter($text);
}

/// Define one exclusive separator that we'll use in the temp saved tags
/// keys. It must be something rare enough to avoid having matches with
/// filterobjects. MDL-18165
define ('EXCL_SEPARATOR', '-%-');

/**
* This is just a little object to define a phrase and some instructions
* for how to process it. Filters can create an array of these to pass
Expand Down Expand Up @@ -322,7 +327,7 @@ function filter_remove_duplicates($linkarray) {
/**
* Extract open/lose tags and their contents to avoid being processed by filters.
* Useful to extract pieces of code like <a>...</a> tags. It returns the text
* converted with some <#x.x#> codes replacing the extracted text. Such extracted
* converted with some <#xEXCL_SEPARATORx#> codes replacing the extracted text. Such extracted
* texts are returned in the ignoretags array (as values), with codes as keys.
*
* param text the text that we are filtering (in/out)
Expand All @@ -343,7 +348,7 @@ function filter_save_ignore_tags(&$text,$filterignoretagsopen,$filterignoretagsc
preg_match_all($pregexp, $text, $list_of_ignores);
foreach (array_unique($list_of_ignores[0]) as $key=>$value) {
$prefix = (string)(count($ignoretags) + 1);
$ignoretags['<#'.$prefix.'.'.$key.'#>'] = $value;
$ignoretags['<#'.$prefix.EXCL_SEPARATOR.$key.'#>'] = $value;
}
if (!empty($ignoretags)) {
$text = str_replace($ignoretags,array_keys($ignoretags),$text);
Expand All @@ -353,7 +358,7 @@ function filter_save_ignore_tags(&$text,$filterignoretagsopen,$filterignoretagsc

/**
* Extract tags (any text enclosed by < and > to avoid being processed by filters.
* It returns the text converted with some <%x.x%> codes replacing the extracted text. Such extracted
* It returns the text converted with some <%xEXCL_SEPARATORx%> codes replacing the extracted text. Such extracted
* texts are returned in the tags array (as values), with codes as keys.
*
* param text the text that we are filtering (in/out)
Expand All @@ -364,7 +369,7 @@ function filter_save_tags(&$text,&$tags) {
preg_match_all('/<([^#%*].*?)>/is',$text,$list_of_newtags);
foreach (array_unique($list_of_newtags[0]) as $ntkey=>$value) {
$prefix = (string)(count($tags) + 1);
$tags['<%'.$prefix.'.'.$ntkey.'%>'] = $value;
$tags['<%'.$prefix.EXCL_SEPARATOR.$ntkey.'%>'] = $value;
}
if (!empty($tags)) {
$text = str_replace($tags,array_keys($tags),$text);
Expand Down

0 comments on commit 60ff6ad

Please sign in to comment.