Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Make include_path functionality of Config and Translator opt-in #4766

Closed
wants to merge 277 commits into from

Conversation

weierophinney
Copy link
Member

#4443 and #4574 both introduce features that use PHP's include_path to identify application assets. While the functionality has reasonable use cases, as implemented, it introduces some performance impact, as the components look on the include_path for each and every file. Additionally, if . is not on the include_path, files that resolve from the current working directory will not resolve.

mtymek and others added 30 commits May 8, 2013 22:02
Merge branch 'hotfix/4438' of git://github.com/leodido/zf2 into leodido-hotfix/4438
Merge branch 'leodido-hotfix/4438' into develop
* added better testing for DbTableGateway paginator
Merge branch 'fmarquis-master' into develop
Merge branch 'YannickDa-SqlSetTable' into develop
Merge branch 'AgentCoop-develop' into develop
Back to dev version

Conflicts:
	README.md
	library/Zend/Version/Version.php
2.2.0 readiness

Conflicts:
	README.md
	library/Zend/Version/Version.php
Conflicts:
	README.md
	library/Zend/Version/Version.php
@ghost ghost assigned weierophinney Jul 1, 2013
- Added `setUseIncludePath()` call to translator adapters
- added flag, Factory::USE_INCLUDE_PATH, as second argument to config
  factory
- You must pass the `$useIncludePath` to either `fromFile()` or
  `fromFiles()` for it to be used. Constants were created with boolean
  values for those using IDEs.
- Created AbstractFileLoader, which provides implementation details for
  when/how to use the include_path to load translation file assets.
- Updated INI, gettext, and PHP loaders to extend AbstractFileLoader.
@weierophinney
Copy link
Member Author

Made this against the wrong branch; closing.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 5a3d9e7 on weierophinney:hotfix/4766 into * on zendframework:master*.

weierophinney added a commit to zendframework/zend-config that referenced this pull request May 15, 2015
…s opt-in

- Added `setUseIncludePath()` call to translator adapters
- added flag, Factory::USE_INCLUDE_PATH, as second argument to config
  factory
weierophinney added a commit to zendframework/zend-config that referenced this pull request May 15, 2015
…t-in only

- You must pass the `$useIncludePath` to either `fromFile()` or
  `fromFiles()` for it to be used. Constants were created with boolean
  values for those using IDEs.
weierophinney added a commit to zendframework/zend-i18n that referenced this pull request May 15, 2015
…s opt-in

- Added `setUseIncludePath()` call to translator adapters
- added flag, Factory::USE_INCLUDE_PATH, as second argument to config
  factory
weierophinney added a commit to zendframework/zend-i18n that referenced this pull request May 15, 2015
… in Translators

- Created AbstractFileLoader, which provides implementation details for
  when/how to use the include_path to load translation file assets.
- Updated INI, gettext, and PHP loaders to extend AbstractFileLoader.
weierophinney added a commit to zendframework/zend-i18n that referenced this pull request May 15, 2015
…include_path flag

- Added to translator loader plugin manager
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet