Repository navigation
Conversation
Map Laravel's app.env to a default WP_ENVIRONMENT_TYPE before WordPress loads, so app.env=local produces local. dev, development, testing and staging are mapped too; an explicit process environment value, a configured constant or a constant already defined still wins. Fixes Pollora#443.
ogorzalka
force-pushed
the
hotfix/443-wordpress-environment
branch
from
October 13, 2026 17:29
3ceb337 to
830eeb6
Compare
Member
|
Thanks for the fix and the thorough tests: the mapping matches #443 exactly. To land it in v13.36.0 I rebased your commit (authorship kept) onto
Checked on a Pollora site: |
…utside production The site now reports its environment as local, so the trackback test needs the wp_should_disable_pings_for_environment filter. The changelog says what the new environment changes for pings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Map Laravel's
app.envto a defaultWP_ENVIRONMENT_TYPEbefore WordPress loads, soapp.env=localproduceslocal. The mapping also handlesdev,development,testing, andstaging, while preserving explicit process environment values, configured constants, and constants that are already defined.Fixes #443.
The documentation submodule points to the companion documentation PR. Its commit is based on current docs
main, so it also advances the older documentation snapshot.Passed individually on PHP 8.4 with
memory_limit=1G:composer test:refacto -- --debugcomposer test:lintcomposer test:typescomposer test:unit: 1990 tests, 4012 assertions, including 28 new environment regression cases.composer test:type-coverage: 99.2%.The regression cases exercise the real bootstrap and constant manager with the repository's WordPress stubs. DDEV and WordPress core integration were not run.