Find programmers and grapic design experts at ScriptLance.com
<? DEW-CODE.COM  
Dew-Code
Welcome, Guest
Please Login or Register.    Lost Password?
Re:DNPL Ban List - Some issues (1 viewing) (1) Guest
Go to bottom Post Reply Favoured: 0
TOPIC: Re:DNPL Ban List - Some issues
#338
Rush (User)
Fresh Boarder
Posts: 8
graphgraph
User Offline Click here to see the profile of this user
Krieger-Lawrence Genealogy
DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 0  
Just a couple of quick things I noticed with the Ban List 1.0 Script:

Not critical:
In the dnplbanlist1.sql file the last line:
Code:

INSERT INTO phplinks_banlist VALUES(4, 'spammer...
Should read:
Code:

INSERT INTO phplinks_banlist VALUES(6, 'spammer...
Duplicate ID causes SQL error when setting up the database with this line:
Code:

INSERT INTO phplinks_banlist VALUES(4, 'cialis', 'string');
A little more critical: Maybe include in the installation.txt to add the database variable to the /include/config.php, e.g.:
Code:

// ******************************************************************* // MySQL Table Names - you don't need to change these if you use one of // the included .sql files to create the tables // *******************************************************************
all the variables then add this line at the end
Code:

$banlist_table = "phplinks_banlist";
Thanks, Rush
 
Report to moderator   Logged Logged  
 
Last Edit: 2008/06/15 09:44 By Rush.
 
  The administrator has disabled public write access.
#339
Dewed (User)
Administrator
Posts: 250
graph
User Offline Click here to see the profile of this user
Dew-Code.com
Re:DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 7  
Thanks for the feedback, I guess I added the email example a bit hastily

These issues have been fixed in the current download
 
Report to moderator   Logged Logged  
 
Nothing to it but to Dew it !
Dew-Code.com
  The administrator has disabled public write access.
#341
fattymattybrewing (User)
Support Your Local Brewer
Expert Boarder
Posts: 105
graphgraph
User Offline Click here to see the profile of this user
Gender: Male Fatty Matty Brewing Location: Southwestern Wisconsin Birthdate: 1974-11-05
Re:DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 3  
Wow now that's some keen eyes! Good catch!
 
Report to moderator   Logged Logged  
 
Lifes Too Short To Drink Crappy Beers
fattymattybrewing.com
  The administrator has disabled public write access.
#347
Rush (User)
Fresh Boarder
Posts: 8
graphgraph
User Offline Click here to see the profile of this user
Krieger-Lawrence Genealogy
Re:DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 0  
Thanks fattymattybrewing...

One more odd issue (finally got around to test the 'add a site' and I come up with this error).

Parse error: syntax error, unexpected $end in ~rootpath/include/add.php on line 352

Though, this is normally caused by an unclosed bracket, I can't seem to find one

Running PHP 5.1.2

Thanks,

Rush
 
Report to moderator   Logged Logged  
 
Last Edit: 2008/06/21 08:02 By Rush.
 
  The administrator has disabled public write access.
#348
Dewed (User)
Administrator
Posts: 250
graph
User Offline Click here to see the profile of this user
Dew-Code.com
Re:DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 7  
Um.. well that makes no sense. Works for me, but not for you? Let me tear into the download file and see what I can see.

PS. Actually I think this is typically caused by one too many closing brackets. For me anyways, I've usually typed a closing parenthetis instead of the curly bracket.. but I'm not finding that here.. still tearing

By the way, by chance did you also add the captcha routine ?
 
Report to moderator   Logged Logged  
 
Last Edit: 2008/06/21 11:12 By Dewed.
 
Nothing to it but to Dew it !
Dew-Code.com
  The administrator has disabled public write access.
#349
Rush (User)
Fresh Boarder
Posts: 8
graphgraph
User Offline Click here to see the profile of this user
Krieger-Lawrence Genealogy
Re:DNPL Ban List - Some issues 6 Months, 3 Weeks ago Karma: 0  
I don't have the captcha routine...

Just to double check, the old add.php works, but the new one coughs up the error.
I'll keep poking around to see if I can figure it also.

Thanks for checking into it,

Rush
 
Report to moderator   Logged Logged  
 
  The administrator has disabled public write access.
Go to top Post Reply
Powered by FireBoardget the latest posts directly to your desktop

Newsflash

Sign up for PayPal and start accepting credit card payments instantly.
Copyright Dew-Code 2008