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

Implement initial version #2

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

Implement initial version #2

wants to merge 34 commits into from

Conversation

acoulton
Copy link
Member

No description provided.

For where you want to opt-out of all the automatic logic and do your
own thing completely.
Use a list of guessers so that we can easily add support for more in
future without having to alter the code in other things.
`false` is a clearer return when we are saying we explicitly don't know
what provider to use.
As we are going to need different attributes for hydration and I don't
want to confuse them.
The dates are the easy ones (so that tests can give just a simple
string). This will also be how we do collections etc.
We can simplify to two distinct sets of concerns:

* StubAs -> cast the value to the expected type
* StubDefault -> generate a default value
`(new StubObjects)->stub()` will be the main entry point. The
individual internal factories then follow `StubFactory` naming.
So that we only have to do the parsing for the first of a type, and then
 are just passing values through the predefined chain.
When you have classes that have some internal logic that needs to be
bypassed - generally e.g. if they directly reference an external
dependency such as the filesystem etc.
Will give us a lot more control over the interface in future, and make
it easier for users to understand how to implement it.
This is ambiguous - do you mean "now" (which is how PHP will interpret
it)
or "null"?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant