just a note for oxpus

Snippets, Add-Ons und Fixes zum Calendar lite/pro Keine Anfragen
Gesperrt
blackolrac
User
User
Beiträge: 2
Registriert: Sa 05.Mär, 2005 13:13

just a note for oxpus

Beitrag von blackolrac » Di 08.Mär, 2005 02:54

its not a i know better thing just to help you, i was implenting some mods off yours

they are a great add to my calendar pro, however i had some second thoughts

what you do is a complete query so i discussed it with a friend off my

he tolds me also that the querie could be lower

so what you use as code as example

Code: Alles auswählen

$sql = "SELECT user_birthday FROM " . USERS_TABLE . " LIMIT 0,1";
if($result = @$db->sql_query($sql))
can also be made simple and better for load

Code: Alles auswählen

if ( isset($userdata['user_birthday']) )
special thx to mosyuis who pointed me that out, i hope you also can use it

grtz Carlo

Benutzeravatar
oxpus
Administrator
Administrator
Beiträge: 29106
Registriert: Mo 27.Jan, 2003 22:13
Wohnort: Bad Wildungen
Kontaktdaten:

Beitrag von oxpus » Di 08.Mär, 2005 08:51

Thanks for that hint, BUT:
What's about the date 01.01.1970? This will be the timestamp = 0 so this check may malfunction.
My check here will proof if the data field on the table really exists to avoid this possible error.

And:
This check will be done once on opening the calendar settings in the ACP. Not really the huge load at this point ;)
Karsten Ude
-={ Das Mädchen für alles }=-
Kein Support per ICQ, Email oder PN! Unaufgeforderte Nachrichten werden ignoriert!
No support per ICQ, Email or PM. Each unasked message will be ignored!

blackolrac
User
User
Beiträge: 2
Registriert: Sa 05.Mär, 2005 13:13

Beitrag von blackolrac » Di 08.Mär, 2005 09:34

thanx for the 1970 answer, never though off that

further, lol trying to save each kb bandwidth i can

Benutzeravatar
oxpus
Administrator
Administrator
Beiträge: 29106
Registriert: Mo 27.Jan, 2003 22:13
Wohnort: Bad Wildungen
Kontaktdaten:

Beitrag von oxpus » Di 08.Mär, 2005 18:48

Query the database will not produce bandwidth, just stressing the server a little bit ;)
Karsten Ude
-={ Das Mädchen für alles }=-
Kein Support per ICQ, Email oder PN! Unaufgeforderte Nachrichten werden ignoriert!
No support per ICQ, Email or PM. Each unasked message will be ignored!

Gesperrt