aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Comments
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #47329 from nextcloud/feat/add-datetime-qbmapper-supportFerdinand Thiessen2024-10-181-10/+10
|\ | | | | feat(AppFramework): Add full support for date / time / datetime columns
| * fix: Prevent breaking change in IQueryBuilderFerdinand Thiessen2024-10-171-10/+10
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * feat(AppFramework): Add full support for date / time / datetime columnsFerdinand Thiessen2024-10-171-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds support for all Doctrine supported types, for the column types only the immutable variants needed to be added. But especially those types are the important ones, as our **Entity** class works by detecting changes through setters. Meaning if it is mutable, changes like `$entity->date->modfiy()` can not be detected, so the immutable types make more sense here. Similar the parameter types needed to be added. `Enity` and `QBMapper` needed to be adjusted so they support (auto map) those types, required when insert or update an entity. Also added more tests, especially to make sure the mapper really serializes the values correctly. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(comments): Document supported types and provide psalm typingJoas Schilling2024-10-171-19/+21
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | feat(comments): Support mentioning emailsJoas Schilling2024-10-171-1/+5
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-192-6/+6
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(psalm): Fix @throws annotationsfix/psalm/throws-annotationsprovokateurin2024-09-171-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Comment): Initialize childrenCount as integerfix/comment/children-count-integerprovokateurin2024-09-151-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-1/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-252-16/+16
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* chore: Move comments event handler to use proper event dispatcherJulius Härtl2024-06-261-0/+3
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* chore: Add SPDX headerAndy Scherzinger2024-05-243-69/+12
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* fix: Apply new coding standard to all filesCôme Chilliet2024-04-022-3/+3
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor: depricate getNumberOfUnreadCommentsForFolder and redo it's ↵Robin Appelman2024-03-071-60/+26
| | | | | | implementation based on getNumberOfUnreadCommentsForObjects Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat(comments): Allow mentions of federated users, groups and teams in the ↵Joas Schilling2024-02-271-1/+11
| | | | | | future Signed-off-by: Joas Schilling <coding@schilljs.com>
* feat(comments): Add a meta data column for commentsJoas Schilling2023-12-142-37/+40
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-231-8/+8
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* fix(comments): Use provided offset in best effort when loading commentsJoas Schilling2023-09-181-0/+16
| | | | | | | | | When we didn't find the "$lastKnownComment" the whole condition was ignored. Now we still use the ID as an offset. This is required as a fall-back for expired messages in Talk and deleted comments in other apps. Signed-off-by: Joas Schilling <coding@schilljs.com>
* Throw if creation date is read before inserting into databaseCôme Chilliet2023-06-202-2/+8
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Cleanup comments codeCarl Schwan2023-06-202-159/+76
| | | | | | | - Fix various psalm issues - Add as much typing as possible while preserving stable API Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Refactors "strpos" calls in lib/private to improve code readability.Faraz Samapoor2023-05-151-2/+2
| | | | Signed-off-by: Faraz Samapoor <fsamapoor@gmail.com>
* Update expire_date column of table commentsVitor Mattos2023-04-261-1/+2
| | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* Split the comments ids by chunksVitor Mattos2023-03-011-8/+16
| | | | | | | | | Split the comments ids by chunks to prevent error with Oracle database that can't do a query with more than 1000 parameters. https://github.com/nextcloud/spreed/issues/8287 Signed-off-by: Vitor Mattos <vitor@php.rio>
* composer run cs:fixCôme Chilliet2023-01-203-4/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Fix forwarding the object id when searching commentsJoas Schilling2022-11-161-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Delete comments that expired nowVitor Mattos2022-07-251-1/+1
| | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* Merge pull request #32865 from nextcloud/bugfix/remove-unecessary-rowblizzz2022-07-082-7/+2
|\ | | | | Remove unecessary row and make compatible with db schema
| * Set as optional argumentVitor Mattos2022-06-131-1/+1
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
| * Remove unecessary row and make compatible with db schemaVitor Mattos2022-06-132-6/+1
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | Allow to expire comments of multiple objects with one callJoas Schilling2022-07-011-5/+9
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Add comments expire dateVitor Mattos2022-06-152-0/+37
|/ | | | | | https://github.com/nextcloud/spreed/pull/7327 Signed-off-by: Vitor Mattos <vitor@php.rio>
* Create query parameters on correct objects to make debugging easierJoas Schilling2022-05-231-4/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix JSON error when comment has no reactionsJoas Schilling2022-05-091-11/+17
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #31703 from nextcloud/techdebt/noid/emojihelper-interfaceJoas Schilling2022-04-292-105/+5
|\ | | | | Emojihelper interface
| * Increase check if support reactionsVitor Mattos2022-04-281-1/+1
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
| * Extract the EmojiService from user status and add an OCP interfaceJoas Schilling2022-04-282-104/+4
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Make the order of reactions reliableJoas Schilling2022-04-281-0/+1
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Allow reactions also with other combined emojisJoas Schilling2022-03-242-2/+109
| | | | | | | | Honerful copying the EmojiService from UserStatus for now I think this should be put into a decent place and then reused, but I couldn't find somethin short term before beta1 Signed-off-by: Joas Schilling <coding@schilljs.com>
* Simplify comment search in special casesJoas Schilling2022-03-181-3/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Update ICommentsManager with reaction methodsVitor Mattos2022-02-031-34/+37
| | | | | | | | | | | | | Fix psalm errors Reorder methods and remove return null Use best pattern on docblock Goals: update https://github.com/ChristophWurst/nextcloud_composer/ with reaction methods. The script https://github.com/ChristophWurst/nextcloud_composer/blob/master/build.sh only get lib/public classes Signed-off-by: Vitor Mattos <vitor@php.rio>
* Merge pull request #30794 from ↵Joas Schilling2022-01-281-14/+41
|\ | | | | | | | | nextcloud/techdebt/noid/allow-to-get-comments-with-verb Allow to get comments with a given set of verbs
| * Allow to get comments with a given set of verbsJoas Schilling2022-01-211-14/+41
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix check after change from string to mb_stringVitor Mattos2022-01-211-2/+2
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | Fix CSJoas Schilling2022-01-211-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Throw if trying to save reaction without supportJoas Schilling2022-01-211-0/+4
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Only check mb_strlen()Joas Schilling2022-01-211-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Remove order from groupConcat as it is not working everywhereJoas Schilling2022-01-211-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Limit the summary and sort it afterwardsJoas Schilling2022-01-211-6/+15
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix groupConcat and ordering on OracleVitor Mattos2022-01-211-4/+5
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | Order the reaction summary, improvements on validationsVitor Mattos2022-01-211-13/+38
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
generated by cgit v1.2.3 (git 2.39.1) at 2025-07-25 18:16:36 +0000