Merge branch '1534-rss-notification' into 'master'

Make RSS blog posts not local: re-enables notification

Closes #1534

See merge request briar/briar!1079
This commit is contained in:
akwizgran
2019-04-18 08:01:16 +00:00
2 changed files with 5 additions and 1 deletions

View File

@@ -240,7 +240,9 @@ class BlogManagerImpl extends BdfIncomingMessageHook implements BlogManager,
MessageId postId = p.getMessage().getId();
BlogPostHeader h =
getPostHeaderFromMetadata(txn, groupId, postId, meta);
BlogPostAddedEvent event = new BlogPostAddedEvent(groupId, h, true);
boolean local = !b.isRssFeed();
BlogPostAddedEvent event =
new BlogPostAddedEvent(groupId, h, local);
txn.attach(event);
} catch (FormatException e) {
throw new DbException(e);

View File

@@ -310,6 +310,7 @@ public class BlogManagerImplTest extends BriarTestCase {
assertTrue(event instanceof BlogPostAddedEvent);
BlogPostAddedEvent e = (BlogPostAddedEvent) event;
assertEquals(blog1.getId(), e.getGroupId());
assertTrue(e.isLocal());
BlogPostHeader h = e.getHeader();
assertEquals(POST, h.getType());
@@ -360,6 +361,7 @@ public class BlogManagerImplTest extends BriarTestCase {
assertTrue(event instanceof BlogPostAddedEvent);
BlogPostAddedEvent e = (BlogPostAddedEvent) event;
assertEquals(rssBlog.getId(), e.getGroupId());
assertFalse(e.isLocal());
BlogPostHeader h = e.getHeader();
assertEquals(POST, h.getType());