Opened 14 years ago

Closed 14 years ago

Last modified 12 years ago

#72 closed patch (fixed)

Default Theme

Reported by: Korey <k.m.fort@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: 0.18.1
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If a theme is set up inside mythtv and then that theme's directory is deleted after the set up, mythfrontend will not start up.

Attachments (2)

default_theme.diff (1.5 KB) - added by Korey <k.m.fort@…> 14 years ago.
Allows the default theme to be G.A.N.T. if the preferred theme cannot be found.
default_theme_no_warning.diff (948 bytes) - added by Korey <k.m.fort@…> 14 years ago.
This diff does the same thing with exception of not producing warnings of reaching the end non-void statement without return.

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by Korey <k.m.fort@…>

Attachment: default_theme.diff added

Allows the default theme to be G.A.N.T. if the preferred theme cannot be found.

Changed 14 years ago by Korey <k.m.fort@…>

This diff does the same thing with exception of not producing warnings of reaching the end non-void statement without return.

comment:1 Changed 14 years ago by Isaac Richards

This patch is completely broken. I don't understand why you're replacing the return with an exit, and even the 'no_warning.diff' version will fall off the end of that function without a return. You're also breaking the 'default' menu theme. Additionally, you're not following the indentation style.

comment:2 Changed 14 years ago by Isaac Richards

Resolution: fixed
Status: newclosed

(In [6789]) Redo the patch from #72 so that it's right. Fall back to gant if the selected theme's not found.

Closes #72.

Note: See TracTickets for help on using tickets.