Issue #3276
closed
CreatedResource can only serialize objects whose href is their pk
Description
The CreatedResource is serialized on a Task as the href to the resource.
Here is the offending line:
https://github.com/pulp/pulp/blob/2904dd1c6fa947244c0bb98ca5b84acba15f188e/pulpcore/pulpcore/app/serializers/task.py#L27
This works for Publications and any other resource whose href has pk as the only url parameter. It doesn't work for resources whose href has url parameters, like RepositoryVersion.
reverse for Publications uses only "pk"
http://pulp3.dev:8000/api/v3/publications/56a08642-a9d8-42bb-91dd-d1f5021124dd/
reverse for RepositoryVersion needs "repository_pk", "number"
http://pulp3.dev:8000/api/v3/repositories/026b1ce6-8959-43a7-ac63-54a809dea4b6/versions/6/
Added by amacdona@redhat.com over 6 years ago
Added by amacdona@redhat.com over 6 years ago
Revision e894e19e | View on GitHub
Make CreatedResource serialize any resource
Using the model's serializer rather than reverse allows us to generate hrefs for objects that use any set of url parameters.
Updated by amacdona@redhat.com over 6 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to amacdona@redhat.com
Updated by amacdona@redhat.com over 6 years ago
- Status changed from ASSIGNED to POST
Updated by amacdona@redhat.com over 6 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|e894e19e134ec7a3a90e7bd733985a664558d6ec.
Updated by bmbouter over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Make CreatedResource serialize any resource
Using the model's serializer rather than reverse allows us to generate hrefs for objects that use any set of url parameters.
closes #3276 https://pulp.plan.io/issues/3276