tmplt / bookwyrm

ncurses utility for downloading publicly available ebooks, plugin support
MIT License
32 stars 5 forks source link

libgen: (line: 104): TypeError("'NoneType' object is not iterable",) #75

Closed tmplt closed 5 years ago

tmplt commented 5 years ago

From the simple query bookwyrm -t 'mass effect'. Line 104 points to self.bookwyrm.feed(item); are we extracting data from a py::dict incorrectly somewhere?

tmplt commented 5 years ago

The seemingly remaining part of a fix for this is in core::get_string(const py::dict &dict, const char *key) which throws an index error when returning the string: basic_string::substr: __pos (which is 4294967295) > this->size() (which is 45).

The py::str(dict[key]) operation passes, as does an explicit std::string str = py::str(dict[key]), but we're not able to return it; weird, because that is the deepest call in the item constructor for that dict.

tmplt commented 5 years ago

Closing in favour of #76.