Skip to content

[stable9.1] getJailedPath expects $path to have a trailing / (#25703) - #25737

Merged
PVince81 merged 1 commit into
stable9.1from
stable9.1-99611d277521b84a943f68d48fe05f290127db7e
Aug 11, 2016
Merged

PVince81 merged 1 commit into
stable9.1from
stable9.1-99611d277521b84a943f68d48fe05f290127db7e

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Contributor

* getJailedPath expects $path to have a trailing / - fixes #25464

* Adding test case for getPathById including a jailed cache where root is just empty
@DeepDiver1975 DeepDiver1975 added this to the 9.1.1 milestone Aug 9, 2016
@mention-bot

Copy link
Copy Markdown

@DeepDiver1975, thanks for your PR! By analyzing the annotation information on this pull request, we identified @icewind1991, @rullzer and @PVince81 to be potential reviewers

@PVince81

PVince81 commented Aug 9, 2016

Copy link
Copy Markdown
Contributor

Code looks good 👍

@butonic

butonic commented Aug 9, 2016

Copy link
Copy Markdown
Contributor

👍

@daorte

daorte commented Aug 11, 2016

Copy link
Copy Markdown

I just applied the DIFF into my 9.1 instance and the shared link creation on a shared (external storage) folder works.

thanks

@PVince81
PVince81 merged commit 80e106f into stable9.1 Aug 11, 2016
@PVince81
PVince81 deleted the stable9.1-99611d277521b84a943f68d48fe05f290127db7e branch August 11, 2016 19:28
@lock

lock Bot commented Aug 5, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants