Clan Adverts

Sponsors

CT on Facebook

Latest Product

Battlefield 3 Stats Module
Battlefield 3 Stats Module

User Box

Anonymous
38.107.179.219
Nickname:

Password:

Security Code
Security Code
Type Security Code


PND Downloads Feed

Phpnuke Downloads
How to Delete or Reset yo...
How to Delete or Reset your PHPNuke Admin Account
Lost your admin password ?  Cant login to your admin account ?This sh...
Back Online & Stuff
Back Online & Stuff
Look at us with our nifty website back up and running… Seriousl...
How to edit the admins in...
How to edit the admins in the forums
Here is a way to make someone admin in your forums on your Evo site. I...
Demo Splash Screen
Demo Splash Screen
Ped @ Clan Themes has released a simple splash screen entrance hack. ...
Free Phpnuke Business The...
Free Phpnuke Business Theme 6
This is the 6th Business theme for Phpnuke that Clan Themes has releas...
Tricked Out Slider
Tricked Out Slider
We have seen some great new things come out of Tricked Out News and th...

Custom Work

PHP Nuke Custom Work

www.clanthemes.com :: View topic - Fix for video stream
Fix for video stream

2 Replies / 760 Views
Post new topic   Reply to topic  

   www.clanthemes.com Forum Index » Bugs/Issues

View previous topic :: View next topic

clyde4210
Reputation: 395.3 Add RepSubtract Rep
Local time: 5:12 PM

blank.gif

 OMG
OMG

0.11 posts per day
Medals: 0
Joined: Oct 24, 2007
Last Visit: 24 Jul 2009
Posts: 167
Points: 7534 
Post Fix for video stream Posted: Sun Jul 12, 2009 2:44 pm
In video stream many users probably never noticed that there tables are out of alignment, rips the themes right border and footer. The fix is easy.

open watch.php
Find
Code:
//***************************************************************
// IF YOU WANT TO LEGALY REMOVE ANY COPYRIGHT NOTICES PLAY FAIR AND CHECK: http://www.steezmatic-designs.com/modules.php?name=Commercial_License
// COPYRIGHT NOTICES ARE GPL SECTION 2(c) COMPLIANT AND CAN'T BE REMOVED WITHOUT STEEZMATIC DESIGNS' AUTHOR WRITTEN AUTHORIZATION
// YOU'RE NOT AUTHORIZED TO CHANGE THE CODE UNTIL YOU ACQUIRE A COMMERCIAL LICENSE
// (http://www.steezmatic-designs.com/modules.php?name=Commercial_License)
//***************************************************************
echo "<br>\n";
OpenTable();
echo "HTTP Video Stream Module<br>By <a href=\"http://www.steezmatic-designs.com\">Steezmatic Designs</a>\n";
CloseTable();
// END OF COPYRIGHT

CloseTable();
include('footer.php');

Replace with
Code:
//***************************************************************
// IF YOU WANT TO LEGALY REMOVE ANY COPYRIGHT NOTICES PLAY FAIR AND CHECK: http://www.steezmatic-designs.com/modules.php?name=Commercial_License
// COPYRIGHT NOTICES ARE GPL SECTION 2(c) COMPLIANT AND CAN'T BE REMOVED WITHOUT STEEZMATIC DESIGNS' AUTHOR WRITTEN AUTHORIZATION
// YOU'RE NOT AUTHORIZED TO CHANGE THE CODE UNTIL YOU ACQUIRE A COMMERCIAL LICENSE
// (http://www.steezmatic-designs.com/modules.php?name=Commercial_License)
//***************************************************************
echo "<br>\n";
OpenTable();
echo "HTTP Video Stream Module<br>By <a href=\"http://www.steezmatic-designs.com\">Steezmatic Designs</a>\n";
CloseTable();
// END OF COPYRIGHT


include('footer.php');


The close table attribute was already called after the copyrite. No need to call it again
 

 
View user's profileSend private messageVisit poster's website Reply with quote

DreAdeDcoRpSE
Reputation: 2221.6 Add RepSubtract Rep
votes: 28
Local time: 5:12 PM
Location: Back of your Mind
usa.gif

 Forums Moderator
Forums Moderator

1.53 posts per day
Medals: 3 (View more...)
Super Dedicated User (Amount: 1)
Site Admin
Site Admin
Joined: Sep 22, 2007
Last Visit: 09 Feb 2012
Posts: 2459
Points: 128879 
Post Re: Fix for video stream Posted: Sun Jul 12, 2009 7:27 pm
Shop Purchases:Enemy Territory Theme (Aviator) for PHP Nuke · Clan Roster 2.0 ·  · Modern Warfare 3 Xtreme Theme
That would be correct except for 1 issue. Your stating that there are 2

CloseTable();

But if you look, there are 2

OpenTable();

You need 2 closing for the 2 openings. The correct fix is simple

