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

It is valid to not have a value for 'value' (the docs even say do not set if not applicable), so I think it's existence should be checked first.
It is in other places like drupal_check_module().

Yep. Solves the issue and applies cleanly.

Oops, wrong version. Here is a drupal 8 one that is the same.

Applied patch in #1 onto D7.23 and the error went away.

+++ b/core/modules/system/system.admin.incundefined
@@ -587,6 +587,8 @@ function theme_status_report($variables) {
+    $requirement['value'] = isset($requirement['value']) ? $requirement['value'] : '';

Hmmm using a ternary operator here seems unnecessary bearing in mind http://fabien.potencier.org/article/48/the-php-ternary-operator-fast-or-not and we're just assigning the same value to itself which seems clunky.

How about we just change

    $output .= '<td class="status-value">' . $requirement['value'];

to

    $output .= '<td class="status-value">';
    if (!empty($requirement['value'])) {
      $output .=  $requirement['value'];
    }

IMHO this is small economy. With the first option code more readable.
At this time, right patch for 7.23

I added this at like 2583 of system.admin.inc. There is a reason why this solution wont work? I mean besides the fact that is a core hack?

if(!isset($requirement['value'])){
		array_push($requirement, "value");
		$requirement['value']="";
	 }

Current error in Drupal 7.26:

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

HAPPY BELATED BIRTHDAY to this issue! ;)

I'll try applying #1 again and if it has issues I'll try #9. Not sure why there's another patch for the same version of Drupal (D.7.23).

The two patches on this issue have a variance where a line is commented or not.

1880224-fix-notice-for-missing-value-on-status-report-page-1.patch

$requirement['value'] = isset($requirement['value']) ? $requirement['value'] : '';

drupal-status_report_undefined_index_value-1880224-9.patch

//$requirement['value'] = isset($requirement['value']) ? $requirement['value'] : '';

Which is correct?

The patch without commenting (//) works for D7.26.

Oh, this is right, i'm not delete comment :(

Patch in #1 works for D7.28 and has worked for almost a year and half but is still not committed to Drupal 7 as of D7.28.

Marking this as RTBC to get some attention since this is lingering.

I cannot verify whether the patch in #6 works for D8. Someone?

Usually we fix the latest dev version first however since #2151101: Convert theme_status_report() to Twig has converted theme_status_report() to twig we no longer have this problem since Twig handles the problem if requirements.value does not exist.

But regardless of this version the patch is not rtbc since it has no test.

Oh and the patch in #9 is not right - patch #1 would fix the issue for D7 - but to prove this you'll need a test.

Patch in #1 works for Drupal 7.34.

#1 fixed it here 7.34 build :)

I fixed it for 7.39

Re-posting the previously RTBC'd #1. With a copy change on the comment. Hope that works well?

@alexpott, tried to make a test for this, but I'm shit at tests... so lmk how I can improve this?

I'm setting this to RTBC because I didn't actually write the fix, I only changed the comment for clarity and wrote the test to prove it fixes the problem.

Patch looks good, thanks!

Patch works for me, thank you..!

Per #17 this is not needed for D8, just want to double check with Fabianx whether we still want to address #8.

RTBC + 1

Looks great to me, the ternary operator thing is not a problem for me and theme_status_report is not in the critical path at all ...

Committed and pushed to 7.x, thanks!