While coding we've put a dsm() in the .install file in hook_requirements. That turns up also outside the Status report on the path 'admin/config' as well.

CommentFileSizeAuthor
#3 run_only_on_systemDOTstatus_route-3093404-3.patch531 byteslolandese

Comments

lolandese created an issue. See original summary.

lolandese’s picture

lolandese commented
Category: Bug report » Feature request
Related issues: +#2954968: hook_requirements bogs down /admin/config

Looking at #2954968: hook_requirements bogs down /admin/config this seems to affect other modules as well, thus depending on core.

Leaving it open to see if something can be done like checking if the current route is system.status. Especially turning on the connection with remote repos this is a performance hog that we wouldn't want to affect other pages.

lolandese’s picture

lolandese commented
Status: Active » Needs review
StatusFileSize
new run_only_on_systemDOTstatus_route-3093404-3.patch531 bytes

  • lolandese committed e73b5b0 on 8.x-1.x 
    Issue #3093404 by lolandese: Run code only when on the Status report...
lolandese’s picture

lolandese commented
Status: Needs review » Fixed

Note that the solution works but we renounce to have a status message on other pages to warn us about possible issues. For the GIT Info Report that is perfectly fine as possible errors do only impact the report itself and no other site functionality.

The requirement errors that we might have:

  • Is shell_exec not disabled?
  • Is Git installed?
  • Are we on a Unix like OS?

Status: Fixed » Closed (fixed)

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