diff options
author | Frédéric Guillot <fred@miniflux.net> | 2018-04-07 14:20:42 -0700 |
---|---|---|
committer | Frédéric Guillot <fred@miniflux.net> | 2018-04-07 14:20:42 -0700 |
commit | 46d67acf224d333942ff87cfce9e60c2dd36ad0a (patch) | |
tree | eaa3f889ca1850392b8958f730632905bb4fbb9f | |
parent | 702256bcc0e829b49a875e7c2b70c5078ff2c665 (diff) |
Avoid unread counter to be off by one
-rw-r--r-- | ui/entry.go | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/ui/entry.go b/ui/entry.go index a67fa68..e2c3c00 100644 --- a/ui/entry.go +++ b/ui/entry.go @@ -266,12 +266,6 @@ func (c *Controller) ShowUnreadEntry(ctx *handler.Context, request *handler.Requ return } - args, err := c.getCommonTemplateArgs(ctx) - if err != nil { - response.HTML().ServerError(err) - return - } - builder = c.store.NewEntryQueryBuilder(user.ID) builder.WithStatus(model.EntryStatusUnread) @@ -301,6 +295,13 @@ func (c *Controller) ShowUnreadEntry(ctx *handler.Context, request *handler.Requ } } + // The unread counter have to be fetched after changing the entry status + args, err := c.getCommonTemplateArgs(ctx) + if err != nil { + response.HTML().ServerError(err) + return + } + response.HTML().Render("entry", args.Merge(tplParams{ "entry": entry, "prevEntry": prevEntry, |