search

plone / ploneorg.addonlisting

Add-on listing for Plone.org
https://plone.org
1 stars 1 forks source link

Cli script #21

Closed ghost closed 7 years ago

ghost commented 7 years ago

added cli-arguments, parsed them and modified utils accordingly

ghost commented 7 years ago

usage: bin/instance -O update_addon_listing < Path to AddOnFolder relative to Zope root > bin/instance -O update_addons < Path to AddOnFolder relative to Zope root > bin/instance -O update_addon < Path to AddOn relative to Zope root >

problems: when run command bin/instance -O Plone2 update_addon_listing Plone2/add-ons from command line, ERROR ploneorg.addonlisting-cli Could not create collective.documentgenerator ERROR ploneorg.addonlisting-cli can't compare datetime.datetime to NoneType ERROR ploneorg.addonlisting-cli Could not create plone.resourceeditor ERROR ploneorg.addonlisting-cli can't compare datetime.datetime to NoneType ERROR ploneorg.addonlisting-cli Could not create Plone ERROR ploneorg.addonlisting-cli The id "Plone" is reserved.

"can't compare datetime.datetime to NoneType" error when running update_all_listing from browserview

loechel commented 7 years ago

@pavithirakc sorry I could not reproduce that, on my system I only get the 

ERROR ploneorg.addonlisting-cli Could not create Plone
ERROR ploneorg.addonlisting-cli The id "Plone" is reserved.

By the way, I did not get the right logging output, no INFO level output on verbose.

loechel commented 7 years ago

I have removed the transaction call around api creation, as that is the wrong way in this context. plone.protect intercepts here. I will look into it, so no problem at the moment for you @pavithirakc

loechel commented 7 years ago

I closed this pull for now, any additional work please into a new branch.