Skip to content

[stable9.1] Get user home folder before deletion#27492

Merged
DeepDiver1975 merged 1 commit into
stable9.1from
stable9.1-deleteuser-gethomeearly
Mar 28, 2017
Merged

[stable9.1] Get user home folder before deletion#27492
DeepDiver1975 merged 1 commit into
stable9.1from
stable9.1-deleteuser-gethomeearly

Conversation

@PVince81

Copy link
Copy Markdown
Contributor

Backport of #26848 to stable9.1

@DeepDiver1975 @jvillafanez @davitol

@PVince81 PVince81 added this to the 9.1.5 milestone Mar 24, 2017
@mention-bot

Copy link
Copy Markdown

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

@PVince81
PVince81 force-pushed the stable9.1-deleteuser-gethomeearly branch from 9571552 to 76166a5 Compare March 27, 2017 09:48
@PVince81

Copy link
Copy Markdown
Contributor Author

Fixed tests to run with older PHP versions

After the deletion getHome() will fail because the user doesn't exist
any more, so we need to fetch that value earlier.
@PVince81
PVince81 force-pushed the stable9.1-deleteuser-gethomeearly branch from 76166a5 to 14969ea Compare March 27, 2017 16:02
@DeepDiver1975

Copy link
Copy Markdown
Member

👍

@DeepDiver1975
DeepDiver1975 merged commit 535b14d into stable9.1 Mar 28, 2017
@DeepDiver1975
DeepDiver1975 deleted the stable9.1-deleteuser-gethomeearly branch March 28, 2017 07:33
@lock

lock Bot commented Aug 3, 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 3, 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.

3 participants