When Status Report viewed (admin/reports/status) error below is shown.

Undefined index: value in theme_status_report() (line 2501 of /..../modules/system/system.admin.inc)
when

Described at http://drupal.org/node/1063542 but apparently never reported here.

Comments

Jean Gionet’s picture

I'm getting the thing using this module.
Any plans for the fix/update? thxs

mesr01’s picture

StatusFileSize
new533 bytes

The system_requirements($variable) hook, in file aes.install returns an array that is lacking the 'value' index. For those interested, here's a very simple fix:

EvanDonovan’s picture

Status:Active» Needs review

If there's a patch, set to needs review.

tstoeckler’s picture

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

This fixes the notice, so I'm setting this to RTBC.
The whole function (aes_requirements()) could use a fair amount of clean-up, but not sure that has to be done in this issue. The patch in itself in consistent with the rest of the function.

EvanDonovan’s picture

I'm in #drupal-contribute right now & want to work on fixing all issues with this module. (I am the current maintainer, though.) Care to find me in there?

EvanDonovan’s picture

Issue tags:+7.x-1.6

Tagging.

jurgenhaas’s picture

Status:Reviewed & tested by the community» Needs review
StatusFileSize
new983 bytes

Just thought that the value could be a bit more descriptive and tell the admin, what's really available and installed. Hope you don't mind that I reset the status to needs review.

Rob C’s picture

(hope this helps a bit and hope the patch is proper this way)
- Fix the outstanding error because of the array() issue, see code (this will fix the error)
- Added everything from #7 (Cause that's interesting, might have to change that later on, to reflect all available anyhow) and added the error message if NONE are enabled
- Added support for $t() on install
( Edit > fix typo )

ambitioustyphoon’s picture

Hi,

I just tried the patch above and received this message:

$ git apply -v aespatch.patch
aespatch.patch:10: trailing whitespace.
// make sure translations won't break on install
aespatch.patch:11: trailing whitespace.
$t = get_t();
aespatch.patch:16: trailing whitespace.
$value = $t('The AES encryption module requires at least one of two things to function: Either the PHP Mcrypt extension has to be installed on the web server. Or the PHP Secure Communications Library (phpseclib) needs to be installed in the AES directory. Check the README.txt for more information.');
aespatch.patch:20: trailing whitespace.
if ($aes_implementations['mcrypt'] && $aes_implementations['phpseclib']) {
aespatch.patch:21: trailing whitespace.
$value = t('Both MCrypt and PHP Secure Communications Library are available.');
fatal: corrupt patch at line 43

Could I be applying it wrong or could there be an issue with the patch?

rtdean93’s picture

Thanks. This patch worked to remove the error display.

EvanDonovan’s picture

Status:Needs review» Fixed

Status:Fixed» Closed (fixed)

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

Motlem’s picture

Rob C, your patch did the trick. I tip my hat to you sir.

  • Commit 5e02b48 on 7.x-1.x, 8.x-2.x by EvanDonovan:
    Issue #1139688 by mesr01, ClusterFCK, tsoeckler, EvanDonovan: Fix for...