Fix documentation for drupal_get_profile() [#2112261]

In #1827112: Convert 'install_profile' variable to Settings we forgot to update the function doc block fordrupal_get_profile()

Critical because we mention variable_get() :)

Part of #1775842: [meta] Convert all variables to state and/or config systems

Comment	File	Size	Author
#4	2112261.4.patch	795 bytes	alexpott
#4
#2	2112261.2.patch	793 bytes	alexpott
#2
#1	2112261.1.patch	820 bytes	alexpott
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

alexpott

he/they

English

🇪🇺🌍

CreditAttribution: alexpott commented 15 October 2013 at 13:14

Status:

Active

» Needs review

File	Size
2112261.1.patch	820 bytes

Here's a patch

Comment #2

alexpott

he/they

English

🇪🇺🌍

CreditAttribution: alexpott commented 15 October 2013 at 13:22

File	Size
2112261.2.patch	793 bytes

Don't really like the test in #1 it's too verbose and doesn't really add value.

Comment #3

damiankloip CreditAttribution: damiankloip commented 15 October 2013 at 13:26

+++ b/core/includes/common.inc
@@ -260,9 +260,8 @@ function drupal_get_region_content($region = NULL, $delimiter = ' ') {
+ * installation state. At all other times, the install_profile setting will be

Should we wrap this setting name in quotes?

+++ b/core/includes/common.inc
@@ -260,9 +260,8 @@ function drupal_get_region_content($region = NULL, $delimiter = ' ') {
+ * available in settings.php.

Howcome you changed from using settings() to settings.php - out of interest :)