Skip to content

don't use deprecated method for requesting memcache - #7531

Merged
rullzer merged 3 commits into
masterfrom
ldap/createdistributed
Dec 18, 2017
Merged

don't use deprecated method for requesting memcache#7531
rullzer merged 3 commits into
masterfrom
ldap/createdistributed

Conversation

@blizzz

Copy link
Copy Markdown
Member

Quick and painless, create() already calls createDistributed(), effectively no change.

@blizzzblizzz added 3. to review Waiting for reviews feature: ldap technical debt 🧱 🤔🚀 labels Dec 15, 2017
@blizzzblizzz added this to the Nextcloud 13 milestone Dec 15, 2017
@codecov

codecovBot commented Dec 15, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7531 into master will increase coverage by <.01%.
The diff coverage is 72.22%.

@@ Coverage Diff @@## master #7531 +/- ##
============================================
+ Coverage 51.17% 51.18% +<.01% 
Complexity 24874 24874 ============================================
Files 1601 1601 Lines 94707 94707 Branches 1368 1368 ============================================
+ Hits 48469 48473 +4 + Misses 46238 46234 -4
Impacted FilesCoverage ΔComplexity Δ
apps/files_sharing/lib/External/Storage.php15.06% <0%> (ø)58 <0> (ø)⬇️
lib/private/Files/ObjectStore/Swift.php0% <0%> (ø)39 <0> (ø)⬇️
lib/private/TemplateLayout.php0% <0%> (ø)45 <0> (ø)⬇️
settings/ajax/uninstallapp.php0% <0%> (ø)0 <0> (ø)⬇️
core/Command/Maintenance/UpdateTheme.php73.33% <100%> (ø)3 <0> (ø)⬇️
lib/private/App/AppManager.php95.58% <100%> (ø)47 <0> (ø)⬇️
lib/private/IntegrityCheck/Checker.php81.85% <100%> (ø)58 <0> (ø)⬇️
apps/theming/lib/ThemingDefaults.php92.08% <100%> (ø)48 <0> (ø)⬇️
apps/user_ldap/lib/Connection.php57.87% <100%> (ø)119 <0> (ø)⬇️
lib/private/Server.php81.01% <100%> (ø)134 <0> (ø)⬇️
... and 8 more

@MorrisJobke

Copy link
Copy Markdown
Member

Let's also fix the other occurrences

blizzzand others added 3 commits December 18, 2017 20:57
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the ldap/createdistributed branch from e2e104c to 094d419CompareDecember 18, 2017 20:07

@rullzerrullzer 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.

Just fixed the test.
BUt other than that 👍

@rullzer
rullzer merged commit a84d11d into masterDec 18, 2017
@rullzer
rullzer deleted the ldap/createdistributed branch December 18, 2017 21:08
@MorrisJobkeMorrisJobke mentioned this pull request Jan 2, 2018
30 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: ldaptechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@MorrisJobke@rullzer@icewind1991