1
tank1955
Fix For Missing Table Row Tag VideoTube v1.84
  • 2009/1/27 2:39

  • tank1955

  • Module Developer

  • Posts: 276

  • Since: 2007/9/7 1


A bug was reported in VideoTube v1.84 that causes right blocks to display twice. This only seems to affect certain themes and when video display format is set to 4-column.

The index.php file tracks the video thumbnail placement by assigning table cell numbers and row numbers to each thumbnail. When this data is passed to the video_display.html template, the template uses these assignments to determine where to place the <tr> and </tr> tags.

First open the modules/videotube/index.php file. Go to line 1049 and you will see the following code:

le="color: #000000"><?php $video_array['rownum']=$rownum; // Pass the video thumbnail information to the template $xoopsTpl->append('videos', $video_array);


Now comment out the video row assignment line by placing two forward slashes at the start of the line. After your changes the code should look like this:

le="color: #000000"><?php //$video_array['rownum']=$rownum; // Pass the video thumbnail information to the template $xoopsTpl->append('videos', $video_array);


Next go to line 977 in the same file. You will see the following code:

le="color: #000000"><?php while ($row = $xoopsDB->fetcharray($result)) { $postnum++; $video_array['toprow']=$toprow; $video_array['postnum']=$postnum; $video_array['ref'] = $row['id'];


Now insert a new line containing the video row assignment. After your changes the code should look like this:

le="color: #000000"><?php while ($row = $xoopsDB->fetcharray($result)) { $postnum++; $video_array['toprow']=$toprow; $video_array['postnum']=$postnum; $video_array['rownum']=$rownum; $video_array['ref'] = $row['id'];


This should take care of the missing tag issue.

2
tzvook
Re: Fix For Missing Table Row Tag VideoTube v1.84
  • 2009/1/27 9:01

  • tzvook

  • Just can't stay away

  • Posts: 875

  • Since: 2003/2/1 2


Hi Tank

10x for answering, but no change ... Still duplicating .