entry.parentgroup doesn't work for history entries about pykeepass HOT 5 CLOSED

BTW: the representation of the items under python3 looks a bit strange, the leading "b" should not be part of the representation but indicate a type, but I'm no specialist

from pykeepass.

I have a possible solution and some tests at https://github.com/BoomerB/pykeepass
but when I try a pull request it also contains the changes for my previous pull request.

from pykeepass.

BTW: the representation of the items under python3 looks a bit strange, the leading "b" should not be part of the representation but indicate a type, but I'm no specialist

I agree with this. The issue is the __str__ function actually returns bytes instead of a str.

https://github.com/pschmitt/pykeepass/blob/0b6240866bdc62d04d41ba7c86cf010ad827ce3f/pykeepass/entry.py#L270

May deserve it's own ticket

from pykeepass.

@whwright the question is: what type should be returned? is it ok to return an unicode string - which is the default for python3 anyway - or does it have to be a plain string?
the easiest would be

return 'Entry: "{} ({})"'.format(item.path, item.username)

if it has to be a plain str, then something like

s = 'Entry: "{} ({})"'.format(item.path, item.username)
if not isinstance(s, str):
    s = s.encode('utf-8')
return s

would do the trick
BTW: we probably should add some tests with non-ascii groups and entries

from pykeepass.

BTW: we probably should add some tests with non-ascii groups and entries

There are entries with cyrillic characters in one of the test databases.

from pykeepass.