system_init() calls path_is_admin() for cli requests

Created on 12 January 2015, over 9 years ago

system_init() has been removed in 8.x, and assets are added in a hook that only runs for HTML page requests.

We can't make that change for 7.x I think, however I just ran into an issue where hook_admin_paths() in a custom module was checking a class that had moved in an update, and this was resulting in fatal errors when running drush updb.

There's absolutely no reason to execute that code for any full bootstrap, so I think it's reasonable to add a drupal_is_cli() check there.

πŸ› Bug report
Status

Needs review

Version

7.0 ⚰️

Component
BaseΒ  β†’

Last updated 30 minutes ago

Created by

πŸ‡¬πŸ‡§United Kingdom catch

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024