MDL-28298 workshop: do not use a constant without explicit inclusion of the library
authorDavid Mudrak <david@moodle.com>
Wed, 13 Jul 2011 08:28:28 +0000 (10:28 +0200)
committerDavid Mudrak <david@moodle.com>
Wed, 13 Jul 2011 08:28:28 +0000 (10:28 +0200)
Instead of including whole formslib just to get the constant defined, I
decided to hard-code the value and put the reference to the comment so
that eventual grep would spot it. Of course that is is not nice at all
but it's lesser of two evils imho.

mod/workshop/locallib.php

index 8f772e2..9aeaa83 100644 (file)
@@ -216,10 +216,11 @@ class workshop {
      * Returns an array of options for the editors that are used for submitting and assessing instructions
      *
      * @param stdClass $context
+     * @uses EDITOR_UNLIMITED_FILES hard-coded value for the 'maxfiles' option
      * @return array
      */
     public static function instruction_editors_options(stdclass $context) {
-        return array('subdirs' => 1, 'maxbytes' => 0, 'maxfiles' => EDITOR_UNLIMITED_FILES,
+        return array('subdirs' => 1, 'maxbytes' => 0, 'maxfiles' => -1,
                      'changeformat' => 1, 'context' => $context, 'noclean' => 1, 'trusttext' => 0);
     }