Opened 9 years ago

Closed 9 years ago

#9914 closed Bug Report - General (fixed)

No picture metadata is downloaded if title has Slash character

Reported by: yiannividalis@… Owned by:
Priority: minor Milestone: 0.25
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Trying to download metadata for a Nip/Tuck? episode I can see no pictures downloaded and the relevant database fields (screenshot, banner, fanart) are blank. There is no problem with other series' episodes.

This happens with current master

MythTV Version : v0.25pre-2687-g6443825
MythTV Branch : master
Network Protocol : 69
Library API : 0.25.20110707-1
QT Version : 4.7.3
Options compiled in:
 linux release use_hidesyms using_alsa using_jack using_oss using_pulse using_pulseoutput using_backend using_bindings_perl using_bindings_python using_bindings_php using_dvb using_frontend using_hdpvr using_iptv using_libfftw3 using_libxml2 using_lirc using_mheg using_opengl_video using_qtdbus using_qtwebkit using_v4l2 using_v4l1 using_x11 using_xrandr using_bindings_perl using_bindings_python using_bindings_php using_mythtranscode using_opengl using_vdpau using_ffmpeg_threads using_live using_mheg using_libxml2

Attachments (2)

pe_idl.diff (724 bytes) - added by robertm 9 years ago.
Percent encode titles.
no-slash.log (19.2 KB) - added by yiannividalis@… 9 years ago.
frontend log with -v database,general

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by robertm

Attachment: pe_idl.diff added

Percent encode titles.

comment:1 Changed 9 years ago by robertm

Status: newinfoneeded_new

Yianni,

Please try the attached patch. If it does not word, please provide frontend logs.

comment:2 Changed 9 years ago by yiannividalis@…

Thanks for the patch, Robert.

I tried it: The files are downloaded but no image info is inserted to the database. The plot and other info are inserted (I reset the episode's details between efforts and press w).

I am attaching the log with -v general,database.

Changed 9 years ago by yiannividalis@…

Attachment: no-slash.log added

frontend log with -v database,general

comment:3 Changed 9 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: infoneeded_newclosed

Replace forward slashes with a safe character in image downloads.

My gut tells me there *may* be another way to do this, but in Unix there are so few reserved characters (one, this one) that I'm comfortable with this simple replacement.

Fixes #9914, tested against the filename "Nip-Tuck 1x01.mpg". The resulting images replace forward slash with dash.

Branch: master Changeset: a38fd4f899c77efb8b135cf5c52e4dc92c98f85b

Note: See TracTickets for help on using tickets.