diff -u b/security_review.site_audit.inc b/security_review.site_audit.inc --- b/security_review.site_audit.inc +++ b/security_review.site_audit.inc @@ -49,7 +49,7 @@ } } } - else { + elseif ($this->registry[$this->module][$this->check]['value']) { $ret_val .= ' Additional: "' . $this->registry[$this->module][$this->check]['value'] . '"'; } } @@ -59,8 +59,7 @@ /** * Implements \SiteAudit\Check\Abstract\getResultInfo(). */ - public function getResultInfo() { - } + public function getResultInfo() {} /** * Implements \SiteAudit\Check\Abstract\getResultPass(). @@ -72,14 +71,12 @@ /** * Implements \SiteAudit\Check\Abstract\getResultWarn(). */ - public function getResultWarn() { - } + public function getResultWarn() {} /** * Implements \SiteAudit\Check\Abstract\getAction(). */ - public function getAction() { - } + public function getAction() {} /** * Implements \SiteAudit\Check\Abstract\calculateScore(). @@ -102,6 +99,13 @@ class SiteAuditCheckSecurityFilePerms extends SecurityReviewSiteAuditCheckAbstract { protected $check = 'file_perms'; + + public function getResultFail() { + if (drush_get_option('detail')) { + return parent::getResultFail(); + } + return $this->registry[$this->module][$this->check]['failure']; + } } class SiteAuditCheckSecurityInputFormats extends SecurityReviewSiteAuditCheckAbstract {