Witam

Podam przykład w odniesieniu do nodeadd.php, ale to samo występuje w nodeedit.php i pewnie w jeszcze kilku innych miejscach.

Wg mnie i mając na uwadze optymalizację zamieniłbym poniższe 2 linijki:

elseif($LMS->GetNodeIDByMAC($nodedata['mac']) && $LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) $error['mac'] = "Podany MAC jest już w bazie!";

na takie 5: elseif($LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) { if($LMS->GetNodeIDByMAC($nodedata['mac'])) $error['mac'] = "Podany MAC jest już w bazie!"; }

Dla ludzi, którzy mają 'allow_mac_sharing' włączone oznaczałoby to jedno zapytanie mniej.

Być może znalazłoby się więcej podobnych miejsc w kodzie.

-------------------- Łukasz Wojciechowski