Project

Profile

Help

Issue #1503

closed

remove Criteria import from serializers

Added by amacdona@redhat.com almost 9 years ago. Updated over 5 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

There is a circular import problem, and the serializer really shouldn't need `pulp.server.db.model.criteria.Criteria`

The reason it is necessary right now is that the criteria needs to be translated from external to internal representation. `pulp/server/webservices/views/serializers/__init__.py:ModelSerializer.translate_criteria` should translate a dictionary representation of criteria and that dict should be used to create the `Criteria` object. Currently a `Criteria` object is created, it is converted to a dict, the dict is translated, and then the translated dict is used create a new `Criteria` object.


Related issues

Related to Pulp - Task #1578: Incoming searches should be translated by the modelCLOSED - WONTFIX

Actions
Actions #1

Updated by mhrivnak almost 9 years ago

  • Triaged changed from No to Yes
Actions #2

Updated by amacdona@redhat.com almost 9 years ago

  • Related to Task #1578: Incoming searches should be translated by the model added
Actions #3

Updated by bmbouter over 5 years ago

  • Status changed from NEW to CLOSED - WONTFIX
Actions #4

Updated by bmbouter over 5 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #5

Updated by bmbouter over 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF