Skip to content

[stable12] Removed cast to integer in getSize#5925

Merged
MorrisJobke merged 1 commit into
nextcloud:stable12from
derkostka:patch-1
Aug 8, 2017
Merged

[stable12] Removed cast to integer in getSize#5925
MorrisJobke merged 1 commit into
nextcloud:stable12from
derkostka:patch-1

Conversation

@derkostka

Copy link
Copy Markdown
Contributor

Fixes - Wrong or no sizes of files/folders #5031 for 32-bit systems a direct cast to integer causes problems.
Backport from #5744

Signed-off-by: Sebastian Kostka sebastian.kostka@gmail.com

Fixes - Wrong or no sizes of files/folders nextcloud#5031 for 32-bit systems a direct cast to integer causes problems.
Backport from nextcloud#5744

Signed-off-by: Sebastian Kostka <sebastian.kostka@gmail.com>
@mention-bot

Copy link
Copy Markdown

@derkostka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @icewind1991, @nickvergessen and @LukasReschke to be potential reviewers.

@derkostka

derkostka commented Jul 29, 2017

Copy link
Copy Markdown
Contributor Author

@LukasReschke @MorrisJobke, please review this backport

@derkostka derkostka changed the title [Stable 12] Removed cast to integer in getSize [stable 12] Removed cast to integer in getSize Jul 29, 2017
@derkostka derkostka changed the title [stable 12] Removed cast to integer in getSize [stable12] Removed cast to integer in getSize Jul 29, 2017
@codecov

codecov Bot commented Jul 29, 2017

Copy link
Copy Markdown

Codecov Report

Merging #5925 into stable12 will decrease coverage by 0.01%.
The diff coverage is 100%.

@@              Coverage Diff               @@
##             stable12    #5925      +/-   ##
==============================================
- Coverage       54.08%   54.07%   -0.02%     
  Complexity      22453    22453              
==============================================
  Files            1382     1382              
  Lines           85867    85867              
  Branches         1328     1328              
==============================================
- Hits            46445    46436       -9     
- Misses          39422    39431       +9
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/FileInfo.php 91.86% <100%> (ø) 60 <0> (ø) ⬇️
apps/comments/lib/EventHandler.php 79.16% <0%> (-8.34%) 7% <0%> (ø)
apps/files_trashbin/lib/Expiration.php 90.32% <0%> (-1.62%) 29% <0%> (ø)
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)
core/js/js.js 61.27% <0%> (-0.56%) 0% <0%> (ø)

@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Jul 31, 2017
@MorrisJobke MorrisJobke added this to the Nextcloud 12.0.2 milestone Jul 31, 2017
@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 1, 2017
@MorrisJobke MorrisJobke merged commit 3313c68 into nextcloud:stable12 Aug 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants