Opened 10 years ago

Closed 9 years ago

#9991 closed Bug Report - General (fixed)

Icon search for channels with "," in their name does not work

Reported by: dekarl@… Owned by: danielk
Priority: minor Milestone: 0.25
Component: MythTV - Mythtv-setup Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no


Attaching the logo to channel 'NDR 90,3' does not work. The search returns a candidate '"NDR 90' without a logo. Lyngsat has it here It appears like the comma is treated as a separator somewhere.

Change History (2)

comment:1 Changed 10 years ago by dekarl@…

Now that I'm done I see there's a nice error reported to the console, too. That's where the part after the comma ends up.

2011-08-17 18:49:11.041244 I  Fetch Icons for channel NDR 90,3
2011-08-17 18:49:16.084672 E  NetworkOperation Error on Finish: No server set to connect to (1): url: '3"'
2011-08-17 18:49:16.095192 E  Server returned an error status code 0 for url: 3"

Manually searching via and looking at makes me think that the csv extraction is broken.

The breakage in the csv parser makes quotes around a comma useless. (the quotes get nicely stripped from the string but have no function) This leaves escaping the "in string comma" with a backslash at as the easy fix (which should work for past releases instead of only future releases, too).

comment:2 Changed 9 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: newclosed

Fixes #9991. Replaces csv parser in icon downloader.

The existing code was overly complicated and didn't account for quotes.

Branch: master Changeset: f0deca75d6024ca36c0c5c26600eda4f846a8644

Note: See TracTickets for help on using tickets.