Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#8 | aes-more-requirements-fixes-issue-1139688-3.patch | 1.7 KB | Rob C |
#7 | aes-1139688.patch | 983 bytes | jurgenhaas |
#2 | aes_requirements_fix.patch | 533 bytes | mesr01 |
Comments
Comment #1
Jean Gionet CreditAttribution: Jean Gionet commentedI'm getting the thing using this module.
Any plans for the fix/update? thxs
Comment #2
mesr01 CreditAttribution: mesr01 commentedThe
system_requirements($variable)
hook, in fileaes.install
returns an array that is lacking the'value'
index. For those interested, here's a very simple fix:Comment #3
EvanDonovan CreditAttribution: EvanDonovan commentedIf there's a patch, set to needs review.
Comment #4
tstoecklerThis 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.
Comment #5
EvanDonovan CreditAttribution: EvanDonovan commentedI'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?
Comment #6
EvanDonovan CreditAttribution: EvanDonovan commentedTagging.
Comment #7
jurgenhaasJust 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.
Comment #8
Rob C CreditAttribution: Rob C commented(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 )
Comment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedHi,
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?
Comment #10
rtdean93 CreditAttribution: rtdean93 commentedThanks. This patch worked to remove the error display.
Comment #11
EvanDonovan CreditAttribution: EvanDonovan commentedCommitted in -dev: http://drupalcode.org/project/aes.git/commit/5e02b48
Comment #13
Motlem CreditAttribution: Motlem commentedRob C, your patch did the trick. I tip my hat to you sir.