API page: https://api.drupal.org/api/drupal/core%21includes%21bootstrap.inc/functi...

Enter a descriptive title (above) relating to function drupal_bootstrap, then describe the problem you have found:

It appears that the docs for parameter $phase are missing DRUPAL_BOOTSTRAP_KERNEL. This is documented here: https://api.drupal.org/api/drupal/core!includes!bootstrap.inc/constant/D...

This information needs to be added to the drupal_bootstrap() docs.

CommentFileSizeAuthor
#1 drupal_bootstrap-docs-are-missing-phases-2039091-1.patch664 bytesghazlewood

Comments

ghazlewood’s picture

ghazlewood
Location Hampshire, UK
commented
Assigned: Unassigned » ghazlewood
Status: Active » Needs review
StatusFileSize
new drupal_bootstrap-docs-are-missing-phases-2039091-1.patch664 bytes

Saw this low hanging fruit and thought I'd patch...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Good enough, thanks! I'll get that committed sometime soon.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Note: I checked 7.x and its documentation is accurate, so this is for 8.x only.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

I just discovered this myself, and was about to file a patch. Thanks!

Committed and pushed to 8.x.

ParisLiakos’s picture

ParisLiakos commented
Status: Fixed » Needs work 
diff --git a/core/includes/bootstrap.inc b/core/includes/bootstrap.inc
old mode 100644
new mode 100755

this change should be reverted

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Fixed

This is being handled over in #2057445: Fix file wrong file perms introduced by 2056415 and 2056417

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Why are all these file permission change problems coming up all of a sudden? It seems like we've had a lot of them lately.

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