I appreciate also that you proposed a pull request (I do accept and merge them) even though in this precise case, I'm going to reject it and replace it with a commit of my own.
No ofense intended and the reason for that is that the change you propose is ok for python2 but would break python3 compatibility and since a few months the default branch of yapsy supports both. And there was also no need to add a new test data file as there are already plenty in the test/plugins folder.
The good news is: with the small compatibility utils (compat.py) used here we have a simple way to do that (using basestring).
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Thanks for reporting this bug and sorry for that.
I appreciate also that you proposed a pull request (I do accept and merge them) even though in this precise case, I'm going to reject it and replace it with a commit of my own.
No ofense intended and the reason for that is that the change you propose is ok for python2 but would break python3 compatibility and since a few months the default branch of yapsy supports both. And there was also no need to add a new test data file as there are already plenty in the test/plugins folder.
The good news is: with the small compatibility utils (compat.py) used here we have a simple way to do that (using basestring).
Bug fix commited at https://sourceforge.net/p/yapsy/code/ci/a0f2f129d572f47784067203aa37f3a27a3fa3fb/