Voting starts in March for the Drupal Association Board election.
While Typed Configuration builds on Typed Data, there are still a few mismatches between both of them, and this should add some consistency in how interfaces and parameters are used in both sides.
There's currently a TypedConfigManagerInterface, but the counterpart in TypedData doesn't exist. Because of this, some methods are defined twice, once in TypedConfigManagerInterface, once in TypedDataManager.
This helps us to address some @todo in \Drupal\Core\TypedData\TypedData: "Add the typed data manager as proper dependency."
1. Add a new interface: TypedDataManagerInterface
2. Use it for typed parameters instead of the class TypedDataManager
3. Get TypedConfigManagerInterface to extend TypedDataManagerInterface
4. Then use the new interface in TypedDataTrait and use it for fixing the dependency issues in TypedData objects
5. More cleanup is possible in once this lands.
User interface changes
Minimal. Use TypedDataManagerInterface instead of TypedDataManager.
Beta phase evaluation
|Issue category||Task because the issue implement OO best practices by using an interface instead of demanding a specific class where TypedDataManager is needed.|
|Issue priority||Normal because while the change makes understanding the typed data system much easier, its not a showstopper.|
|Disruption||Minimal. Use TypedDataManagerInterface instead of TypedDataManager to typehint. There is little use in core, and only pretty specialized code depends on it.|
FAILED: [[SimpleTest]]: [PHP 5.5 MySQL] Unable to apply patch 2488568-51.patch. Unable to apply patch. See the log in the details link for more information. View
|#47||2488568.47.patch||45.19 KB||Jose Reyero|
PASSED: [[SimpleTest]]: [PHP 5.5 MySQL] 113,623 pass(es). View
|#47||2488568.44-47.diff.txt||1000 bytes||Jose Reyero|
|#44||2488568-36-44-diff.txt||1.35 KB||Jose Reyero|