New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when trying to Export/Import Metadata to/from Marvin #4

Closed
the3rdhbtaylor opened this Issue Sep 14, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@the3rdhbtaylor

the3rdhbtaylor commented Sep 14, 2013

Mac OS X 10.8.5
calibre 1.3
iOS Reader Applications plugin 1.1.0
MarvinXD plugin 1.0.1

Marvin XD recognizes the library and shows green/yellow/red/white status. When I double click on a yellow book that has metadata differences highlighted:

  1. "Import Metadata to Calibre" seems to work. The book status background changes to green. But if I go back in to MArvin XD, the book reverts to a yellow background.
  2. "Export metadata to Marvin" throws an Unhandled exception, with the following details.
calibre, version 1.3.0
ERROR: Unhandled exception: <b>TypeError</b>:can only concatenate list (not "NoneType") to list

calibre 1.3  isfrozen: True is64bit: True
Darwin-12.5.0-x86_64-i386-64bit Darwin ('64bit', '')
('Darwin', '12.5.0', 'Darwin Kernel Version 12.5.0: Mon Jul 29 16:33:49 PDT 2013; root:xnu-2050.48.11~1/RELEASE_X86_64')
Python 2.7.5
OSX: ('10.8.5', ('', '', ''), 'x86_64')
Traceback (most recent call last):
  File "calibre_plugins.marvin_manager.book_status", line 1078, in dispatch_double_click
  File "calibre_plugins.marvin_manager.book_status", line 1908, in show_view_metadata_dialog
  File "calibre_plugins.marvin_manager.book_status", line 5590, in _update_metadata
  File "calibre_plugins.marvin_manager.book_status", line 5442, in _update_marvin_metadata
  File "calibre_plugins.marvin_manager.book_status", line 2208, in _build_metadata_update
TypeError: can only concatenate list (not "NoneType") to list

The progress bar window stays on screen, and I have to exist calibre to remove it.

@GRiker

This comment has been minimized.

Collaborator

GRiker commented Sep 14, 2013

I've posted an update to Marvin XD (version 1.0.2) which should fix this problem.

@GRiker GRiker closed this Sep 14, 2013

@the3rdhbtaylor

This comment has been minimized.

the3rdhbtaylor commented Sep 14, 2013

I can confirm that item 2 is resolved (for most cases, see below), but item 1 still exits. When I try to import TO calibre, it looks like it works, but exiting Marvin XD and going back in still shows the item as yellow.

While the updated iOS driver and updated Marvin XD help with item 2, I see a new/different error, at least for a specific use case. If the Author Sort attribute has a mismatch, the following unhandled exception occurs when trying to export to Marvin:

calibre, version 1.3.0
ERROR: Unhandled exception: <b>TypeError</b>:list indices must be integers, not NoneType

calibre 1.3  isfrozen: True is64bit: True
Darwin-12.5.0-x86_64-i386-64bit Darwin ('64bit', '')
('Darwin', '12.5.0', 'Darwin Kernel Version 12.5.0: Mon Jul 29 16:33:49 PDT 2013; root:xnu-2050.48.11~1/RELEASE_X86_64')
Python 2.7.5
OSX: ('10.8.5', ('', '', ''), 'x86_64')
Traceback (most recent call last):
  File "calibre_plugins.marvin_manager.book_status", line 1078, in dispatch_double_click
  File "calibre_plugins.marvin_manager.book_status", line 1908, in show_view_metadata_dialog
  File "calibre_plugins.marvin_manager.book_status", line 5592, in _update_metadata
  File "calibre_plugins.marvin_manager.book_status", line 5488, in _update_marvin_metadata
TypeError: list indices must be integers, not NoneType

If I need to create this as a new issue, please let me know and I'd be happy to do so.

@GRiker

This comment has been minimized.

Collaborator

GRiker commented Sep 14, 2013

Yes, please create a new issue with a single problem.

Also, please include the specific Author Sort values in both calibre and Marvin causing the crash.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment