Non-core contributed modules

More than one area, text string "Computer Rooms" is in the drob down list

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Not a bug
  • Affects Version/s: 1.9, 1.9.1, 1.9.2, 1.9.3, 1.9.4
  • Fix Version/s: None
  • Component/s: Block: Mrbs
  • Labels:
    None
  • Environment:
    Moodle 1.9.5+ (Build: 20090617)
    MRBS (Build: 2009060100)
  • Database:
    MySQL
  • Difficulty:
    Difficult
  • Affected Branches:
    MOODLE_19_STABLE

Description

Hello,
we have the problem when you create more then one area in the mrbs block with rooms in this area, there is a entry "Computer Room" in the drop down list. If you choose this wrong entry the system makes funny things - when you the try to look at a other area - then there are rooms of this area missing.
We have 1000 Moodle Systems with this problem.

Activity

Hide
Anthony Borrow added a comment -

Mike - I am assigning this to you since it appears to be a feature. It seems it may be causing some confusion so it may be best to either provide an option for it or turn it off. Peace - Anthony

Show
Anthony Borrow added a comment - Mike - I am assigning this to you since it appears to be a feature. It seems it may be causing some confusion so it may be best to either provide an option for it or turn it off. Peace - Anthony
Hide
Anthony Borrow added a comment -

Jens - You may want to simply comment out the line (~416) in edit_entry.php

print "this.document.writeln(\" <option value=\\\"IT\\\">".get_string('computerrooms','block_mrbs')."\")\n";

That should resolve the issue for you until Mike gets a chance to explain the intention of the "special IT option" he comments in the code.

Peace - Anthony

Show
Anthony Borrow added a comment - Jens - You may want to simply comment out the line (~416) in edit_entry.php print "this.document.writeln(\" <option value=\\\"IT\\\">".get_string('computerrooms','block_mrbs')."\")\n"; That should resolve the issue for you until Mike gets a chance to explain the intention of the "special IT option" he comments in the code. Peace - Anthony
Hide
Jens-Peter misch added a comment -

Anthony - thank you very much for your answer...I will change our code at the weekend...

Show
Jens-Peter misch added a comment - Anthony - thank you very much for your answer...I will change our code at the weekend...
Hide
Anthony Borrow added a comment -

I am resolving this as not a but. The issue seems to be resolved and it appeared to be an intentional feature that Mike Worth added. Peace - Anthony

Show
Anthony Borrow added a comment - I am resolving this as not a but. The issue seems to be resolved and it appeared to be an intentional feature that Mike Worth added. Peace - Anthony

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: