[mythtv-commits] Ticket #3293: Autodiscovery Patch for MythFrontend

[MythTV]

#3293: Autodiscovery Patch for MythFrontend
--------------------+-------------------------------------------------------
 Reporter:  dblain  |        Owner:  dblain
     Type:  patch   |       Status:  new   
 Priority:  minor   |    Milestone:  0.21  
Component:  mythtv  |      Version:  head  
 Severity:  low     |   Resolution:        
  Mlocked:  0       |  
--------------------+-------------------------------------------------------

Comment(by Matthew Wire <devel at mrwire.co.uk>):

 I don't like the way the pin settings changed with the second patch.  IMO
 the methodology should be:[[BR]]
 blank = all access[[BR]]
 anything else = pin restricted[[BR]]
 [[BR]]
 I modified the checking line like this so that pin codes consisting of 0's
 will still be validated:
     if ((( sSecurityPin.length() != 0 ) || ( sSecurityPin.left(1) == "0"
 )) && ( sPin != sSecurityPin ))
 This also means that people upgrading don't suddenly get locked out of
 their setup because they haven't specified a pin (like I did :( )
 [[BR]]

 I also think the search button could be removed since servers pop up and
 disappear from the list anyway.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/3293#comment:5>
MythTV <http://svn.mythtv.org/trac>
MythTV