Fix week_field generating invalid week numbers (according to W3 and Browser) #22190
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey guys,
the week_field form helper currently uses
%W
when converting the input value. This results in week numbers being zero-based, which is invalid according to the W3 spec (http://www.w3.org/TR/html-markup/datatypes.html#form.data.week) and the browser in turn selects the week before the intended week.The first commit adds a spec to illustrate the correct behavior, not receiving a value of
2015-W00
but expecting the correct2015-W01
.The second commit fixes this spec by replacing
%W
with%V
, differences according to the ruby doc:Please let me know if I should fix/squash/append anything, since this is my first rails contribution 🎉
Cheers
Christoph