Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Clear content breaks if (some) forests are read-only.. #323

Closed
grtjn opened this issue Dec 15, 2014 · 3 comments
Closed

Clear content breaks if (some) forests are read-only.. #323

grtjn opened this issue Dec 15, 2014 · 3 comments

Comments

@grtjn
Copy link
Contributor

grtjn commented Dec 15, 2014

Clear on a read-only forest obviously not allowed. There could be a reason to not want clean content to happen, but better to disable clean content through app_specific override then. I think it makes sense to simply skip read-only forests at clean content..

@dmcassel
Copy link
Collaborator

Does clean content cause an error now if some forests are read-only, or does it just silently leave that content alone?

@grtjn
Copy link
Contributor Author

grtjn commented Dec 15, 2014

With 'breaks' I meant throws an error and potentially only clears partially, if at all

grtjn added a commit to grtjn/roxy that referenced this issue Jan 20, 2015
paxtonhare added a commit that referenced this issue Jan 20, 2015
Fixed #323: clean of read-only forests breaks
@grtjn
Copy link
Contributor Author

grtjn commented Jan 21, 2015

Fixed in dev

@grtjn grtjn closed this as completed Jan 21, 2015
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

2 participants