Removed PassAll from ChangeFilter and fixed sort not being descending order.
the build was successful
the build was successful
This commit is contained in:
@@ -57,7 +57,7 @@ func main() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func logListedChanges(changes *services.ChangeService) {
|
func logListedChanges(changes *services.ChangeService) {
|
||||||
sub := changes.Subscribe(context.Background(), models.ChangeFilter{PassAll: true})
|
sub := changes.Subscribe(context.Background(), models.ChangeFilter{})
|
||||||
|
|
||||||
for change := range sub {
|
for change := range sub {
|
||||||
log.Printf("Change: Author=%#+v Model=%#+v Op=%#+v", change.Author, change.Model, change.Op)
|
log.Printf("Change: Author=%#+v Model=%#+v Op=%#+v", change.Author, change.Model, change.Op)
|
||||||
|
|||||||
@@ -47,7 +47,7 @@ func (r *changeRepository) List(ctx context.Context, filter models.ChangeFilter)
|
|||||||
}
|
}
|
||||||
|
|
||||||
changes := make([]*models.Change, 0, initialSize)
|
changes := make([]*models.Change, 0, initialSize)
|
||||||
err := r.changes.Find(query).All(&changes)
|
err := r.changes.Find(query).Sort("-date").Limit(limit).All(&changes)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -104,10 +104,6 @@ func (change *Change) Objects() []interface{} {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (change *Change) PassesFilter(filter ChangeFilter) bool {
|
func (change *Change) PassesFilter(filter ChangeFilter) bool {
|
||||||
if filter.PassAll {
|
|
||||||
return true
|
|
||||||
}
|
|
||||||
|
|
||||||
if filter.Author != nil && change.Author != *filter.Author {
|
if filter.Author != nil && change.Author != *filter.Author {
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
@@ -172,6 +168,4 @@ type ChangeFilter struct {
|
|||||||
EarliestDate *time.Time
|
EarliestDate *time.Time
|
||||||
Author *string
|
Author *string
|
||||||
Limit *int
|
Limit *int
|
||||||
|
|
||||||
PassAll bool // DO NOT EXPOSE
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user