:: Forums ⇒

#76: Re: Drastic CVS / Dragonfly 9.1.x changes Author: alva, Location: The Netherlands

Posted: Mon Jan 23, 2006 11:35 am
----

DJMaze wrote:

And what does empty(false) is according to php.net/empty ?
It's similar to dragonflycms.org/Proje...d=731.html

Please investigate before you report a bug.

DJMaze, I do investigate first and I had visited the page you're linking to, be it the auto-translation. Hope you don't mind my occasional mistake and please forgive me for not seeing the similarity to JamieWolf's report on missing "?>".

Thanks for fixing the HTTP_X_FORWARD_FOR.

Any comments on my ban_ipn padding report? I can see in my _security table that the entries are not padded, even though it's a varchar(17) BINARY.

Thanks for your time, I'm really trying not to waiste it and I do compensate my asking here with al lot of answering elsewhere.

#77: Re: Drastic CVS / Dragonfly 9.1.x changes Author: alva, Location: The Netherlands

Posted: Tue Jan 24, 2006 9:22 am
----
I went ahead with padding $ip ('$ip/') where ban_ipn is involved in includes/classes/security.php. Now for the admin-panel to show ip's properly I modified admin/modules/security.php.

PHP:


if ($result = $db->query('SELECT * FROM '.$prefix."_security WHERE ban_type = 0 ORDER BY ban_string $limit")) {
    while ($row = $db->fetch_array($result, SQL_ASSOC)) {
        if (!empty($row['ban_ipn'])) {
            $ip = inet_ntop($row['ban_ipn']);

I changed the last line in the snippet (151) to:

PHP:


$ip = inet_ntop(substr($row['ban_ipn'],0,-1));

Apart from this modification I padded in this file too $ip a couple of times in occurences of INSERT and SELECT involving ban_ipn.

All seems to work better for me, now. I'm not getting all the emails anymore (some of which were identical)

#78: Re: Drastic CVS / Dragonfly 9.1.x changes Author: DJ Maze, Location: http://tinyurl.com/5z8dmv

Posted: Tue Jan 24, 2006 10:57 am
----
Alva stop flooding this topic with useless information.
The current CVS already pads the ip with a slash.

You will NOT recieve any support in here anyway especialy now that we know you're not using the latest cvs.

Topic is now locked to prevent more useless replies and because CVS has no support anyway.

-> Announcements

All times are GMT

Go to page Previous 1, 2, 3, 4, 5, 6 :| |: Page 6 of 6