Multiple fixes

Hi amagdy,

The best way to grab the attention of the maintainers is to visit http://drupal.org/project/quiz and on the right side 'Maintainers for Quiz' section you can see a list of developers. You can view their personal details and send them a mail about extending your help for module development.
According to me you have done enough to get the attention from the maintainers. By the way your patch works fine once again:)

...will this patch resolve the issue in http://drupal.org/node/1480128

The patches appeared to be against the most recent dev release on the site (2012-Mar-03). I applied them and they seem to be working...at least I'm not receiving the PDOException. I hope the code/patch is in the next release.

Special Thanks amagdy!

there is a small mistake in the patchfile (git complains on patching because one header is missing) but as a far s i only use the multiplechoice question type it works anyway afterwards

Same problem:

 patch -p1 < quiz.patch
patching file question_types/multichoice/multichoice.classes.inc
patching file question_types/quiz_question/quiz_question.module
Hunk #1 FAILED at 97.
1 out of 1 hunk FAILED -- saving rejects to file question_types/quiz_question/quiz_question.module.rej
patching file quiz.module

So this patch doesn't work with current dev version? (7.x-4.x-dev 2012-Mar-03)

Actually when I patch the module through Netbeans, it only applies to one file, but the others are skipped.

I don't know if I can patch them manually, since I don't know what changes have been made since the previous dev release.

Hi there,
please look here for manual patching. It works, if you are careful.

I've rerolled the patch against the latest -dev. It definitely makes a couple of errors go away. Haven't looked closely at the changes, but it seems like it should probably be committed.

+++ question_types/quiz_question/quiz_question.module
@@ -97,15 +97,16 @@ function quiz_question_node_access($node, $op, $account) {
-      if (user_access('edit any quiz', $account) || (user_access('edit own quiz', $account) && ($account->uid == $node->uid))) {
+      if (user_access('edit any quiz content', $account) ¶

trailing white space

+++ quiz.module
@@ -3048,7 +3046,8 @@ function _quiz_resume_existing_quiz($quiz, $uid, $rid) {
-        if (!in_array($question['vid'], $already_answered) && $question['random'] === TRUE) {
+      	// checks if question[random] is set
+      	if (!in_array($question['vid'], $already_answered) && isset($question['random']) && $question['random'] === TRUE) {

use spaces for indentation.

Can you create a working patch out of it then klausi ? Current patch (#14) gives errors...

Ok, thank you for sharing. I look forward for this patch.

As am I, Quiz is quite a crucial part of my website as well.

Patch has been commited. Thanks!

Great!

Thanks to all of you!