Skip to content
This repository was archived by the owner on Jan 30, 2020. It is now read-only.

Conversation

@marcguyer
Copy link
Contributor

  • Is this related to documentation?

Copy link
Member

@froschdesign froschdesign left a comment

Choose a reason for hiding this comment

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

The update is great! 👍

Only one change is necessary: revert the changes on spelling.

### Localized Booleans

As mentioned previously, `Zend\Filter\Boolean` can also recognise localized "yes" and "no" strings.
As mentioned previously, `Zend\Filter\Boolean` can also recognize localized "yes" and "no" strings.
Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason why the spelling was changed here? So far we have no definition for a standard dialect in the documentation.

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 gave it some thought and research when making the change. I decided to change it for 2 reasons: There doesn't seem to be any other indication that it is written in a specific dialect -- e.g., no other usage of ise vs ize. So the first reason is for consistency. The 2nd reason is because I ran across this while researching it. I figured the ize variant was "better" globally. I'm no expert in linguistics but, there you go.

Copy link
Member

Choose a reason for hiding this comment

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

@marcguyer
Thank you for your detailed answer. In the past I have done the same research several times.
The conclusion is we have no definition for a standard dialect. Therefore, I see no need to change anything in this direction and we should do this kind of discussion in the documentation issue tracker.

Thanks again for your effort!

@froschdesign froschdesign merged commit 922fe11 into zendframework:master Mar 14, 2019
@froschdesign
Copy link
Member

@marcguyer
Thanks! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants