Opened 17 years ago

Closed 17 years ago

#7590 closed defect (fixed)

programinfo needs to init isVideo when loading from DB

Reported by: John Patrick Poet <jppoet@…> Owned by: danielk
Priority: minor Milestone: unknown
Component: MythTV - General Version: unknown
Severity: low Keywords:
Cc: Ticket locked: no

Description 

ProgramInfo::ProgramInfo(const MSqlQuery &query, ...)

Does not initialize isVideo. This causes ProgramInfo::ToMap() to randomly fill in starttime, endtime, startdate, etc., incorrectly.

Trivial patch attached.

Attachments (1)

programinfo-isVideo.patch​ (635 bytes ) - added by John Patrick Poet <jppoet@…> 17 years ago.
Initialize isVideo in ProgramInfo constructor from MSqlQuery

Download all attachments as: .zip

Change History (3)

by John Patrick Poet <jppoet@…>, 17 years ago

Attachment: programinfo-isVideo.patch​ added

Initialize isVideo in ProgramInfo constructor from MSqlQuery

comment:1 by danielk, 17 years ago

Owner: changed from Isaac Richards to danielk
Status: new → assigned

comment:2 by danielk, 17 years ago

Resolution: → fixed
Status: assigned → closed

(In [22854]) Fixes #7590. Initialize all ProgramInfo members in ctors.

This fixes the uninitialized variable John discovered, plus a couple other instances. It also reorders the clone function so as to make it easier to verify it in the future.

Note: See TracTickets for help on using tickets.