Code ERRORS v1.3.7

a complete banner management system, any number of banners can be added
Forum rules
The content in this forum is dated Dec. 21 2005 and can be used as Archive only. This Forum is LOCKED and READ ONLY !

Code ERRORS v1.3.7

Postby Brando on Mon 18. Apr, 2005 21:42

Line 592:
Code: Select all
banner_type = '$banner_type', banner_width = '$banner_width', banner_height = '" . str_replace("\'", "''", $banner_width) . "',


Should be:
Code: Select all
banner_type = '$banner_type', banner_width = '$banner_width', banner_height = '$banner_height',


Line 610 (a break in the line, caused database error for me):
Code: Select all
VALUES ('$banner_id','" . str_replace("\'", "''", $banner_name) . "', '$banner_active', '$banner_spot', '" . str_replace("\'", "''", $banner_description) . "', '" . str_replace("\'", "''", $banner_url) . "', '$banner_click', '$banner_view', '".$owner['user_id']."', '$banner_level', '$banner_level_type', '$time_type', '$time_begin', '$time_end', '$date_begin', '$date_end', '" . str_replace("\'", "''", $banner_comment) . "','$banner_type',ø'" . str_replace("\'", "''", $banner_width) . "','" . str_replace("\'", "''",
$banner_height) . "','$banner_filter','$banner_filter_time','$banner_weigth')";


Should be:
Code: Select all
VALUES ('$banner_id','" . str_replace("\'", "''", $banner_name) . "', '$banner_active', '$banner_spot', '" . str_replace("\'", "''", $banner_description) . "', '" . str_replace("\'", "''", $banner_url) . "', '$banner_click', '$banner_view', '".$owner['user_id']."', '$banner_level', '$banner_level_type', '$time_type', '$time_begin', '$time_end', '$date_begin', '$date_end', '" . str_replace("\'", "''", $banner_comment) . "','$banner_type',ø'" . str_replace("\'", "''", $banner_width) . "','" . str_replace("\'", "''", $banner_height) . "','$banner_filter','$banner_filter_time','$banner_weigth')";
Brando
Poster
Poster
 
Posts: 22
Joined: Tue 22. Apr, 2003 02:06

Postby Beeveer on Tue 03. May, 2005 23:23

Another problem, many people have, is this: ø

Remove that token and it will work properly!

This line:

Code: Select all
VALUES ('$banner_id','" . str_replace("\'", "''", $banner_name) . "', '$banner_active', '$banner_spot', '" . str_replace("\'", "''", $banner_description) . "', '" . str_replace("\'", "''", $banner_url) . "', '$banner_click', '$banner_view', '".$owner['user_id']."', '$banner_level', '$banner_level_type', '$time_type', '$time_begin', '$time_end', '$date_begin', '$date_end', '" . str_replace("\'", "''", $banner_comment) . "','$banner_type',ø'" . str_replace("\'", "''", $banner_width) . "','" . str_replace("\'", "''", $banner_height) . "','$banner_filter','$banner_filter_time','$banner_weigth')";


Should be:

Code: Select all
VALUES ('$banner_id','" . str_replace("\'", "''", $banner_name) . "', '$banner_active', '$banner_spot', '" . str_replace("\'", "''", $banner_description) . "', '" . str_replace("\'", "''", $banner_url) . "', '$banner_click', '$banner_view', '".$owner['user_id']."', '$banner_level', '$banner_level_type', '$time_type', '$time_begin', '$time_end', '$date_begin', '$date_end', '" . str_replace("\'", "''", $banner_comment) . "','$banner_type','" . str_replace("\'", "''", $banner_width) . "','" . str_replace("\'", "''", $banner_height) . "','$banner_filter','$banner_filter_time','$banner_weigth')";
User avatar
Beeveer
brilliant supporter
 
Posts: 264
Joined: Tue 30. Nov, 2004 19:13
Location: Boskoop, South-Holland, the Netherlands

Postby Niels on Sat 07. May, 2005 14:57

thanks for your feed back, some how I missed these, I have uploaded a new version of banner_admin.php witch should corect these bugs.

thanks for reporting
(if a how-to is EM ready, it will mostly be bullet prof, since a machine is more picky than a human. :D)
User avatar
Niels
Poster
Poster
 
Posts: 4390
Joined: Sat 27. Jul, 2002 15:46


Return to Complete banner [2.0.6/EM]

Who is online

Users browsing this forum: No registered users and 1 guest

cron