[mythtv-commits] Ticket #5312: Don't cache default values in the settings cache

[MythTV]

#5312: Don't cache default values in the settings cache
----------------------------------------------+-----------------------------
 Reporter:  sphery <mtdean at thirdcontact.com>  |       Owner:  ijr    
     Type:  patch                             |      Status:  new    
 Priority:  minor                             |   Milestone:  unknown
Component:  mythtv                            |     Version:  head   
 Severity:  medium                            |     Mlocked:  0      
----------------------------------------------+-----------------------------
 If a setting is not found in the database, the settings cache will store
 the default value passed to the Get*Setting() function.  This means that
 future calls to Get*Setting() for that same setting value will end up with
 the default specified in the first call.

 The attached simple patch changes this behavior by simply caching the
 value pulled from the DB (if appropriate) before assigning the default
 value.  There is no new code in the patch, it simply rearranges 2 blocks
 of existing code.  Once applied, Myth's behavior when getting settings
 values will be the same whether the settings cache is active or not.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/5312>
MythTV <http://www.mythtv.org/>
MythTV