salimoha / googlecl

Automatically exported from code.google.com/p/googlecl
0 stars 0 forks source link

--yes switch causes delete to fail if it comes after --title switch #408

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
> picasa delete --title ".*Eye.*" --yes
Traceback (most recent call last):
  File "/opt/local/bin/google", line 488, in run_interactive
    run_once(options, args)
  File "/opt/local/bin/google", line 627, in run_once
    task.run(client, options, args)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/googlecl/picasa/__init__.py", line 220, in _run_delete
    client.DeleteEntryList(entries, entry_type, options.prompt)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/googlecl/base.py", line 127, in delete_entry_list
    self.Delete(item.GetEditLink().href)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/googlecl/service.py", line 64, in retry_delete
    return self.retry_operation(*args, **kwargs)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/googlecl/base.py", line 387, in retry_operation
    raise unexpected
GooglePhotosException: (500, 'Internal Server Error', 'Error: 10 
STORAGE_UNAVAIL')

Expected promptless-delete of all albums with "Eye" in the title.  
Authentication already completed.

Snow Leopard, Porticus compile of googleCL through MacPorts (v0.93-1).

Listing works to list all images in all albums via:

> picasa list --title ".*Eye.*"

or listing all album titles via:

> picasa list-albums --title ".*Eye.*"

Delete with prompt succeeds:

> picasa delete --title ".*Eye.*"

And rearranging the --yes switch before the task also succeeds:

> picasa --yes delete --title ".*Eye.*"

So it seems to me that either the feature needs to be better documented or some 
of the token parsing logic needs work. ^_^

Original issue reported on code.google.com by nils...@gmail.com on 21 Jul 2011 at 4:21

GoogleCodeExporter commented 9 years ago
Tested on the code from trunk and Ubuntu 12.10. The `--yes` param works at all 
positions without any problems.

Original comment by szy...@endpoint.com on 29 Dec 2012 at 8:08