Opened 11 years ago

Closed 11 years ago

#4884 closed patch (invalid)

MythContext::SaveSettingOnHost fixing

Reported by: michael bishop <clever@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

the old way to save a setting was to delete then insert which seems a bit ugly and on a few ocasions has caused a setting to become lost on my end(crash between delete and insert)

this patch will insert or update in a single query and completely skip the insertorupdate if the setting isnt actualy being changed

Attachments (1)

contextdiff (1.8 KB) - added by michael bishop <clever@…> 11 years ago.
patch from 'svn diff'

Download all attachments as: .zip

Change History (2)

Changed 11 years ago by michael bishop <clever@…>

Attachment: contextdiff added

patch from 'svn diff'

comment:1 Changed 11 years ago by laga

Resolution: invalid
Status: newclosed

According to Michael Bishop, this patch is broken. It looks like the necessary database schema updates would break existing configs. Since he hasn't experienced this problem in months and the patch is just a workaround for broken MySQLds, I'm closing this ticket.

Note: See TracTickets for help on using tickets.