We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
replaceAll
As explained here #106
The replaceAll paramer in the Create() method exists only as a workaround for the File_Adapter.
I think that the Create method, in its suggested implementation should always stick to what it's supposed to do, that is creating.
Overriding = [Deleting the content and then creating] should be dealt with a call to Delete and then Create.
If someone wants the Create method to have an option to override, there is always the possibility of using the config.
config
This change will affect ALL TOOLKITS.
Regarding the File_Adapter, the change is done in the same PR that fixes this: #145
The text was updated successfully, but these errors were encountered:
Closed by #151
Sorry, something went wrong.
alelom
Successfully merging a pull request may close this issue.
As explained here #106
The
replaceAll
paramer in the Create() method exists only as a workaround for the File_Adapter.I think that the Create method, in its suggested implementation should always stick to what it's supposed to do, that is creating.
Overriding = [Deleting the content and then creating] should be dealt with a call to Delete and then Create.
If someone wants the Create method to have an option to override, there is always the possibility of using the
config
.Important note
This change will affect ALL TOOLKITS.
Regarding the File_Adapter, the change is done in the same PR that fixes this: #145
The text was updated successfully, but these errors were encountered: