diff --git a/src/main/java/com/commafeed/backend/dao/FeedEntryStatusDAO.java b/src/main/java/com/commafeed/backend/dao/FeedEntryStatusDAO.java index 21e2972f..bfe305cb 100644 --- a/src/main/java/com/commafeed/backend/dao/FeedEntryStatusDAO.java +++ b/src/main/java/com/commafeed/backend/dao/FeedEntryStatusDAO.java @@ -295,11 +295,13 @@ public class FeedEntryStatusDAO extends GenericDAO { private void orderBy(CriteriaQuery query, Join entryJoin, ReadingOrder order) { - Path orderPath = entryJoin.get(FeedEntry_.updated); - if (order == ReadingOrder.asc) { - query.orderBy(builder.asc(orderPath)); - } else { - query.orderBy(builder.desc(orderPath)); + if (order != null) { + Path orderPath = entryJoin.get(FeedEntry_.updated); + if (order == ReadingOrder.asc) { + query.orderBy(builder.asc(orderPath)); + } else { + query.orderBy(builder.desc(orderPath)); + } } } @@ -337,26 +339,24 @@ public class FeedEntryStatusDAO extends GenericDAO { public void markSubscriptionEntries(FeedSubscription subscription, Date olderThan) { List statuses = findBySubscription(subscription, true, - ReadingOrder.desc, false); + null, false); saveOrUpdate(markList(statuses, olderThan)); } public void markCategoryEntries(User user, List categories, Date olderThan) { List statuses = findByCategories(categories, user, - true, ReadingOrder.desc, false); + true, null, false); saveOrUpdate(markList(statuses, olderThan)); } public void markStarredEntries(User user, Date olderThan) { - List statuses = findStarred(user, ReadingOrder.desc, - false); + List statuses = findStarred(user, null, false); saveOrUpdate(markList(statuses, olderThan)); } public void markAllEntries(User user, Date olderThan) { - List statuses = findAll(user, true, ReadingOrder.desc, - false); + List statuses = findAll(user, true, null, false); saveOrUpdate(markList(statuses, olderThan)); }