From 079a73c92edb2a0786a6a916343f8f7b47ec06a1 Mon Sep 17 00:00:00 2001 From: Athou Date: Fri, 19 Jul 2013 12:02:20 +0200 Subject: [PATCH] fetch the entry instead of creating an empty one --- .../backend/services/FeedEntryService.java | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/commafeed/backend/services/FeedEntryService.java b/src/main/java/com/commafeed/backend/services/FeedEntryService.java index 10bbfeea..065c7f13 100644 --- a/src/main/java/com/commafeed/backend/services/FeedEntryService.java +++ b/src/main/java/com/commafeed/backend/services/FeedEntryService.java @@ -3,6 +3,7 @@ package com.commafeed.backend.services; import javax.ejb.Stateless; import javax.inject.Inject; +import com.commafeed.backend.dao.FeedEntryDAO; import com.commafeed.backend.dao.FeedEntryStatusDAO; import com.commafeed.backend.dao.FeedSubscriptionDAO; import com.commafeed.backend.model.FeedEntry; @@ -18,6 +19,9 @@ public class FeedEntryService { @Inject FeedSubscriptionDAO feedSubscriptionDAO; + + @Inject + FeedEntryDAO feedEntryDAO; public void markEntry(User user, Long entryId, Long subscriptionId, boolean read) { @@ -27,8 +31,10 @@ public class FeedEntryService { return; } - FeedEntry entry = new FeedEntry(); - entry.setId(entryId); + FeedEntry entry = feedEntryDAO.findById(entryId); + if (entry == null) { + return; + } FeedEntryStatus status = feedEntryStatusDAO.getStatus(sub, entry); @@ -61,8 +67,10 @@ public class FeedEntryService { return; } - FeedEntry entry = new FeedEntry(); - entry.setId(entryId); + FeedEntry entry = feedEntryDAO.findById(entryId); + if (entry == null) { + return; + } FeedEntryStatus status = feedEntryStatusDAO.getStatus(sub, entry);