aboutsummaryrefslogtreecommitdiffstats
path: root/apps/comments/lib/Notification
Commit message (Collapse)AuthorAgeFilesLines
* feat(notifications): Migrate server INotifiers to new exceptionsJoas Schilling2024-06-251-5/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore: Add SPDX headerAndy Scherzinger2024-05-272-41/+8
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* feat: rename users to account or personVincent Petry2024-02-131-1/+1
| | | | | | Replace translated text in most locations Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Refactor comments appHamid Dehnavi2023-09-282-23/+7
| | | | Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
* Merge pull request #38605 from fsamapoor/replace_strpos_calls_in_comments_appCôme Chilliet2023-06-291-2/+2
|\ | | | | Refactors "strpos" calls in /apps/comments
| * Refactors "strpos" calls in /apps/comments to improve code readability.Faraz Samapoor2023-06-121-2/+2
| | | | | | | | Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>
* | Cleanup comments codeCarl Schwan2023-06-201-16/+7
|/ | | | | | | - Fix various psalm issues - Add as much typing as possible while preserving stable API Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Get rid of all useless calls to setParsedSubject now that setRichSubject is ↵Côme Chilliet2022-11-291-19/+0
| | | | | | able to do it Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Use user displayname cache for comment mentionsJoas Schilling2022-08-191-5/+5
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Modernize comments appCarl Schwan2022-03-162-21/+7
| | | | | | | | * Use php 7.4 typed properties * More typed methods (and replace existing php doc in some cases) * query -> get for Container Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* l10n: Correct spellingValdnet2021-12-151-2/+2
| | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-042-2/+0
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Bump nextcloud/coding-standard from 0.3.0 to 0.5.0dependabot-preview[bot]2021-02-181-2/+2
| | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 0.3.0 to 0.5.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v0.3.0...v0.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Improve mention matchesJoas Schilling2021-02-061-1/+5
| | | | | | The previous regex didn't correctly match users with . ' or - Signed-off-by: Joas Schilling <coding@schilljs.com>
* Update license headers for 19Christoph Wurst2020-04-292-0/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-102-14/+12
| | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use elseif instead of else ifChristoph Wurst2020-04-101-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove trailing and in between spacesChristoph Wurst2020-04-091-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headersChristoph Wurst2019-12-052-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update shipped implementations of the INotifierJoas Schilling2019-07-151-2/+25
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Populate the mention-notification with the actual messageJoas Schilling2018-11-081-41/+86
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Revert "Populate the mention-notification with the actual message"Arthur Schiwon2018-10-311-99/+41
| | | | This reverts commit d295ff53672d542f3b09ff0becd6a4a20ba1d494.
* Populate the mention-notification with the actual messageJoas Schilling2018-10-301-41/+99
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix unit testsMorris Jobke2017-11-271-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* that has since been deleted, ID to nodeAllan Nordhøy2017-11-271-3/+3
| | | | | | You were mentioned in a group chat by a user that has since been deleted Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update license headersMorris Jobke2017-11-062-2/+7
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix CommentsRoeland Jago Douma2017-08-011-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Get the user folder of the correct userJoas Schilling2017-03-091-6/+7
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Cleanup comment testsJoas Schilling2017-01-091-8/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Set the comment link on render instead of creationJoas Schilling2017-01-092-6/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Use the comment icon for the notificationsJoas Schilling2016-11-231-0/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Set the rich object string for comment notificationsJoas Schilling2016-11-231-13/+49
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* move mention extraction to (I)Comment and report mentions via DAVArthur Schiwon2016-10-191-13/+17
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* apply some polishArthur Schiwon2016-10-141-6/+0
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* emit pre-update event for commentsArthur Schiwon2016-10-121-1/+3
| | | | | | | * notifications can be cleaned up, no polluted DB * updating comments will re-notify users or remove notifications, depending on the message Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* adjust wording of the notificationArthur Schiwon2016-10-081-3/+3
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Notificacations for simple @-mentioning in commentsArthur Schiwon2016-10-072-0/+245
(WIP) notify user when mentioned in comments Fix doc, and create absolute URL for as notification link. PSR-4 compatibility changes also move notification creation to comments app Do not notify yourself unit test for controller and application smaller fixes - translatable app name - remove doubles in mention array - micro perf optimization - display name: special label for deleted users, keep user id for users that could not be fetched from userManager Comment Notification-Listener Unit Test fix email adresses remove notification when triggering comment was deleted add and adjust tests add missing @license tags simplify NotificationsController registration appinfo simplification, php docs make string easier to translate adjust test replace dispatcher-based listeners with a registration method and interface safer to not pass optional data parameter to setSubject for marking as processed. ID and mention suffices Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> update comment Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>