search

3ilogiclms / moodle-block_learning_plan

This plugin serves as a database and plan for all learning activities in the organziation, where such activities are organized for a more structured learning program.
8 stars 12 forks source link

Some view errors #2

Closed negations closed 10 years ago

negations commented 11 years ago

The block now loads on a 2.5.2 site, but shows a few errors with debug on. They don't seem to impact the functions, but putting them here for reference:

Notice: Undefined variable: row in /var/www/site11/blocks/learning_plan/view.php on line 58

Warning: Invalid argument supplied for foreach() in /var/www/site11/lib/outputcomponents.php on line 1554

Notice: Undefined variable: errors in /var/www/site11/blocks/learning_plan/learning_plan_form.php on line 35

Notice: Undefined variable: setting in /var/www/site11/blocks/learning_plan/view.php on line 240

On creating and trying to save a plan:

I get the same notice and warning as above (line 58 and 1554) and also:

Notice: Undefined variable: errors in /var/www/site11/blocks/learning_plan/learning_plan_form.php on line 35 You should really redirect before you start page output line 741 of /lib/outputrenderers.php: call to debugging() line 2557 of /lib/weblib.php: call to core_renderer->redirect_message() line 72 of /blocks/learning_plan/view.php: call to redirect()

and the system then gives me:

Error output, so disabling automatic redirect.

As well as:

Coding problem: $PAGE->context was not set. You may have forgotten to call require_login() or $PAGE->set_context(). The page may not display correctly as a result line 451 of /lib/pagelib.php: call to debugging() line 734 of /lib/pagelib.php: call to moodle_page->magic_get_context() line 1311 of /lib/weblib.php: call to moodle_page->__get() line 1103 of /lib/pagelib.php: call to format_string() line 39 of /blocks/learning_plan/view.php: call to moodle_page->set_heading()

Similar messages appear for all of the functions.

3ilogiclms commented 10 years ago

Thanks negation for raising this issues, we have removed all notices & warnings.