summaryrefslogtreecommitdiffstats
path: root/apps/comments/lib
Commit message (Collapse)AuthorAgeFilesLines
* Update license headers for 19Christoph Wurst2020-04-298-0/+8
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add visibility to all constantsChristoph Wurst2020-04-101-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-109-29/+22
| | | | | | | | | | | | | | | 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-094-5/+5
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-092-4/+4
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use exactly one empty line after the namespace declarationChristoph Wurst2020-04-094-4/+0
| | | | | | For PSR2 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix (array) indent style to always use one tabChristoph Wurst2020-04-091-5/+5
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update the license headers for Nextcloud 19Christoph Wurst2020-03-311-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #20170 from nextcloud/techdebt/remove-unused-importsChristoph Wurst2020-03-271-1/+0
|\ | | | | Remove unused imports
| * Remove unused importsChristoph Wurst2020-03-251-1/+0
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use the short array syntax, everywhereChristoph Wurst2020-03-261-1/+1
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headers for 18Christoph Wurst2019-12-201-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix comments search result to work with multibyte stringsMichał Węgrzynek2019-12-091-5/+5
| | | | | | Currently, the searching in comments breaks up, if comments contain multibyte characters and string manipulation logic in getRelevantMessagePart happens to cut through them rendering the resulting string invalid (not UTF-8 compliant). This patch replaces all string manipulating functions in this code to multibyte aware ones. Signed-off-by: Michał Węgrzynek <michal.wegrzynek@malloc.com.pl>
* Update license headersChristoph Wurst2019-12-0515-20/+31
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Some php-cs fixesRoeland Jago Douma2019-11-221-1/+0
| | | | | | | | | | | * Order the imports * No leading slash on imports * Empty line before namespace * One line per import * Empty after imports * Emmpty line at bottom of file Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix comments app init and LoadSidebar eventJohn Molakvoæ (skjnldsv)2019-11-043-7/+64
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Move over comments to the new eventRoeland Jago Douma2019-09-232-9/+49
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Keep the old method as a fallback and adjust the testsJoas Schilling2019-07-161-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Update shipped implementations of the INotifierJoas Schilling2019-07-152-11/+26
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Move comments to webpackRoeland Jago Douma2019-02-261-3/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #13819 from ↵Morris Jobke2019-02-071-1/+0
|\ | | | | | | | | nextcloud/bugfix/noid/unify-html-encoding-handling-with-other-ros-apps Unify the HTML encoding handling with other ROS apps
| * Unify the HTML encoding handling with other ROS appsJoas Schilling2019-01-251-1/+0
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | No need to explicitly load bundle dependenciesRoeland Jago Douma2019-02-041-1/+0
|/ | | | | | oc-webdav-backbone is loaded by default Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge the str_replacementsJoas Schilling2019-01-241-1/+1
| | | | 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>
* Fix the comments UI and activities for space-mentionsJoas Schilling2018-11-071-1/+6
| | | | 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>
* Redirect guests to login if they follow the link of a comment ↵Joas Schilling2018-10-021-17/+29
| | | | | | mention-notification Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add pagination supportJoas Schilling2018-07-251-17/+30
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Allow to search by commentsJoas Schilling2018-07-253-1/+208
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Correctly register everything via the Application classJoas Schilling2018-07-251-0/+51
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #8355 from nextcloud/fix-comments-num-indexMorris Jobke2018-02-261-2/+4
|\ | | | | Fix comments (and systemtags) when involving users with numerical ids
| * fix creating comments when file is accessible to users with numeric idsArthur Schiwon2018-02-141-3/+4
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | Remove unused import statementsMorris Jobke2018-02-142-2/+0
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix array method usages with a proper callMorris Jobke2018-01-251-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* 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-069-21/+36
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge branch 'master' into autocomplete-guiArthur Schiwon2017-11-011-0/+6
|\
| * Add a doc why this should not be removedJoas Schilling2017-10-231-0/+6
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix comment sorterArthur Schiwon2017-10-301-2/+13
| | | | | | | | | | | | | | 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>
* | configurable amount of autocomplete results in commentsArthur Schiwon2017-10-272-0/+50
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | ensure that sorting is stableArthur Schiwon2017-10-221-3/+14
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | AutoCompletion backendArthur Schiwon2017-10-221-0/+93
|/ | | | | | | | | | * 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>
* Use the wrapper to get the parameters which was added in #6369Joas Schilling2017-09-191-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Allow translations of the comments activity in the emails tooJoas Schilling2017-09-052-13/+35
| | | | | | | Message and object data is not available on emails atm, so we can not use them either... Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix CommentsRoeland Jago Douma2017-08-014-7/+7
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use PNG version of the icons for shipped activitiesJoas Schilling2017-06-201-1/+5
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>