Quiz module configuration forms are located in path admin/quiz/* while the recommended path is admin/config/quiz/*. Ideally it should be under one of the existing categories, could be "Content authoring" or it can define its own (like admin menu module).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

liquidcms’s picture

maybe quiz results should be under reports; not sure what quiz score short answer questions is

and rest should definitely be under admin/config

liquidcms’s picture

and i think this fixes.. but against 4.x-dev

liquidcms’s picture

Issue summary: View changes

Added appropriate path to description.

jhedstrom’s picture

Issue summary: View changes
FileSize
15.33 KB

I couldn't get the patch in #2 to apply, but this is basically the same idea with a few tweaks to shorten overly redundant paths (admin/config/content/quiz instead of admin/config/content/quiz/settings).

jhedstrom’s picture

Status: Active » Needs review
mikran’s picture

Status: Needs review » Needs work

one minor thing

+++ b/includes/quiz_stats/quiz_stats.module
@@ -35,7 +35,7 @@ function quiz_stats_permission() {
+  $items['admin//reportsquiz/stats/creator'] = array(

That can't be right

jhedstrom’s picture

Status: Needs work » Needs review
FileSize
588 bytes
15.33 KB

This fixes the error in #5.

mikran’s picture

Status: Needs review » Reviewed & tested by the community

It works

mikran’s picture

Version: 7.x-5.x-dev » 7.x-4.x-dev

Ohh and the patch was against 4.x branch ofc

nielsonm’s picture

Rolled a patch for 5.x.

nielsonm’s picture

Version: 7.x-4.x-dev » 7.x-5.x-dev
Status: Reviewed & tested by the community » Needs review

The last submitted patch, 2: quiz-proper_admin_menu_location.patch, failed testing.

The last submitted patch, 3: quiz-admin-menu-cleanup-1257710-03.patch, failed testing.

The last submitted patch, 6: quiz-admin-paths-1257710-06.patch, failed testing.

djdevin’s picture

Status: Needs review » Active
djdevin’s picture

Status: Active » Needs review
nielsonm’s picture

Reuploading the 5.x patch per @djdevin's suggestion.