Wen Hao Chuang added a comment - 05/Oct/07 02:29 AM

Wen Hao Chuang added a comment - 05/Oct/07 02:29 AM another quick fix. The string should go into /lang/en_utf8/admin.php (based on my understanding of moodle convention). Just added the string based on moodle convention and got it committed into 1.8_STABLE, will commit it to HEAD and other branches later too.

Howard Miller added a comment - 05/Oct/07 04:31 PM

Howard Miller added a comment - 05/Oct/07 04:31 PM I was kinda hoping for an error message that described what the problem might be. Ordinarily I would just have added the string myself without causing "tracker noise", but I had no idea what the error meant.

Wen Hao Chuang added a comment - 06/Oct/07 02:06 AM

Wen Hao Chuang added a comment - 06/Oct/07 02:06 AM Sorry Howard if I seems to be rude to jump right in to solve the problem right away, my apologies. I'm still trying to get myself familiar with the workflow here on the moodle bug tracker so please bear with me... (newbie here!) By the way I think for this particular error, it usually throw out from from admin/settings.php (please correct me if I'm wrong).. ---- line #40 $adminroot = admin_get_root(); $root = $adminroot->locate($PAGE->section); if (!is_a($root, 'admin_settingpage')) { error(get_string('sectionerror', 'admin')); die; } if (!($root->check_access())) { error(get_string('accessdenied', 'admin')); die; } $CFG->pagepath = 'admin/setting/'.$section;

---

Actually if you directly point your local browser to http://localhost/moodle18/moodle/admin/settings.php you could get this error message right away. It probably could happen with different scenario so don't know if we could be more descriptive about it. One scenario (like directly access that page with teacher permission) would be similar to the error message that you would be getting (login as instructor while you don't have permissions to directly access /admin/settings.php page) for access /admin/ folder: Sorry, but you do not currently have permissions to do that (Change site configuration) (see screenshot) But then this is just one scenario, there should be other scenarios I think.. so maybe just throw out a "Section Error!" message for now until we could find a better alternative? Again just my 2c

Petr Skoda added a comment - 20/Dec/07 03:10 AM

Petr Skoda added a comment - 20/Dec/07 03:10 AM hi, thanks for the fix this error is thrown if the url contains no or wrong section name - there is not much we can do with that info, please reopen if you have any idea how to improve it I have also added link to site/admin/ into the error() call

Nicolas Martignoni added a comment - 09/Feb/08 05:38 AM

Nicolas Martignoni added a comment - 09/Feb/08 05:38 AM Verified, closing.