Katapult is utilizing non-standard SQL which results in SQL errors in PostgreSQL and other standards compiant DBMS'.

Bug #120575 reported by Michael D. Stemle, Jr.
12
Affects Status Importance Assigned to Milestone
katapult (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Set your amarok to use a PostgreSQL database (I'm using 8.2), import your library, and then try to get katapult to work with a music track. Notice that it doesn't show anything.

Watch your postgresql logs and you'll see errors such as:

2007-06-15 08:53:38 CDT STATEMENT: SELECT a.name, t.title, t.deviceid, d.lastmountpoint, t.url, i.path, album.name FROM tags t LEFT JOIN statistics s ON t.url = s.url AND t.deviceid = s.deviceid LEFT JOIN artist a ON t.artist = a.id LEFT JOIN album ON t.album = album.id LEFT JOIN images i ON ( a.name = i.artist AND album.name = i.album) LEFT JOIN devices d ON t.deviceid = d.id WHERE 1 AND (t.title LIKE "%Doc/F%" OR a.name LIKE "%Doc/F%") ORDER BY a.name, t.title, s.percentage DESC
2007-06-15 08:53:39 CDT ERROR: column "%Doc/F/%" does not exist at character 372

This is because double-quotes are being used in the SQL generated by Katapult when single-quotes are the standard for string literals in SQL.

Revision history for this message
Michael D. Stemle, Jr. (manchicken) wrote : Fixed.

Here's a patch that fixes this issue for me. I recommend that we have somebody running MySQL and somebody running SQLite try this patch out as well.

Revision history for this message
Anthony Mercatante (tonio) wrote :

katapult (0.3.1.4-0ubuntu6) gutsy; urgency=low

  * Add kubuntu_01_clean_SQL.patch
    Fixes LP: #120575

 -- Anthony Mercatante <email address hidden> Fri, 29 Jun 2007 17:25:09 +0200

Changed in katapult:
status: New → Fix Released
Martin Meredith (mez)
Changed in katapult:
assignee: nobody → mez
status: New → Fix Committed
Revision history for this message
Rich Johnson (nixternal) wrote :

This didn't fix the MySQL issue, I will mark bug 122699 a duplicate of this report as it was tested in MySQL and didn't work. Thanks!

Changed in katapult:
importance: Undecided → Medium
status: Fix Released → Confirmed
Revision history for this message
Martin Meredith (mez) wrote :

Fix released in sync from debian

katapult (0.3.2.1-3) unstable; urgency=low

  * Updated Slovak Translation .po (Closes: #441628)
  * Added patch to make katapult link against QT
  * Updated Rules for ability to build with dash (Closes: #459167)
  * Bumped Standards Version
  * Added Patch to fix issues with MySQL 5 BC Breaks

 -- Martin Meredith <email address hidden> Mon, 10 Mar 2008 16:41:04 +0000

Changed in katapult:
status: Confirmed → Fix Released
Martin Meredith (mez)
Changed in katapult:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.