Skip to content

the FN is optional, carrying the displayname if present - #8822

Merged
blizzz merged 1 commit into
masterfrom
fix/8641/undefined-index
Mar 19, 2018
Merged

the FN is optional, carrying the displayname if present#8822
blizzz merged 1 commit into
masterfrom
fix/8641/undefined-index

Conversation

@blizzz

Copy link
Copy Markdown
Member

fixes#8641

fixes a warning that is logged if the field is not present

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@codecov

codecovBot commented Mar 14, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8822 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@ Coverage Diff @@## master #8822 +/- ##
============================================
+ Coverage 51.89% 51.89% +<.01% - Complexity 25267 25269 +2 
============================================
Files 1605 1605 Lines 94771 94775 +4 Branches 1377 1377 ============================================
+ Hits 49180 49187 +7 + Misses 45591 45588 -3
Impacted FilesCoverage ΔComplexity Δ
...private/Collaboration/Collaborators/MailPlugin.php76.63% <100%> (+0.9%)31 <0> (+2)⬆️
apps/files_trashbin/lib/Trashbin.php72.46% <0%> (-0.25%)136% <0%> (ø)
lib/private/Files/ObjectStore/SwiftFactory.php57.14% <0%> (+4.39%)36% <0%> (ø)⬇️

@blizzz

Copy link
Copy Markdown
MemberAuthor

tested successfully (#8641 (comment)) taking as +1 → merging

@blizzz
blizzz merged commit 7ae42f1 into masterMar 19, 2018
@blizzz
blizzz deleted the fix/8641/undefined-index branch March 19, 2018 16:57
@blizzz

Copy link
Copy Markdown
MemberAuthor

backported to 13 only, patch does not apply to 12.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Errors from MailPlugin.php when trying to share files

2 participants

@blizzz@nickvergessen