Do not allow subtitile to be set for manual recordings

[billstei@…]

[Note the following original report was done with mythweb. Specifying the subtitle for manual recordings is not supported in the backend and mythweb should not allow it.]

When setting a manual recording the manually specified Subtitle is replaced incorrectly by the Startdate and Starttime and (re)placed into the "program" table. This occurs on line #3313 of scheduler.cpp in the Scheduler::UpdateManuals() method with the following code:

query.bindValue(":SUBTITLE", startdt.toLocalTime());

I recommend that the query of the "record" table on line #3222 of scheduler.cpp include "subtitle" along with an additional line ~3236:

QString subtitle = query.value(THE_VALUE_AFTER_TITLE).toString();

A simple test for the bug can be done by setting the Recording Started System Event to the following:

logger "The Subtitle is: %SUBTITLE%"

and then checking /var/log/syslog for the correct subtitle.

[gigem]

Specifying a subtitle for manual recordings is not supported and should not be allowed. How are you doing so? With mythweb? If so, it's a bug in mythweb for allowing it in the first place.

[billstei@…]

Yes, mythweb.

[gigem]

Thanks for following up.

[billstei@…]

Assuming that the Subtitle entry is removed from the mythweb Manual Recording Schedule UI, does the same not-available policy apply to the "description" field ? I see in the "record" table it is set to "Manually scheduled", but in the "program" table this is not copied and is blank. It would be nice to have at least one field available for manual comments (and via mythweb), which are passed to, in this case, %DESCRIPTION% as a System Event parameter to System Event handlers.

[gigem]

Yes, the same applies to the description field. It's another case where the mythweb developer over committed without knowing the underlying backend support. I'm sorry but I don't see a big enough need to let the user specify a subtitle or description in addition to the title. Manual recordings are expected to be a very seldom used, last resort when the guide data is not and cannot be correct. If you do intend to keep a manual recording long term for some reason and want better meta data, I believe there is already a way to edit that after the fact.

[sphery]

Yes, in Watch Recordings, highlight the recording and choose MENU|Recording Options|Change Recording Metadata, and then edit the title, subtitle, and/or description as desired.

[David Engel <david@…>]

In 64c2d2606832e5350e38c0441cde3d024ef9b4bf/mythweb:

Error: Processor CommitTicketReference failed

GIT backend not available