Page MenuHomeDevCentral
Differential D36

PhpFpmEnvironment::getEnvironmentVariables refactoring
ClosedPublic
Actions

Authored by dereckson on Nov 8 2015, 22:58.
Tags
Referenced Files
Unknown Object (File)
Thu, Apr 16, 09:53
Unknown Object (File)
Wed, Apr 15, 13:00
Unknown Object (File)
Thu, Apr 9, 10:18
Unknown Object (File)
Tue, Apr 7, 08:58
Unknown Object (File)
Tue, Apr 7, 07:09
Unknown Object (File)
Tue, Apr 7, 07:09
Unknown Object (File)
Sun, Apr 5, 19:02
Unknown Object (File)
Sat, Apr 4, 20:13
View All Files
Subscribers
None

Details

Reviewers
Sandlayth
Commits
rDPHPVIIFPMdf34942bdc01: PhpFpmEnvironment::getEnvironmentVariables refactoring
rDPHPFPMdf34942bdc01: PhpFpmEnvironment::getEnvironmentVariables refactoring
Summary

Split PhpFpmEnvironment::getEnvironmentVariables in more discrete methods.

New static methods:

  • addTempEnvironmentVariables: add TMP, TEMP, TMPDIR env variable to an array
  • addPathEnvironmentVariables: add PATH env variable to an array
  • addHardcodedEnvironmentVariables: add both to an array
Test Plan

compare output before and after the change

Diff Detail

Repository
rDPHPFPM Docker image for Nginx, php-fpm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 77.Nov 8 2015, 22:58
dereckson retitled this revision from to PhpFpmEnvironment::getEnvironmentVariables refactoring.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: Sandlayth.
Herald added a project: Nasqueron Docker deployment squad. · View Herald TranscriptNov 8 2015, 22:58
Sandlayth accepted this revision.Nov 10 2015, 09:21
Sandlayth edited edge metadata.
This revision is now accepted and ready to land.Nov 10 2015, 09:21
Closed by commit rDPHPFPMdf34942bdc01: PhpFpmEnvironment::getEnvironmentVariables refactoring (authored by dereckson, committed by dereckson). · Explain WhyNov 10 2015, 13:50
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
files/
usr/
local/
lib/
php-fpm-env/
38 lines

Diff 90

View Options

files/usr/local/lib/php-fpm-env/PhpFpmEnvironment.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator