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

Fixed service registration so it won't break when no 'themes' folder yet exists. #65

Merged
merged 1 commit into from
Nov 22, 2017

Conversation

mpenna
Copy link
Contributor

@mpenna mpenna commented Nov 22, 2017

This handles the situation when config('themes.paths.absolute') returns either null or the returned path does not yet exist. Previously this situation would break the traversing of the themes root folder, in search of theme subfolders, which would throw a "must call one of in() or append() methods before iterating over a Finder" FileSystem exception.

@kaidesu kaidesu merged commit 5dea58e into caffeinated:master Nov 22, 2017
@kaidesu
Copy link
Contributor

kaidesu commented Nov 22, 2017

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants