Skip to content

[stable13] format self-mentions, but don't offer them - #7932

Merged
danxuliu merged 1 commit into
stable13from
7914-stable13
Jan 18, 2018
Merged

[stable13] format self-mentions, but don't offer them#7932
danxuliu merged 1 commit into
stable13from
7914-stable13

Conversation

@blizzz

Copy link
Copy Markdown
Member

Backport of #7914

comments should compile mentions also if done by author
it is used by clients for formatting reasons, there is no reason not format
the author if her handle is included in the comment body.
It is unrelated to sending out notifications.
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
do not offer the handle of the current user for auto completion
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
add types to php doc
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@codecov

codecovBot commented Jan 18, 2018

Copy link
Copy Markdown

Codecov Report

Merging #7932 into stable13 will increase coverage by <.01%.
The diff coverage is 100%.

@@ Coverage Diff @@## stable13 #7932 +/- ##
==============================================
+ Coverage 51.23% 51.23% +<.01% - Complexity 24972 24974 +2 
==============================================
Files 1607 1607 Lines 95027 95031 +4 Branches 1376 1376 ==============================================
+ Hits 48683 48686 +3 - Misses 46344 46345 +1
Impacted FilesCoverage ΔComplexity Δ
lib/private/Comments/Comment.php96.26% <ø> (-0.07%)54 <0> (-1)
lib/private/Collaboration/Collaborators/Search.php76.92% <ø> (ø)12 <0> (ø)⬇️
...private/Collaboration/Collaborators/UserPlugin.php84.5% <100%> (+1.43%)21 <3> (+3)⬆️
lib/private/Files/Cache/Propagator.php94.93% <0%> (-1.27%)16% <0%> (ø)

@danxuliudanxuliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@danxuliu
danxuliu merged commit a0499e0 into stable13Jan 18, 2018
@danxuliu
danxuliu deleted the 7914-stable13 branch January 18, 2018 13:20
@MorrisJobkeMorrisJobke mentioned this pull request Jan 18, 2018
3 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@blizzz@rullzer@danxuliu