Due to #1766 some Feeds might now have items that have `null` set as author instead of `""` clients need to handle this.
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Add API v1.3 adding routes for starring/unstarring items by id and general fixes (#1727)
https://nextcloud.github.io/news/api/api-v1-3/
- Improve styling of tables in articles (#1779)
- Allow fetching feeds that omit guid by using link as stand-in (#1785)
Fixed
- Fix updated api not returning any item after marking item as read (#1713)
- Fix deprecation warning for strip_tags() on a null value (#1766)
- Fix selected item being set incorrectly when using default ordering or newest first ordering (#1324)
- Fix doubling the height of the content area (#1796)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
* Use Feed Link as GUID when Feed omits Guid.
As noted in nextcloud/news#1702 some feeds omit the GUID and are
therefore not a valid RSS feed.
nextcloud/news#1738 resolved the issue to allow valid feeds to
update correctly when an invalid feed is present.
This commit allows parsing of the invalid feed as well by assuming
that the item link of the feed is unique to the feed and using
it in place of the GUID when the feed omits the GUID.
This will allow NextCloud News to accept and behave like many other
popular feed aggregators when presented with such an invalid feed.
Signed-off-by: Accalia <Accalia@Elementia.me>
* Add basic Logging when using fallback guid
Signed-off-by: Accalia <Accalia@Elementia.me>
* Add basic Logging when using fallback guid - Fix Fatfinger Typo
Signed-off-by: Accalia <Accalia@Elementia.me>
* Add basic Logging when using fallback guid - Update tests to account for additional logging
Signed-off-by: Accalia <Accalia@Elementia.me>
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
this way it works
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
add changelog entry
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Partly fix test
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
test passing
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Fixed
- Fix import of items when feed does not exist (1742)
- Fix malformed feeds (without GUIDs) stopping the update process (#1738)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Display the message instead of the response when catching an exception
during feed logo fetching. As there is no response and thus no method
`getResponse()` in `ConnectException`, this fixes a "call to undefined
method"-error that prevented feeds from fetching if fetching the logo
resulted in a `ConnectException`.
Signed-off-by: Michael Hamann <michael@content-space.de>
Changed
- Drop support for Nextcloud 20 (#1514)
- Use better sql commands, that were not possible with Nextcloud 20 (#1514)
- Add support for Nextcloud 23 (#1585)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Updated "New Folder" and "All articles" icons to differentiate them from "Subscribe" and "All articles". (#1542)
Fixed
- Mark the latest post in a feed as read when clicking on the right arrow key. (#1546)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Give both add folder and unread feed icons their own unique icons instead of sharing them with subscribe and all articles.
Signed-off-by: Greg Ross <greg@toolstack.com>
Changed
- Add changelog and DCO notice to CONTRIBUTING.md (#1521)
- Download feed logos via guzzle to have better error handling (#1533)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Remove dependency's large test files from release (#1519)
- Fix spelling of "receive" in log files (#1520)
Note: Nextcloud 20 support will be dropped in Oct 2021, this is very likely the last version to support Nextcloud 20. This also means that PHP 7.2, will no longer be supported by news.
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Added new `news:updater:update-user` command to update the feeds of a single user (#1360).
Fixed
- Removed spurious requests for `.../apps/news/%7B%7B%20::Content.getFeed(item.feedId).faviconLink%20%7D%7D` (#1488)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
The markup for favicon `<img>` element in feed item used `src` instead of
`ng-src`, causing browser to make broken requests for
`.../apps/news/%7B%7B%20::Content.getFeed(item.feedId).faviconLink%20%7D%7D`
Changing the attribute to `ng-src` fixed that.
Signed-off-by: mormegil <mormegil@centrum.cz>
There are no additional changes compared to the latest beta.
Changed
- News now requires a 64bit OS
- v2 API implementation (folder part)
- Implemented sharing news items between nextcloud users (#1191)
- Updated the news items table in DB to include sharer data (#1191)
- Added route for sharing news items (#1191)
- Added share data in news items serialization (#1191)
- Added tests for the news items share feature (#1191)
- Added sharing articles with nextcloud users (#1217)
- Added sharing articles on social media (Facebook, Twitter) or mail (#1217)
- Allow installation on Nextcloud v22
- Remove deprecated API endpoints and occ command (#935)
- /api/v1-2/user
- /api/v1-2/user/avatar
- ./occ news:updater:all-feeds
- added feed search (#1402)
Fixed
- allow calling `/items?getRead=false` without a feed/folder (#1380#1356)
- newestId does not return newest ID but last updated (#1339)
- removed reference for deleted repair-steps (#1399)
- Fix NotNullConstraintViolation when sharing news items with users (#1406)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Allow installation on Nextcloud v22
- Remove deprecated API endpoints and occ comand (#935)
- /api/v1-2/user
- /api/v1-2/user/avatar
- ./occ news:updater:all-feeds
Fixed
- allow calling `/items?getRead=false` without a feed/folder (#1380#1356)
- newestId does not return newest ID but last updated (#1339)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- News now requires a 64bit OS
- v2 API implementation (folder part)
- Implemented sharing news items between nextcloud users (#1191)
- Updated the news items table in DB to include sharer data (#1191)
- Added route for sharing news items (#1191)
- Added share data in news items serialization (#1191)
- Added tests for the news items share feature (#1191)
- Added sharing articles with nextcloud users (#1217)
- Added sharing articles on social media (Facebook, Twitter) or mail (#1217)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Add BATS as integration tests (#1213)
- Update FeedFetcher to import categories from feeds (#1248)
- Update serialization of item to include categories (#1248)
- Make PHPStan stricter (#955)
- Search: Add folder search (#1215)
- Improve test coverage (#1263)
- Allow directly adding a feed without going through the discovery process (#1265)
Fixed
- Do not show deleted feeds in item list (#1214)
- Fix update queries (#1211)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed
- Remove outdated item DB code. ( #1056)
- Stop returning all feeds after marking folder as read. (#1056)
- Always fetch favicon (#1164)
- Use feed logo instead of favicon if it exists and is square (#1164)
- Add CI for item lists (#1180)
Fixed
- Item list throwing error for folder and "all items" (#1180)
- Articles with high IDs can be placed lower than articles with low IDs (#1147)
- Feeds are accidentally moved on rename (#1189)
- Item list not using ID for offset (#1188)
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Currently, an article with an ID of 1000 will show up
earlier than one with ID = 900, because the IDs are being treated
as strings and compared alphabetically. (1 comes before 9, and
the comparison stops there.)
We need to parse them as integers to ensure that 900 is less than
1000.
Signed-off-by: Tucker McKnight <tucker.mcknight@gmail.com>
The logo of the feed is prefered if it is a square picture,
else the favicon is returned.
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Co-authored-by: Sean Molenaar <SMillerDev@users.noreply.github.com>
