During Drupalcon Austin session: Securing Your Drupal Site: Advice For Site Builders And Coders [0] @greggles tried to access the module's admin page from the module list page.

The configure property had not been set in the module's .info file.

[0] https://austin2014.drupal.org/session/securing-your-drupal-site-advice-s...

CommentFileSizeAuthor
#1 security_review-configuration-property-2279293-2.patch413 bytesdinarcon

Comments

dinarcon’s picture

dinarcon commented
Issue summary: View changes
StatusFileSize
new security_review-configuration-property-2279293-2.patch413 bytes

This patch adds the required property.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Awesome - thanks so much!

scor’s picture

scor commented
Status: Active » Reviewed & tested by the community

Looks good me, patch works. thanks @dinarcon! fast turn around while at DrupalCon :D

coltrane’s picture

coltrane
he/him
commented
Status: Reviewed & tested by the community » Fixed

Thanks @dinarcon for the patch and scor and greggles for review :) Committed! http://drupalcode.org/project/security_review.git/commit/16e7b72

  • Commit 16e7b72 on 7.x-1.x authored by dinarcon, committed by coltrane: 
    Issue #2279293 by dinarcon: Added Configuration page property in .info...

Status: Fixed » Closed (fixed)

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