There is a simple typo in the multivariate.module file that prevents administrators (other than user 1) from accessing /multivariate. The admin page checks for "administer multivariate studies" but permission checked is for "administer multivariate entities".

Will attach patch fix in comments.

CommentFileSizeAuthor
#2 multivariate-admin_permissions-1896828-1.patch2.92 KBpivica
#1 multivariate-admin_permissions-1896828.patch727 bytesdshumaker

Comments

dshumaker’s picture

dshumaker commented
Status: Active » Needs review
StatusFileSize
new multivariate-admin_permissions-1896828.patch727 bytes

Attached patch fix.

pivica’s picture

pivica commented
StatusFileSize
new multivariate-admin_permissions-1896828-1.patch2.92 KB

Thanks.

But 'administer multivariate studies' permissions is used on more places in code, for example in multivariate_access() callback also, and it is maybe more descriptive for the end user instead of 'administer multivariate entities'.

Attached patch is renaming all 'administer multivariate entities' to 'administer multivariate studies'. Can you please test - don't forget to run update.php because menu item access arguments are changes so we need to clear menu cache.

danillonunes’s picture

danillonunes commented
Status: Needs review » Reviewed & tested by the community

Patch from #2 tested and working fine. The only thing is that there's some misleading white spaces in the .install chunk.

pivica’s picture

pivica commented
Status: Reviewed & tested by the community » Fixed

Thx for review. Also fixed white spaces error.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.