Skip to content

[stable10] get rid of "the share fields of the last share should include" step#35298

Merged
individual-it merged 2 commits into
stable10from
stable10-getRidOfShareFields
May 22, 2019
Merged

[stable10] get rid of "the share fields of the last share should include" step#35298
individual-it merged 2 commits into
stable10from
stable10-getRidOfShareFields

Conversation

@individual-it

Copy link
Copy Markdown
Member

backport of #35295

this was added in #35204 but causes problems when an invalid request is done but the original `lastShareData` need to be accessed e.g `\Sharing::getMimeTypeOfLastSharedFile`
@individual-it

Copy link
Copy Markdown
Member Author

CI failed with fatal: expected shallow/unshallow, got ERR upload-pack: not our ref 2cf25198cd16e95295a4afe645843d9a650a1187 restarting

@codecov

codecov Bot commented May 22, 2019

Copy link
Copy Markdown

Codecov Report

Merging #35298 into stable10 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             stable10   #35298   +/-   ##
===========================================
  Coverage       64.64%   64.64%           
  Complexity      20089    20089           
===========================================
  Files            1289     1289           
  Lines           76943    76943           
  Branches         1300     1300           
===========================================
  Hits            49738    49738           
  Misses          26821    26821           
  Partials          384      384
Flag Coverage Δ Complexity Δ
#javascript 53.85% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.78% <ø> (ø) 20089 <ø> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d52d3c7...0278bed. Read the comment docs.

1 similar comment
@codecov

codecov Bot commented May 22, 2019

Copy link
Copy Markdown

Codecov Report

Merging #35298 into stable10 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             stable10   #35298   +/-   ##
===========================================
  Coverage       64.64%   64.64%           
  Complexity      20089    20089           
===========================================
  Files            1289     1289           
  Lines           76943    76943           
  Branches         1300     1300           
===========================================
  Hits            49738    49738           
  Misses          26821    26821           
  Partials          384      384
Flag Coverage Δ Complexity Δ
#javascript 53.85% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.78% <ø> (ø) 20089 <ø> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d52d3c7...0278bed. Read the comment docs.

@individual-it individual-it merged commit 75b6118 into stable10 May 22, 2019
@delete-merged-branch delete-merged-branch Bot deleted the stable10-getRidOfShareFields branch May 22, 2019 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants