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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

christefano’s picture

Status: Active » Needs review
catch’s picture

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

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

chx’s picture

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
FileSize
534 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

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

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

Status: Patch (to be ported) » Reviewed & tested by the community
FileSize
636 bytes

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

Dries’s picture

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.