Code:
 //***************************************************************
// IF YOU WANT TO LEGALY REMOVE ANY COPYRIGHT NOTICES PLAY FAIR AND CHECK: http://www.steezmatic-designs.com/modules.php?name=Commercial_License
// COPYRIGHT NOTICES ARE GPL SECTION 2(c) COMPLIANT AND CAN'T BE REMOVED WITHOUT STEEZMATIC DESIGNS' AUTHOR WRITTEN AUTHORIZATION
// YOU'RE NOT AUTHORIZED TO CHANGE THE CODE UNTIL YOU ACQUIRE A COMMERCIAL LICENSE
// (http://www.steezmatic-designs.com/modules.php?name=Commercial_License)
//***************************************************************
echo "<br>\n";
OpenTable();
echo "HTTP Video Stream Module<br>By <a href=\"http://www.steezmatic-designs.com\">Steezmatic Designs</a>\n";
CloseTable();
// END OF COPYRIGHT

CloseTable();
include('footer.php');


Just change that to this,

Code:
//***************************************************************
// IF YOU WANT TO LEGALY REMOVE ANY COPYRIGHT NOTICES PLAY FAIR AND CHECK: http://www.steezmatic-designs.com/modules.php?name=Commercial_License
// COPYRIGHT NOTICES ARE GPL SECTION 2(c) COMPLIANT AND CAN'T BE REMOVED WITHOUT STEEZMATIC DESIGNS' AUTHOR WRITTEN AUTHORIZATION
// YOU'RE NOT AUTHORIZED TO CHANGE THE CODE UNTIL YOU ACQUIRE A COMMERCIAL LICENSE
// (http://www.steezmatic-designs.com/modules.php?name=Commercial_License)
//***************************************************************
echo "<br>\n";
OpenTable();
echo "HTTP Video Stream Module<br>By <a href=\"http://www.steezmatic-designs.com\">Steezmatic Designs</a>\n";
CloseTable();
// END OF COPYRIGHT
include('footer.php');
CloseTable();
 


All I did was move the include('footer.php'); line from outside the last CloseTable(); to the inside.
 

 
View user's profileSend private messageVisit poster's website Reply with quote

clyde4210
Reputation: 395.3 Add RepSubtract Rep
Local time: 5:12 PM

blank.gif

 OMG
OMG

0.11 posts per day
Medals: 0
Joined: Oct 24, 2007
Last Visit: 24 Jul 2009
Posts: 167
Points: 7534 
Post Fix for video stream Posted: Mon Jul 13, 2009 12:30 pm
Thanks never noticed the other open table. Although I believe one open table at the beginning is enough and that second open table call is un-needed. The reason I say this because open table is telling it there will be transferring and receiving data. There is nothing being done inside that second open table at all, so one open and close table will work Without adding a extra un-nneded connection to the database to do nothing. It's not opening anything.

Removing that second open table and close table is the better way of handling it.
Code:
//***************************************************************
//***************************************************************
// IF YOU WANT TO LEGALY REMOVE ANY COPYRIGHT NOTICES PLAY FAIR AND CHECK: http://www.steezmatic-designs.com/modules.php?name=Commercial_License
// COPYRIGHT NOTICES ARE GPL SECTION 2(c) COMPLIANT AND CAN'T BE REMOVED WITHOUT STEEZMATIC DESIGNS' AUTHOR WRITTEN AUTHORIZATION
// YOU'RE NOT AUTHORIZED TO CHANGE THE CODE UNTIL YOU ACQUIRE A COMMERCIAL LICENSE
// (http://www.steezmatic-designs.com/modules.php?name=Commercial_License)
//***************************************************************
echo "<br>\n";
echo "HTTP Video Stream Module<br>By <a href=\"http://www.steezmatic-designs.com\">Steezmatic Designs</a>\n";
// END OF COPYRIGHT

include('footer.php');
CloseTable();


No need for that second open table or second close table as it is not needed. It is not sending or receiving data, it is plain old html in php form. Sure it will create it's own little table but nothing is being done inside of it other than displaying html. I guess it's preference but for me calling the database to do nothing is adding another connection to slow the script down.

Look at most or any basic install modules and you'll see they do not make little tables for there copyrites. I'm more concerned with keeping the site as fast as possible without making unnecessary database calls. Same would apply for the layout.php as well.

If Someone wanted it in it's own table they could use that copyrite another way.
 

 
View user's profileSend private messageVisit poster's website Reply with quote

Post new topic   Reply to topic  
   www.clanthemes.com Forum Index » Bugs/Issues


 
2 Replies / 760 Views
Page 1 of 1
All times are GMT
Display posts from previous:   
 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You can download files in this forum
 

Company Information
Home · About Us · Affiliates · Advertising
Link To Us · Newsletter · Contact Us · Site Map
All content © Clan Themes 2006 - 2012
Based on RavenNuke