Closed (fixed)
Project:
Quiz
Version:
7.x-4.x-dev
Component:
Code - Short/long answer
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Jan 2013 at 23:23 UTC
Updated:
22 Jun 2014 at 16:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sivaji_ganesh_jojodae commented+1 for committing this patch. But falcon is trying to limit the new features request as to make the quiz 7.x stable sooner, so i'm leaving this issue at his discretion.
Did a quick review of patch, I'm sure it needs work.
1. Two return?
2. $form['#validate'] must be array string given. ($form['#validate'][] = 'long_answer_config_validate';)
Comment #2
michaelk commentedOh, yep, the double return was just a copy/paste error when I was generating the patch off of our live code.
The $form['#validate'] was just the same code used in the multichoice question code for its configuration section.
I went ahead and made both changes, and have attached an updated version of the patch.
Comment #3
sivaji_ganesh_jojodae commentedSince this is not a show stopper i committed this patch to git.