Skip to content
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

Add new Banshee import plugin #3141

Merged
merged 10 commits into from Apr 14, 2019
Merged

Add new Banshee import plugin #3141

merged 10 commits into from Apr 14, 2019

Conversation

Phidica
Copy link
Contributor

@Phidica Phidica commented Mar 5, 2019

Check-list

  • There is a linked issue discussing the motivations for this feature or bugfix
  • Unit tests have been added where possible
  • I've added / updated documentation for any user-facing features. (Not applicable)
  • Performance seems to be comparable or better than current master

What this change is adding / fixing

Merging this pull request would add a new plugin that imports song information from the database of the Banshee music player, to improve the quality of life for users migrating from Banshee to QL. It was modelled after the pre-existing Rhythmbox importer and works similarly to match the URI stored in the Banshee db to songs QL has added. We import the ratings, play/skip counts, date of most recent play, and date added to library.

The Banshee database is SQLite, so we simply use the standard Python sqlite3 libary. No new dependencies are added.

I have written and included a test program for the importer.

The plugin is no more complex than the Rhythmbox importer, and, as Banshee has ceased further development, would require very little future maintenance.

catch an exception that was missed, fix use of ngettext,
fix path conversion
@Phidica
Copy link
Contributor Author

Phidica commented Apr 14, 2019

Any more comments on this?

@lazka lazka merged commit ad574e9 into quodlibet:master Apr 14, 2019
@Phidica Phidica deleted the bansheeimport branch April 15, 2019 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants