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

\OCP\Files::searchByMime returns incorrect path for shared items #4516

Closed
VicDeo opened this issue Aug 19, 2013 · 5 comments
Closed

\OCP\Files::searchByMime returns incorrect path for shared items #4516

VicDeo opened this issue Aug 19, 2013 · 5 comments

Comments

@VicDeo
Copy link
Member

VicDeo commented Aug 19, 2013

Expected behaviour

Shared item should have a valid path in \OCP\Files::searchByMime result

Actual behaviour

it has path like '/Shared/files/filename.ext' that cannot be resolved correctly

Steps to reproduce

  1. Use \OC\Files\Filesystem::getFileInfo for any shared file in \OCP\Files::searchByMime

Server configuration

not relevant

ownCloud version:
master

@MTGap

@MTGap
Copy link
Contributor

MTGap commented Aug 24, 2013

I believe that will be the case for any of the methods in the shared cache. I'll try to correct that in #4382

Edit: Actually, that's not true. It's just for searchByMime.

@VicDeo
Copy link
Member Author

VicDeo commented Jan 31, 2014

Fixed in OC6

@VicDeo VicDeo closed this as completed Jan 31, 2014
@VicDeo
Copy link
Member Author

VicDeo commented Feb 3, 2014

Still doesn't work properly (For shared files in subdirectories only).

Steps:

  1. Login as A
  2. Create dir subdirectory
  3. Put a file test.odt in this directory
  4. Share the file with user B
  5. Login as B
  6. Use \OCP\Files::searchByMime('application/vnd.oasis.opendocument.text');

Expected result:
As user B sees it in the root of his Shared dir
path should be 'test.odt'

Actual result:
path is 'dir/test.odt'

Shared/dir/test.odt is a ticket to nowhere for B, since correct path is Shared/test.odt

@DeepDiver1975
Copy link
Member

@icewind1991 @schiesbn

@VicDeo
Copy link
Member Author

VicDeo commented Feb 6, 2014

Fixed with #7089

@VicDeo VicDeo closed this as completed Feb 6, 2014
@lock lock bot locked as resolved and limited conversation to collaborators Aug 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants