#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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/070.76eeeb630be5b53fedf259abe57d9355%40djangoproject.com.
