4teamwork / ftw.zipexport

Zip Export for Plone
2 stars 6 forks source link

Improve error handling when zipping is not allowed #26

Open shylux opened 10 years ago

shylux commented 10 years ago

When trying to use zip export on a non-allowed type it throws a NotFound error. This was coded this way because we assumed that the user has to enter the zipping url directly to reach this error. So throwing a NotFound error would make sence. However with this bug https://github.com/4teamwork/ftw.zipexport/issues/25 we have to question this decision. An appropriate error message which tells us that zipping on this content is not allowed would be much more helpful than just a NotFound. Maybe even a permission denied error would be enough.