Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4.0 #119

Merged
merged 141 commits into from Jan 25, 2014
Merged

4.0 #119

merged 141 commits into from Jan 25, 2014

Conversation

@mnapoli
Copy link
Member

@mnapoli mnapoli commented Sep 29, 2013

Build Status Coverage Status

Complete issue list

Expect changes in the YAML/array configurations, …

See also: Thoughts for PHP-DI 4.0

Main goals:

  • interoperability with "standards", other containers and frameworks (Symfony 2 mainly)
  • modern: target PHP 5.5 and 5.6? 5.5 brings major changes in simplicity of configuration with ::class (still compatible with PHP 5.3 though)
  • simpler config syntax (no need to learn a syntax)
  • compiled container (or at least leave the door open for this)

TODO:

  • Documentation: use the term "autowiring"
  • Documentation: entry alias
  • Re-make all tests compatible PHP 5.3
  • Documentation: migration from 3.* to 4.0
  • Documentation : update the "how it works article"
  • Simplify/less differences in PHP "code" configuration and PHP "array" configuration
  • ContainerBuilder should take a PHP file, so that it can be cached/compiled and not re-executed each time
  • Remove definition validation (unused?)
  • Document Z2 and Symfony 2 integration
  • Rename ->withConstructor() to ->constructor()?
  • Rename ->withMethod() to ->method()
  • Rename ->withProperty() to ->property()
  • phpDoc on methods of ObjectDefinitionHelper
  • ContainerBuilder should take a file, not a DefinitionSource
  • Definition::isCacheable() -> add a CacheableDefinition interface instead
  • Improve code coverage to 80%
  • Fix this documentation: https://github.com/mnapoli/PHP-DI/blob/4.0/doc/container-configuration.md
  • Improve comment on call to deprecated method in ContainerBuilder::buildProxyFactory (Ocramius/ProxyManager#137)
  • Link in documentation to 3.x documentation (GitHub)
  • Update lazy injection documentation

Preview of the new website:

phpdi4

mnapoli added 30 commits Sep 29, 2013
Conflicts:
	change-log.md
Conflicts:
	doc/getting-started.md
	src/DI/Container.php
	src/DI/Definition/Source/AnnotationDefinitionSource.php
	tests/UnitTests/DI/Annotation/InjectTest.php
	tests/UnitTests/DI/ContainerTest.php
	tests/UnitTests/DI/Definition/Source/AnnotationDefinitionSourceTest.php
	tests/UnitTests/DI/Definition/Source/ReflectionDefinitionSourceTest.php
mnapoli added 27 commits Jan 8, 2014
…ith '__construct()'
Conflicts:
	doc/container-configuration.md
	news/README.md
mnapoli added a commit that referenced this pull request Jan 25, 2014
4.0
@mnapoli mnapoli merged commit 6205719 into master Jan 25, 2014
1 check passed
1 check passed
default The Travis CI build passed
Details
@mnapoli mnapoli added the version label Aug 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.