8000 [Config] Delegate creation of ConfigCache instances to a factory. by mpdude · Pull Request #14178 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Config] Delegate creation of ConfigCache instances to a factory. #14178

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.

Sign up for GitHub

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

Closed
wants to merge 24 commits into from
Closed

[Config] Delegate creation of ConfigCache instances to a factory. #14178

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
02159ae
Delegate creation of ConfigCache instances to a factory.
mpdude Apr 2, 2015
98aa48b
Fix wrong method name
mpdude Apr 2, 2015
3744edf
Add E_USER_DEPRECATED warning
mpdude Apr 2, 2015
282a75a
CS fix
mpdude Apr 2, 2015
ae29b82
CS fixes
mpdude Apr 2, 2015
1730fc3
Remove "constructor" docblocks
mpdude Apr 2, 2015
770ee09
DocBlock fixes
mpdude Apr 2, 2015
2a058fd
Fix DocBlock
mpdude Apr 2, 2015
8491dfd
Fix deprecation warnings
mpdude Apr 2, 2015
beb307c
Fix unnecessary re-initialization from cache
mpdude Apr 2, 2015
dbec920
Add deprecation notice in UPGRADE-2.7
mpdude Apr 4, 2015
5a91716
Tests need a recent version of the Config component
mpdude Apr 4, 2015
f2c3a24
Explicitly add symfony/config dependency in an desperate attempt to m…
mpdude Apr 4, 2015
35488bd
Suggestions from GH
mpdude Apr 5, 2015
17a55ec
FabBot compliance
mpdude Apr 6, 2015
07ecf57
Bump composer dep
mpdude Apr 6, 2015
a89db7a
As in aa82fb065d970c1fd02bf35790e4da8970f45e7b
mpdude Apr 6, 2015
7938349
Update composer.json
mpdude Apr 6, 2015
cb9917f
Don't mess with composer.json where it is not necessary
mpdude Apr 7, 2015
2016537
The HttpKernel for sure depends on the Config component
mpdude Apr 7, 2015
beb0a92
Cleanup composer.json
mpdude Apr 7, 2015
afee2bb
Make sure at least symfony/config 2.7 is installed when loaded as a t…
mpdude Apr 7, 2015
4832e1a
symfony/config dev-master cannot be used unless changes from this PR …
mpdude Apr 7, 2015
7f90bae
Grammar fix in comment
mpdude Apr 7, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
CS fixes
  • Loading branch information
@mpdude
mpdude committed Apr 6, 2015
commit ae29b82502d15e9b58983278a04d674f19a2a3fc
4 changes: 2 additions & 2 deletions src/Symfony/Component/Config/ConfigCacheFactoryInterface.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ interface ConfigCacheFactoryInterface
* Creates a cache instance and (re-)initializes it if necessary.
*
* @param string $file The absolute cache file path
* @param callable $callable The callable to be executed when the cache needs to be filled (i. e. is not fresh). The cache will be passed as the only parameter to this callback.
* @param callable $callable The callable to be executed when the cache needs to be filled (i. e. is not fresh). The cache will be passed as the only parameter to this callback
*
* @return ConfigCacheInterface $configCache The cache instance.
* @return ConfigCacheInterface $configCache The cache instance
*/
public function cache($file, $callable);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to change file argument name here and to make it more generic.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about $resource?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know... I did not intend to change the way ConfigCache works with this PR, so...?

}
2 changes: 1 addition & 1 deletion src/Symfony/Component/Config/DefaultConfigCacheFactory.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class DefaultConfigCacheFactory implements ConfigCacheFactoryInterface
/**
* Constructor.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not necessary :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... but seen in many other places :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... but we are not doing it anymore :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aye sir :)

*
* @param bool $debug The debug flag to pass to ConfigCache.
* @param bool $debug The debug flag to pass to ConfigCache
*/
public function __construct($debug)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to use false as default value for $debug

{
Expand Down
0