Issue Details (XML | Word | Printable)

Key: MDL-15206
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Tim Hunt
Reporter: Nicolas Connault
Votes: 2
Watchers: 5
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Essay question type broken

Created: 11/Jun/08 04:34 PM   Updated: 17/Jun/08 10:51 AM
Return to search
Component/s: Questions
Affects Version/s: 1.9.1
Fix Version/s: 1.9.2, 2.0

Participants: Jacques Monnard, Jerome Mouneyrac, Nicolas Connault, Skip Marshall and Tim Hunt
Security Level: None
QA Assignee: Jerome Mouneyrac
Resolved date: 13/Jun/08
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
Create an essay question with no feedback or general feedback. Previewing or attempting the quiz leads to a blank page. PHP error indicates:

Fatal error: Call to undefined method question_essay_qtype::extra_answer_fields() in /srv/www/htdocs/moodle_19/question/type/questiontype.php on line 440

extra_answer_fields() is NOT defined anywhere in this class or in any of its children classes. extra_question_fields(), however, is defined.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Jacques Monnard added a comment - 12/Jun/08 08:42 PM
I just noticed this bug as well when doing a backup of a course with quizzes.
Apparently, the following three lines were deleted by mistake at line 104 in question/type/questiontype.php :

function extra_answer_fields() { return null; }

I've just added them back in, and everything seems to work fine.


Skip Marshall added a comment - 13/Jun/08 04:16 AM
Nice catch. This resolved the problem.

Tim Hunt added a comment - 13/Jun/08 12:16 PM
Nicholas, you have a comment above saying 'fixed', but the bug is not actually resolved as fixed. Then there are the other comments. So I am confused. Can you help me. Thanks.

Nicolas Connault added a comment - 13/Jun/08 12:23 PM
Yes Tim, I fixed it in HEAD but forgot 1.9. I am closing this now

Jerome Mouneyrac added a comment - 17/Jun/08 10:51 AM
tested in 1.9. It works. Thanks.