MDL-24542 filter_urltolink does not display debugging warning if called by format_string
authorDavid Mudrak <david.mudrak@gmail.com>
Thu, 7 Oct 2010 19:28:36 +0000 (19:28 +0000)
committerDavid Mudrak <david.mudrak@gmail.com>
Thu, 7 Oct 2010 19:28:36 +0000 (19:28 +0000)
Ideally, filter class should declare if it supports format_text() or
format_string() or both. Also there could be another $option passed
informing the filter about the type of the text.

filter/urltolink/filter.php

index f1ed7ae..72bb8b0 100644 (file)
@@ -46,7 +46,9 @@ class filter_urltolink extends moodle_text_filter {
      */
     public function filter($text, array $options = array()) {
         if (!isset($options['originalformat'])) {
-            debugging('filter_urltolink requires originalformat option to be provided', DEBUG_DEVELOPER);
+            // if the format is not specified, we are probably called by {@see format_string()}
+            // in that case, it would be dangerous to replace URL with the link because it could
+            // be stripped. therefore, we do nothing
             return $text;
         }
         if (in_array($options['originalformat'], explode(',', $this->get_global_config('formats')))) {