#33589: Incomplete string escaping in formats for calendar.
-------------------------------------+-------------------------------------
Reporter: Shrikant Dhayje | Owner: Ankur Roy
Type: Bug | Status: assigned
Component: contrib.admin | Version: 4.0
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
bug,regex,Inaccurate |
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by SanderBeekhuis):
Hi all, just chiming in without much django contribution experience
One can, in theory, set any `DATE_INPUT_FORMATS` in ones `settings.py`
I set the following
{{{
USE_L10N = False
DATE_INPUT_FORMATS = ["%Y-'''--%m-\t-%d"]
}}}
in my settings.py. It allows me to test the code paths discussed in this
patch. If I use admin date picker to pick dates they are formatted using
the logic in this JavaScript file.
However even with a single `'` or `\t` in the format the behavior does not
seem work in a very sensible way. Improving them would take more then the
proposed patch. So dropping these replaces might make the most sense.
When I get time to work on this issue again I can try write to write a JS
tests for this in the future. If that would be nice?
--
Ticket URL: <https://code.djangoproject.com/ticket/33589#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/010701814df3d073-80770d99-b9fc-441b-ab8e-b769cb37f35a-000000%40eu-central-1.amazonses.com.
