Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I have been working on a re-write of the question management tab for quizzes so it uses Drupal-native ajax in the form. I have created a patch of the changes and would like to submit it for review and committing if no bugs are found.
Comment | File | Size | Author |
---|---|---|---|
#2 | quiz_fapi_ajax.patch | 48.47 KB | orbmantell |
fapi_ajax.patch | 82.74 KB | orbmantell | |
Comments
Comment #1
Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commentedYour patch seems to remove the feature introduced in #1876104: Add option to mark a question as doubtful while answering. In brief it is doing something different from what has been mention in issue summary.
Comment #2
orbmantell CreditAttribution: orbmantell commentedI just re-wrote and re-rolled the patch... it should no longer delete functionality.
Comment #3
orbmantell CreditAttribution: orbmantell commentedIs there anything I can do to help get this reviewed?
Comment #4
orbmantell CreditAttribution: orbmantell commentedAccidentally closed the wrong issue
Comment #5
Sherbet CreditAttribution: Sherbet commentedI will try this out and will let you know how it fares. Will this also fix the issues I experienced here http://drupal.org/node/1861046 ?
Comment #6
orbmantell CreditAttribution: orbmantell commentedThis should fix any issues with the question admin page for a quiz.
I did simplify the form a bit (mainly the question search) because I was running into issues making all the old functionality work and I figured a simple working module is better than a fancy broken one. Once this is committed I can work on adding back in some of that functionality.
Comment #7
bpadaria CreditAttribution: bpadaria commentedNo questions are listed in "Manage Questions" tab.
Comment #8
djdevinFixed in 5.x.
Comment #9
falcon CreditAttribution: falcon commentedComment #12
djdevin