Skip to content

check userExists later, saves lookups for appData_INSTANCEID userids - #7361

Merged
MorrisJobke merged 1 commit into
masterfrom
cheap-check-first
Dec 1, 2017
Merged

check userExists later, saves lookups for appData_INSTANCEID userids#7361
MorrisJobke merged 1 commit into
masterfrom
cheap-check-first

Conversation

@blizzz

Copy link
Copy Markdown
Member

The easy and cheap check should be preferred. No need to request other user backends unnecessarily. For instance, I observed it was queried for appdata_$INSTANCEID.

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

codecovBot commented Dec 1, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7361 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

@@ Coverage Diff @@## master #7361 +/- ##
============================================
- Coverage 50.91% 50.78% -0.14% - Complexity 24698 24778 +80 
============================================
Files 1586 1586 Lines 94113 94358 +245 Branches 1361 1361 ============================================
- Hits 47921 47920 -1 - Misses 46192 46438 +246
Impacted FilesCoverage ΔComplexity Δ
apps/files_trashbin/lib/Storage.php78.94% <100%> (ø)35 <0> (ø)⬇️
lib/private/legacy/util.php49.08% <0%> (-9.61%)304% <0%> (+80%)
lib/private/Files/Cache/Propagator.php94.93% <0%> (-1.27%)16% <0%> (ø)
config/config.sample.php0% <0%> (ø)0% <0%> (ø)⬇️

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.

3 participants

@blizzz@rullzer@MorrisJobke