53 Commits (master)

Author	SHA1	Message	Date
Ferdinand Thiessen	d6d6747a73	refactor: apply rector rules for PHPUnit 10 ... Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>	2 months ago
Robin Appelman	aa15f9d16d	chore: run rector ... Signed-off-by: Robin Appelman <robin@icewind.nl>	6 months ago
Robin Appelman	29e39c0a2e	chore: run rector on tests ... Signed-off-by: Robin Appelman <robin@icewind.nl>	7 months ago
Joas Schilling	0d79156bb2	test: Speed up comments test ... Signed-off-by: Joas Schilling <coding@schilljs.com>	7 months ago
Joas Schilling	7304756e03	test: Sort the reactions before comparing ... Signed-off-by: Joas Schilling <coding@schilljs.com>	8 months ago
Joas Schilling	d46d2d1acf	test: Fix coding standards ... Signed-off-by: Joas Schilling <coding@schilljs.com>	8 months ago
Joas Schilling	10424f248b	test(comments): Make Comments test result output properly readable ... Signed-off-by: Joas Schilling <coding@schilljs.com>	8 months ago
Joas Schilling	e2cd436205	test: Improve comments manager test output in case of failure ... Signed-off-by: Joas Schilling <coding@schilljs.com>	9 months ago
Ferdinand Thiessen	db94e10af0	fix: Prevent breaking change in IQueryBuilder ... Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>	1 year ago
Ferdinand Thiessen	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>	1 year ago
Christoph Wurst	49dd79eabb	refactor: Add void return type to PHPUnit test methods ... Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	1 year ago
Anna Larch	6c36c54dc6	feat(db): switch from settype to casts ... Signed-off-by: Anna Larch <anna@nextcloud.com>	1 year ago
Daniel Kesselberg	af6de04e9e	style: update codestyle for coding-standard 1.2.3 ... Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>	1 year ago
Julius Härtl	9713dd3fa9	chore: Move comments event handler to use proper event dispatcher ... Signed-off-by: Julius Härtl <jus@bitgrid.net>	2 years ago
Andy Scherzinger	1f7e2ba599	chore: Add SPDX header ... Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>	2 years ago
Andrew Summers	6910832170	Refactor `OC\Server::getCommentsManager` ... Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>	2 years ago
Robin Appelman	eb5832b8fa	refactor: depricate getNumberOfUnreadCommentsForFolder and redo it's implementation based on getNumberOfUnreadCommentsForObjects ... Signed-off-by: Robin Appelman <robin@icewind.nl>	2 years ago
Joas Schilling	6056928dc6	feat(comments): Add a meta data column for comments ... Signed-off-by: Joas Schilling <coding@schilljs.com>	2 years ago
Vitor Mattos	5ed6722dd2	Cover expire_date with unit tests ... Signed-off-by: Vitor Mattos <vitor@php.rio>	3 years ago
Vitor Mattos	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>	3 years ago
Vitor Mattos	7a0648757a	Delete comments that expired now ... Signed-off-by: Vitor Mattos <vitor@php.rio>	3 years ago
Joas Schilling	198224bf7a	Allow to expire comments of multiple objects with one call ... Signed-off-by: Joas Schilling <coding@schilljs.com>	4 years ago
Vitor Mattos	3be1217c04	Follow the most common pattern used in project ... ``` git grep strval|wc -l 32 git grep "(string)"|wc -l 481 ``` Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Vitor Mattos	c59b0c2ff7	Add comments expire date ... https://github.com/nextcloud/spreed/pull/7327 Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Joas Schilling	902476fa20	Extract the EmojiService from user status and add an OCP interface ... Signed-off-by: Joas Schilling <coding@schilljs.com>	4 years ago
Joas Schilling	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>	4 years ago
Vitor Mattos	7bb01b6ccc	Refactor and fixes on tests ... Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Vitor Mattos	b9f74584a1	Fix check after change from string to mb_string ... Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Joas Schilling	189f9f96ce	Limit the summary and sort it afterwards ... Signed-off-by: Joas Schilling <coding@schilljs.com>	4 years ago
Vitor Mattos	f7cd995612	Order the reaction summary, improvements on validations ... Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Vitor Mattos	1ce894a50c	Disable reactions if database don't support utf8mb4 ... Fix column size Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Vitor Mattos	52affa9ddd	Skip if dont support 4 bytes UTF8 ... Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Vitor Mattos	425b5cf081	Add unit tests ... Signed-off-by: Vitor Mattos <vitor@php.rio>	4 years ago
Joas Schilling	1f1244fdcc	Fix PHP CS ... Signed-off-by: Joas Schilling <coding@schilljs.com>	5 years ago
Joas Schilling	8600e51df5	Fix manager creation and testing ... Signed-off-by: Joas Schilling <coding@schilljs.com>	5 years ago
Morris Jobke	234b510652	Change PHPDoc type hint from PHPUnit_Framework_MockObject_MockObject to \PHPUnit\Framework\MockObject\MockObject ... Signed-off-by: Morris Jobke <hey@morrisjobke.de>	5 years ago
Christoph Wurst	caff1023ea	Format control structures, classes, methods and function ... 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>	6 years ago
Christoph Wurst	afbd9c4e6e	Unify function spacing to PSR2 recommendation ... Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	6 years ago
Christoph Wurst	2ee65f177e	Use the shorter phpunit syntax for mocked return values ... Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	6 years ago
Roeland Jago Douma	3a7cf40aaa	Mode to modern phpunit ... Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>	6 years ago
Roeland Jago Douma	c007ca624f	Make phpunit8 compatible ... Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>	6 years ago
Joas Schilling	f50abde7ac	Add proper comment offset support ... The offset is based on the last known comment instead of limit-offset, so new comments don't mess up requests which get the history of an object- Signed-off-by: Joas Schilling <coding@schilljs.com>	8 years ago
Arthur Schiwon	d5f1cef642	fix comment sorter ... background: we have a flat hierarchy of comments, not a tree. therefore we can also remove again the unnecessary additions. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>	8 years ago
Arthur Schiwon	fd6daf8d19	AutoCompletion backend ... * introduce a Controller for requests * introduce result sorting mechanism * extend Comments to retrieve commentors (actors) in a tree * add commenters sorter * add share recipients sorter Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>	8 years ago
Joas Schilling	20f8d1094a	Can not insert auto increment on oracle ... Signed-off-by: Joas Schilling <coding@schilljs.com>	9 years ago
Robin Appelman	212d9fd277	Add test for getting unread comment count by folder ... Signed-off-by: Robin Appelman <robin@icewind.nl>	9 years ago
Roeland Jago Douma	eaf152efeb	Fix DateTime comparisson ... Datetime now returns microseconds. But if the database doesn't store those comparing them for equality most likely fails. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>	9 years ago
Arthur Schiwon	5d98ab83e9	resolve displayname via manager and registerable resolvers ... Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>	9 years ago
Arthur Schiwon	1bcd2ca8e3	emit pre-update event for comments ... * 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>	9 years ago
Arthur Schiwon	e1073cf442	Notificacations for simple @-mentioning in comments ... (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>	9 years ago