No matter if you have or you don't have geoip databases installed, you get a notice message in each page saying
Notice: Undefined index: title in Drupal\system\SystemManager->Drupal\system\{closure}() (line 116 of core/modules/system/src/SystemManager.php)

Reproducible with drupal core 8.1.8

CommentFileSizeAuthor
#2 2794961-undefined-index-title-in-status-page.patch1.44 KBcrevillo

Issue fork geoip-2794961

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

crevillo created an issue. See original summary.

crevillo’s picture

crevillo CreditAttribution: crevillo as a volunteer commented
FileSize
2794961-undefined-index-title-in-status-page.patch1.44 KB

Added patch. It looks like the title is defined at the beginning but then lost. this patch solves it.

brianV’s picture

brianV CreditAttribution: brianV as a volunteer commented
Status: Active » Reviewed & tested by the community

Confirmed, this makes the title appear.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented 
+++ b/geoip.install
@@ -35,24 +35,24 @@ function geoip_requirements($phase) {
+      $requirements['geoip_local_database'] = array_merge($requirements['geoip_local_database'], [
...
+      $requirements['geoip_local_database'] = array_merge($requirements['geoip_local_database'], [
...
+      $requirements['geoip_local_database'] = array_merge($requirements['geoip_local_database'], [

We could actually just do += []. Trying that and writing test for requirements.

  • mglaman committed d161fa2 on 8.x-2.x authored by crevillo 
    Issue #2794961 by crevillo: Notice: Undefined index: title in Drupal\...
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
Status: Reviewed & tested by the community » Fixed

Thanks! Added test as well.

Status: Fixed » Closed (fixed)

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

ppseftogiannis made their first commit to this issue’s fork.

ppseftogiannis opened merge request !2