#32978: Improve an error message on loaddata/dumpdata when PyYAML is not
installed.
-------------------------------------+-------------------------------------
Reporter: Brad | Owner: Bal
Type: | Krishna Jha
Cleanup/optimization | Status: assigned
Component: Core | Version: 3.2
(Serialization) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
serializers,yaml,dumpdata,loaddata |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Brad):
Interestingly, I found a comment in `django/core/serializers/pyyaml.py`:
`# Requires PyYaml (https://pyyaml.org/), but that's checked for in
__init__."`
However, I'm not seeing any such check in
`django/core/serializers/__init__.py`. (I expected a HAS_PYYAML or
something like that.)
There are also some tests in `tests/serializers/test_yaml.py` that test
specifically for
`YAML_IMPORT_ERROR_MESSAGE = r'No module named yaml'`
So those will need to be revised.
--
Ticket URL: <https://code.djangoproject.com/ticket/32978#comment:6>
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/070.76eeeb630be5b53fedf259abe57d9355%40djangoproject.com.
