Skip to content

Commit

Permalink
Fix date filtering being ambiguous
Browse files Browse the repository at this point in the history
  • Loading branch information
dralley committed Apr 12, 2018
1 parent 3087143 commit dda4d7d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 11 deletions.
6 changes: 2 additions & 4 deletions pulpcore/pulpcore/app/viewsets/repository.py
Original file line number Diff line number Diff line change
Expand Up @@ -190,15 +190,13 @@ class RepositoryVersionFilter(filterset.FilterSet):
version_min = filters.NumberFilter(name='number', lookup_expr='gte')
version_max = filters.NumberFilter(name='number', lookup_expr='lte')

created_after = filters.IsoDateTimeFilter(name='created', lookup_expr='gte')
created_before = filters.IsoDateTimeFilter(name='created', lookup_expr='lte')

created = filters.IsoDateTimeFilter(name='created')
content = RepositoryVersionContentFilter(label="Content HREF is equivalent to")

class Meta:
model = RepositoryVersion

fields = ['version_min', 'version_max', 'created_after', 'created_before', 'content']
fields = ['version_min', 'version_max', 'created', 'content']


class RepositoryVersionViewSet(NamedModelViewSet,
Expand Down
10 changes: 3 additions & 7 deletions pulpcore/pulpcore/app/viewsets/task.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,17 +14,13 @@ class TaskFilter(filterset.FilterSet):
state_in_list = CharInFilter(name='state', lookup_expr='in')
worker = HyperlinkRelatedFilter(name='worker')

started_after = filters.IsoDateTimeFilter(name='started_at', lookup_expr='gte')
started_before = filters.IsoDateTimeFilter(name='started_at', lookup_expr='lte')

finished_after = filters.IsoDateTimeFilter(name='finished_at', lookup_expr='gte')
finished_before = filters.IsoDateTimeFilter(name='finished_at', lookup_expr='lte')
started_at = filters.IsoDateTimeFilter(name='started_at')
finished_at = filters.IsoDateTimeFilter(name='finished_at')

class Meta:
model = Task
fields = ('state', 'state_in_list', 'worker',
'started_after', 'started_before',
'finished_after', 'finished_before')
fields = ('state', 'state_in_list', 'worker', 'started_at', 'finished_at')


class TaskViewSet(NamedModelViewSet,
Expand Down

0 comments on commit dda4d7d

Please sign in to comment.