[mythtv-commits] Ticket #1118: mythweb: editing power search yeilds sql error

[MythTV]

#1118: mythweb: editing power search yeilds sql error
------------------------------+---------------------------------------------
 Reporter:  jonese at zener.com  |        Owner:  xris    
     Type:  defect            |       Status:  reopened
 Priority:  minor             |    Milestone:  0.19    
Component:  mythweb           |      Version:          
 Severity:  medium            |   Resolution:          
------------------------------+---------------------------------------------
Comment (by derek at battams.ca):

 I actually noticed this when writing my patch for #1144, but forgot about
 it until I read this ticket.  The problem, I believe, is on line 52 of
 mythweb/modules/tv/schedules_custom.php:

 Is: case rectype_once:
 Should be: case rectype_findone:

 The other issue that caught my eye is that if the code chooses to ignore
 the schedule in this case statement then it sets the $_POST['record']
 value to zero.  I found this odd because a value of zero means to delete
 the rule, which explains the behaviour described.  I'd suggest just
 bailing out with a reload and a JS alert saying 'request ignored' or
 something like that.  I think deleting the rule is the incorrect path in
 this case, but didn't touch the code because I'm not sure if there's a
 reason for the choice.  But changing the one line above should fix the bug
 described.

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