Skip to content

[stable13] Get correct version of an app - #9436

Merged
MorrisJobke merged 1 commit into
stable13from
backport/9373/stable13
May 14, 2018
Merged

[stable13] Get correct version of an app#9436
MorrisJobke merged 1 commit into
stable13from
backport/9373/stable13

Conversation

@rullzer

Copy link
Copy Markdown
Member

Backport of #9373

Related to #8929
We should get the version of the app. Not of the appfolder. Else there
is no way to properly compare the versions.
Now note that installing in 1 go will still fail. But at least on the
next page load the new version will be properly detected.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzerrullzer added bug 3. to review Waiting for reviews labels May 9, 2018
@rullzerrullzer added this to the Nextcloud 13.0.3 milestone May 9, 2018
@codecov

codecovBot commented May 9, 2018

Copy link
Copy Markdown

Codecov Report

Merging #9436 into stable13 will not change coverage.
The diff coverage is 0%.

@@ Coverage Diff @@## stable13 #9436 +/- ##
===========================================
Coverage 51.36% 51.36% Complexity 25051 25051 ===========================================
Files 1609 1609 Lines 95342 95342 Branches 1376 1376 ===========================================
Hits 48973 48973 Misses 46369 46369
Impacted FilesCoverage ΔComplexity Δ
lib/private/legacy/app.php57.44% <0%> (ø)222 <0> (ø)⬇️

@MorrisJobke
MorrisJobke merged commit 1aa149a into stable13May 14, 2018
@MorrisJobke
MorrisJobke deleted the backport/9373/stable13 branch May 14, 2018 11:33
@MorrisJobkeMorrisJobke mentioned this pull request May 31, 2018
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

@rullzer@nickvergessen@MorrisJobke