5dda39a61e
refactor(dbal): move to modern calls
...
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com >
Signed-off-by: Carl Schwan <carlschwan@kde.org >
2026-06-24 10:35:00 +02:00
2b7415ef4d
Merge pull request #60960 from nextcloud/carl/querybuilder-magic-number
...
refactor: Use new IQueryBuilder::MAX_IN_PARAMETERS
2026-06-10 23:09:30 +02:00
250635a9a8
Merge pull request #61150 from nextcloud/carl/speedup-tests
...
Speedup tests execution
2026-06-10 17:23:13 +02:00
0fee41df7c
perf: Test chunking with a lower amount of entries
...
Use a lower array_chunk values when running the code in the tests for
the comment manager.
Signed-off-by: Carl Schwan <carlschwan@kde.org >
2026-06-10 11:49:29 +02:00
b1e77f4294
fix(comments): Use capped memory cache for comments
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2026-06-09 07:17:59 +02:00
6e44f30652
fix(comments): load comments styles when the comments app is loaded
...
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de >
2026-06-08 16:51:56 +02:00
994565fec7
refactor: Use new IQueryBuilder::MAX_IN_PARAMETERS
...
Signed-off-by: Carl Schwan <carlschwan@kde.org >
2026-06-03 13:45:29 +02:00
1ab09ec753
chore: Apply new coding standard to all files
...
The diff can be checked using: git diff --ignore-all-space --ignore-blank-lines
To see only the changes not related to blank lines.
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com >
2026-06-01 13:46:39 +02:00
e0ba4d71b6
chore: add missing Override attribute to OC
...
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de >
2026-04-28 21:29:27 +02:00
7b6078875b
refactor: Run rector on lib/private
...
Signed-off-by: Carl Schwan <carl.schwan@nextcloud.com >
2026-02-06 13:50:18 +01:00
b4dfdf6492
refactor: Prefer using IFunctionBuilder than createFunction
...
Signed-off-by: Carl Schwan <carlschwan@kde.org >
2026-01-23 16:58:38 +01:00
3c9b937e28
refactor(comment): Port away from deprecated event comment constant
...
Create new events to replace deprecated CommentsEvent constant and use
them when creating CommentsEvents.
On the listener side, we can't yet use these events as deck still send
the old events.
Also fixes some issues reported by psalm level 3 on the comment app.
Signed-off-by: Carl Schwan <carl.schwan@nextcloud.com >
2025-12-04 17:38:53 +01:00
996be0f441
fix(comments): Don's catch invalid DriverException
...
These are nowadays also OCP\DB\Exception and if the id is not a numeric
deleting by id is also non working.
Signed-off-by: Carl Schwan <carl.schwan@nextclound.com >
2025-09-02 11:55:58 +02:00
c21b8169ff
refactor(querybuilder): Port away from qb::execute() in lib/
...
Replace by either executeStatement or executeQuery
Signed-off-by: Carl Schwan <carl.schwan@nextclound.com >
2025-09-02 11:55:56 +02:00
94e2af0302
perf(comments): Also cache the comments count
...
Since we now have an easy way to fetch the comments count.
Signed-off-by: Carl Schwan <carl.schwan@nextclound.com >
2025-08-21 11:30:56 +02:00
fa5548a1f3
perf(comments): Add a way to get comments for multiple objects at the same time
...
Signed-off-by: Carl Schwan <carl.schwan@nextclound.com >
2025-08-21 11:18:57 +02:00
6600c1ad86
feat(comments): Allow to filter by topmost parent id
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2025-07-10 12:58:46 +02:00
db94e10af0
fix: Prevent breaking change in IQueryBuilder
...
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de >
2024-10-17 18:31:44 +02:00
ee02e3246d
feat(AppFramework): Add full support for date / time / datetime columns
...
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 >
2024-10-17 18:31:42 +02:00
9836e9b164
chore(deps): Update nextcloud/coding-standard to v1.3.1
...
Signed-off-by: provokateurin <kate@provokateurin.de >
2024-09-19 14:21:20 +02:00
8af7ecb257
chore: adjust code to adhere to coding standard
...
Signed-off-by: Anna Larch <anna@nextcloud.com >
2024-09-05 21:23:38 +02:00
af6de04e9e
style: update codestyle for coding-standard 1.2.3
...
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de >
2024-08-25 19:34:58 +02:00
9713dd3fa9
chore: Move comments event handler to use proper event dispatcher
...
Signed-off-by: Julius Härtl <jus@bitgrid.net >
2024-06-26 18:13:07 +02:00
dae7c159f7
chore: Add SPDX header
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de >
2024-05-24 13:11:22 +02:00
ec5133b739
fix: Apply new coding standard to all files
...
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com >
2024-04-02 14:16:21 +02:00
eb5832b8fa
refactor: depricate getNumberOfUnreadCommentsForFolder and redo it's implementation based on getNumberOfUnreadCommentsForObjects
...
Signed-off-by: Robin Appelman <robin@icewind.nl >
2024-03-07 19:19:07 +01:00
6056928dc6
feat(comments): Add a meta data column for comments
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2023-12-14 14:32:00 +01:00
aa5f037af7
chore: apply changes from Nextcloud coding standards 1.1.1
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com >
2023-11-23 10:36:13 +01:00
4d2217073a
fix(comments): Use provided offset in best effort when loading comments
...
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 >
2023-09-18 20:09:32 +02:00
7aa97dcc23
Throw if creation date is read before inserting into database
...
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com >
2023-06-20 12:14:57 +02:00
9725d00329
Cleanup comments code
...
- Fix various psalm issues
- Add as much typing as possible while preserving stable API
Signed-off-by: Carl Schwan <carl@carlschwan.eu >
2023-06-20 10:55:20 +02:00
7ab50ab010
Update expire_date column of table comments
...
Signed-off-by: Vitor Mattos <vitor@php.rio >
2023-04-26 11:29:56 -03:00
e9295f739e
Split the comments ids by chunks
...
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 >
2023-03-01 13:08:17 -03:00
f5c361cf44
composer run cs:fix
...
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com >
2023-01-20 11:45:08 +01:00
3f4fed3550
Fix forwarding the object id when searching comments
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-11-16 12:08:34 +01:00
7a0648757a
Delete comments that expired now
...
Signed-off-by: Vitor Mattos <vitor@php.rio >
2022-07-25 11:30:20 -03:00
c98e44507d
Merge pull request #32865 from nextcloud/bugfix/remove-unecessary-row
...
Remove unecessary row and make compatible with db schema
2022-07-08 23:11:10 +02:00
198224bf7a
Allow to expire comments of multiple objects with one call
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-07-01 15:59:29 +02:00
c59b0c2ff7
Add comments expire date
...
https://github.com/nextcloud/spreed/pull/7327
Signed-off-by: Vitor Mattos <vitor@php.rio >
2022-06-15 11:58:29 -03:00
45cdb36d5c
Remove unecessary row and make compatible with db schema
...
Signed-off-by: Vitor Mattos <vitor@php.rio >
2022-06-13 13:35:34 -03:00
5b82ad0b30
Create query parameters on correct objects to make debugging easier
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-05-23 13:02:36 +02:00
4e0595fb0f
Fix JSON error when comment has no reactions
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-05-09 09:53:49 +02:00
0d87335751
Merge pull request #31703 from nextcloud/techdebt/noid/emojihelper-interface
...
Emojihelper interface
2022-04-29 18:31:29 +02:00
7c85f8dea4
Make the order of reactions reliable
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-04-28 16:13:41 +02:00
a75a22f3ac
Increase check if support reactions
...
Signed-off-by: Vitor Mattos <vitor@php.rio >
2022-04-28 07:09:44 -03:00
902476fa20
Extract the EmojiService from user status and add an OCP interface
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-04-28 07:09:44 -03:00
367be7c55f
Allow reactions also with other combined emojis
...
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 >
2022-03-24 15:13:09 +01:00
d05dca0c1c
Simplify comment search in special cases
...
Signed-off-by: Joas Schilling <coding@schilljs.com >
2022-03-18 09:22:16 +01:00
8ec7c5c8ae
Update ICommentsManager with reaction methods
...
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 >
2022-02-03 12:52:18 -03:00
e8c66d0f89
Merge pull request #30794 from nextcloud/techdebt/noid/allow-to-get-comments-with-verb
...
Allow to get comments with a given set of verbs
2022-01-28 16:41:42 +01:00
Git'Fellow
Benjamin Gaussorgues
Louis
Carl Schwan
Joas Schilling
Ferdinand Thiessen
Côme Chilliet
Carl Schwan
Carl Schwan
provokateurin
Anna Larch
Daniel Kesselberg
Julius Härtl
Andy Scherzinger
Robin Appelman
Carl Schwan
Vitor Mattos
blizzz
Joas Schilling