Hi,
Op 23-08-11 22:35, Karl Johan Kleist schreef:
It just occurred to me, that it's probably not very smart to use the get_fs()
helper function in restore_main(). We can probably not assume that the
filestorage file already exists. So I've restored the old code in
restore_main()
As you indicate here, this might need a bit more thinking, as now a
restore could place the filestorage in a wrong location.
Also, collective.recipe.filestorage offers more ways to name the
additional filestorages and I guess your current code only handles the
default location. Fair enough, really, but I wonder if we can do better.
Also, it occured to me that these additional filestorages may come with
their own blobstorages (oh joy...). That may need a bit of thinking too.
I haved moved your code to a new branch:
https://svn.plone.org/svn/collective/buildout/collective.recipe.backup/branches/kleist-recipe-filestorage
I have also reverted those changes on trunk.
Biggest reason is that this clears the way for merging back my long
overdue blobstorage backup support branch and do a release. We can work
on collective.recipe.filestorage later.
Thanks for your contribution.
--
Maurits van Rees
Web App Programmer at Zest Software: http://zestsoftware.nl
Personal website: http://maurits.vanrees.org/
_______________________________________________
Product-Developers mailing list
[email protected]
https://lists.plone.org/mailman/listinfo/plone-product-developers
