Notices like this show up:

Notice: Undefined index: value in theme_status_report() (line 2552 of /srv/http/XXXX/www/modules/system/system.admin.inc).

Patch attached in comment.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kotnik’s picture

Status: Active » Needs review
FileSize
3.77 KB

Patch with fix attached.

gmclelland’s picture

The patch in #1 fixed the problem notice in my install profile. Thanks

jsacksick’s picture

Ok according to the code in system.admin.inc, you're right, a check is done on the description but not on the value...


      // Output table row(s)
      if (!empty($requirement['description'])) {
        $output .= '<tr class="' . $severity['class'] . ' merge-down"><td class="status-icon">' . $severity['icon'] . '</td><td class="status-title">' . $requirement['title'] . '</td><td class="status-value">' . $requirement['value'] . '</td></tr>';
        $output .= '<tr class="' . $severity['class'] . ' merge-up"><td colspan="3" class="status-description">' . $requirement['description'] . '</td></tr>';
      }
      else {
        $output .= '<tr class="' . $severity['class'] . '"><td class="status-icon">' . $severity['icon'] . '</td><td class="status-title">' . $requirement['title'] . '</td><td class="status-value">' . $requirement['value'] . '</td></tr>';
      }

Can you just keep the description for the first message ?

kotnik’s picture

Kept the description for the first message. Patch attached.

jsacksick’s picture

Version: 7.x-1.0-rc1 » 7.x-1.x-dev
Status: Needs review » Fixed

Committed ! thanks

Status: Fixed » Closed (fixed)

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