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

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

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:

Status:Active» Needs review

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

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.

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?

Issue tags:+7.x-1.6

Tagging.

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.

(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 )

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?

Thanks. This patch worked to remove the error display.

Status:Needs review» Fixed

Status:Fixed» Closed (fixed)

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

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...