hook_page_top doesn't return anything is there isn't a user-supplied path in the config. So we can't display a message without a link. Here's a small patch so we can.

CommentFileSizeAuthor
#6 site_status_message-3076633-6.patch1.04 KBaaronbauman
#2 site_status_message-3076633-2.patch1.39 KBdkosbob

Comments

dkosbob created an issue. See original summary.

dkosbob’s picture

dkosbob commented
StatusFileSize
new site_status_message-3076633-2.patch1.39 KB
drupalgideon’s picture

drupalgideon
Location London
commented

I don't think the D8 version of this module has worked for quite some time and is far behind the development of the 7 module.

It will be re-written completely in the future. I just need the time to make it the same as the 7 version.

I won't be patching this module just yet as I don't have the capacity to test. But patches are more than welcome and for other users of this module I suggest using composer to add any necessary patches until the rewrite is done.

Thanks!

  • DrupalGideon committed 9829e95 on 8.x-1.x authored by dkosbob 
    Issue #3076633 by dkosbob: Message does not display if no link provided
drupalgideon’s picture

drupalgideon
Location London
commented
Status: Active » Needs review
aaronbauman’s picture

aaronbauman
he/him
commented
StatusFileSize
new site_status_message-3076633-6.patch1.04 KB

Patch was applied, but with fuzz so that it didn't actually address the issue.
The OP remains a problem.

Re-roll attached.

drupalgideon’s picture

drupalgideon
Location London
commented
Status: Needs review » Fixed

This won't be fixed for Drupal 8 but has been addressed in the new D9/D10 version of the module.

https://www.drupal.org/project/site_status_message/releases/2.0.0-alpha1

Status: Fixed » Closed (fixed)

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