Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

if object_name empty for file activity, derive file path from filecache info path #875

Merged
merged 1 commit into from
Jul 31, 2020

Conversation

mrow4a
Copy link
Contributor

@mrow4a mrow4a commented Jul 17, 2020

For some activities, no object_name is stored (for rename, tags, comment, tag). Thus path cannot be derived from object_name and preview are wrongly shown:

Zrzut ekranu 2020-07-17 o 17 31 28

This PR fixes it

Zrzut ekranu 2020-07-26 o 11 08 50

  • fix issue
  • add tests

@codecov
Copy link

codecov bot commented Jul 17, 2020

Codecov Report

Merging #875 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #875   +/-   ##
=========================================
  Coverage     84.12%   84.12%           
  Complexity      504      504           
=========================================
  Files            47       47           
  Lines          1896     1896           
=========================================
  Hits           1595     1595           
  Misses          301      301           
Impacted Files Coverage Δ Complexity Δ
lib/Controller/OCSEndPoint.php 97.12% <100.00%> (ø) 48.00 <0.00> (ø)

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 df3c30d...fd25909. Read the comment docs.

@mrow4a mrow4a force-pushed the bugfix/fix-not-found-preview branch from 05a808a to d0356b9 Compare July 21, 2020 18:10
@mrow4a mrow4a marked this pull request as ready for review July 21, 2020 18:11
@mrow4a mrow4a force-pushed the bugfix/fix-not-found-preview branch from d0356b9 to ca8da3a Compare July 21, 2020 18:11
@mrow4a mrow4a force-pushed the bugfix/fix-not-found-preview branch from ca8da3a to 611abc4 Compare July 26, 2020 09:06
@mrow4a
Copy link
Contributor Author

mrow4a commented Jul 26, 2020

@DeepDiver1975 could you re-review ?

Copy link
Member

@VicDeo VicDeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works

@karakayasemi karakayasemi force-pushed the bugfix/fix-not-found-preview branch from 611abc4 to fd25909 Compare July 31, 2020 12:16
@karakayasemi
Copy link
Contributor

Rebased to be able to merge

@karakayasemi karakayasemi merged commit f51e871 into master Jul 31, 2020
@delete-merged-branch delete-merged-branch bot deleted the bugfix/fix-not-found-preview branch July 31, 2020 13:00
@davitol davitol mentioned this pull request Aug 11, 2020
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants