|
Severin, I didn't find that lines in 1.9.
What is certain, is that calendar SHOULD NEVER REFER to assignement or any explicit module semantics. This a breakover on modularity and uncoupling. I suggest the best solution should be : When generating events in assessment, to twick the description there, and store a shadowed description in event table. The cron function for assessments shoud then scan for opening assessments, point again the calendar related events and restablish correct description. This is a first solution not using the event framework. There should be some using it, but I'm not sure there is a great movement for developping event approach. I would have liked to have MartinD actual position on the question... I think I can work about 15 minutes on the patch (without events). These lines are in 1.9.1...
Séverin Terrier made changes - 17/Jun/08 09:52 PM
I agree this is definitely not a nice hack, unfortunately it relies on other code from 1.9.x, backport is not trivial.
Anybody wants to work on this?
Petr Skoda made changes - 06/Jul/08 07:58 AM
Petr Skoda made changes - 09/Jul/08 10:12 PM
Is there some code I can manually plug into a php file for this bug? We are still using 1.8 and have instructors posting exams (don't ask....) in the Description field of the Advanced Upload of Files Assignment Activity. I thought we would upgrade to 1.9 this Fall so I wouldn't have to worry about this particular issue, but we have not upgraded.. sadly. (don' ask....)
Thanks in advance!!!
Martin Dougiamas made changes - 15/Oct/08 04:01 PM
Petr Skoda made changes - 22/Jan/09 08:08 AM
Martin Dougiamas made changes - 15/May/09 05:54 PM
Martin Dougiamas made changes - 26/Oct/09 12:16 PM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// TODO: rewrite this hack somehow
and
if ($assignmentinstance->description_is_hidden()){//force not to show description before availability
$event->description = get_string('notavailableyet', 'assignment');