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
TASK: Introduce internal flow package key value object #3338
TASK: Introduce internal flow package key value object #3338
Conversation
- Custom Package.php must always implement the `PackageKeyAwareInterface` (which they do by extending the Flow Package) - $packageClassInformation will never be empty at this point as `detectFlowPackageFilePath` is guaranteed to return or will throw.
to `FlowPackageKey::guessComposerPackageName`
maybe we should take it down a notch the motivation for this pr was that i have seen some custom package key implementations like i started to non-breaking allow the also i meant to use the interface FusionAutoIncludeHandler
{
public function loadFusionFromPackage(FlowPackageKey $packageKey, FusionSourceCodeCollection $sourceCodeCollection): FusionSourceCodeCollection;
} But this requires us to also retrieve the in the end it might be that we are better of keeping the apis only string based instead of tying it to the
so it seems what i really wanted to do was to create a Either way i think its safe to say in order to get this merged and not delayed that the |
Mmmm, the thing is |
Removed the new In the end this is just a nice little refactoring but not a feature. |
This is now ready to be merged. And i will refrain for now to extend this into a bigger refactoring, changing public api and adding a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't necessarily like the public API of the new VO, but it's not worse than we had before 😆
Aaaand i added nice unit tests so it should definitely be better than before :D |
Upgrade instructions
Review instructions
The concept around the flow package key might be dated, still major parts rely on this and we could use a little strict typing around ^^
Also possible refactoring in the future to a composer key might be easier if phpstan can tell us the difference between the types instead of refactoring one string to another.
Checklist
FEATURE|TASK|BUGFIX
!!!
and have upgrade-instructions