Browse files

MDL-27896 editor - added sanity check to pick wrong context passed to…

… editor creation
  • Loading branch information...
1 parent b51aac3 commit 1449e4faa247b79295ba9040a955a77c46638103 Rajesh Taneja committed Jul 28, 2011
Showing with 14 additions and 0 deletions.
  1. +14 −0 lib/filelib.php
View
14 lib/filelib.php
@@ -111,6 +111,20 @@ function file_prepare_standard_editor($data, $field, array $options, $context=nu
$options['noclean'] = false;
}
+ //sanity check for passed context. This function doesn't expect $option['context'] to be set
+ //But this function is called before creating editor hence, this is one of the best places to check
+ //if context is used properly. This check notify developer that they missed passing context to editor.
+ if (isset($context) && !isset($options['context'])) {
+ //if $context is not null then make sure $option['context'] is also set.
+ debugging('Context for editor is not set in editoroptions. Hence editor will not respect editor filters', DEBUG_DEVELOPER);
+ } else if (isset($options['context']) && isset($context)) {
+ //If both are passed then they should be equal.
+ if ($options['context']->id != $context->id) {
+ $exceptionmsg = 'Editor context ['.$options['context']->id.'] is not equal to passed context ['.$context->id.']';
+ throw new coding_exception($exceptionmsg);
+ }
+ }
+
if (is_null($itemid) or is_null($context)) {
$contextid = null;
$itemid = null;

0 comments on commit 1449e4f

Please sign in to comment.