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

Update scanFile() and scanChildren() to the new signature of the parent ... #14682

Merged
merged 1 commit into from
Mar 4, 2015

Conversation

nickvergessen
Copy link
Contributor

...class

Fix #14681

@DeepDiver1975 should backport to stable8

@LukasReschke @butonic review please

@nickvergessen nickvergessen added this to the 8.1-current milestone Mar 4, 2015
@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@MorrisJobke
Copy link
Contributor

👍

@ghost
Copy link

ghost commented Mar 4, 2015

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/10183/
Test PASSed.

@PVince81
Copy link
Contributor

PVince81 commented Mar 4, 2015

👍

PVince81 pushed a commit that referenced this pull request Mar 4, 2015
…ated

Update scanFile() and scanChildren() to the new signature of the parent ...
@PVince81 PVince81 merged commit aa11b83 into master Mar 4, 2015
@PVince81 PVince81 deleted the issue/14681-noop-scanner-outdated branch March 4, 2015 09:07
@PVince81
Copy link
Contributor

PVince81 commented Mar 4, 2015

@karlitschek backport?

@DeepDiver1975
Copy link
Member

@karlitschek backport?

yes - issue was reported against oc8

@DeepDiver1975
Copy link
Member

@nickvergessen please open a backport pr - THX

@nickvergessen
Copy link
Contributor Author

you guys where quick this time.
Will try if we can get a test for this to avoid it in the future and then backport both

@karlitschek
Copy link
Contributor

yes. please backport

@lock lock bot locked as resolved and limited conversation to collaborators Aug 13, 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.

Internal Server Error after attempting to do "occ files:scan"
6 participants