From: Rene R. <re...@gr...> - 2007-06-05 21:12:02
|
On Tue, 2007-06-05 at 13:12 -0700, Bishop wrote: > I just looked at CVS, and the most recent source seems to show line > 221 as something other than a comment. Are we all using the most > recent source? > > http://bobs.cvs.sourceforge.net/bobs/bobs/inc/class_backup.php?annotate=1.14 > 221 : return $result; > > The funny part is, $result doesn't appear to be set in that context at > all. While hecking to see if it ever *was* set, I did notice that the > diff from 1.11 to 1.12 seems to alter similar code: > > http://bobs.cvs.sourceforge.net/bobs/bobs/inc/class_backup.php?r1=1.11&r2=1.12&diff_format=u > > Was this one maybe overlooked for a while? Yeah, $results should have been removed. It doesn't do any harm so I haven't fixed it in cvs. $result was used before templates were used in BOBS. Cheers Rene |