A search for "system_php info_all" didn't turn anything up, so here goes. I can't remember how many times I've had to create a separate phpinfo file to see all the PHP settings and variables because system_php() displays INFO_GENERAL and INFO_CONFIGURATION instead of INFO_ALL. This patch corrects this.

CommentFileSizeAuthor
#8 243524-phpinfo-D7.patch636 bytesdave reid
#5 phpinfo_d6.patch534 byteschx
system_php_info_all.patch300 byteschristefano

Comments

christefano’s picture

christefano commented
Status: Active » Needs review
catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Yep, I wanted to look something up the other day and was disappointed to see it missing from the phpinfo.

Couldn't find anything about this going in originally, so on the assumption that this discussion hasn't already and been rejected, and that the patch is fine and works. RTBC.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

chx’s picture

chx commented
Title: display INFO_ALL output when calling system_php() » system_php() is useless this way
Version: 7.x-dev » 6.x-dev
Category: feature » bug
Priority: Normal » Critical
Status: Closed (fixed) » Reviewed & tested by the community
StatusFileSize
new phpinfo_d6.patch534 bytes

Here is a D6 patch. I had the unfortunate opportunity to rely on this page today. That was unpleasant. You can't tell, say, whether you have an opcode operating. Whether GD has JPEG, PNG or GIF support. According to manual, nothign is ALL so I rolled that before I found this issue.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: system_php() is useless this way » display INFO_ALL output when calling system_php()
Category: bug » feature
Priority: Critical » Normal

Looks good to go.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 6.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to Drupal 6. Let's backport the simplifiation to D7.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Patch (to be ported) » Reviewed & tested by the community
StatusFileSize
new 243524-phpinfo-D7.patch636 bytes

Easy enough. Also going to go ahead and just mark as RTBC since this was already agreed upon.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

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