Optimization of lookup_gmt function

[IgorG]

Hello,

While profiling vicidial on huge list loading I have found, that function lookup_gmt, that called for every lead, take much time for execution. Main reason - it includes every time when called dbconnect.php, and cause reading astguiclient.conf reading.

I have improved this place by changing include(dbconnect.php) to global $link, as soon as $link to database is the only data needed from dbconnect.php. Does this change proper and can it be committed to feature version of vicidial?

[mflorell]

Please submit your patch and documentation to the issue tracker and post a link on this thread.

[IgorG]

Submited: http://www.vicidial.org/VICIDIALmantis/view.php?id=319

[mflorell]

Fixed in SVN trunk and 2.2.0

This gave an estimated 40% reduction in time to load leads in my tests.

[williamconley]

whoa. way to go IgorG.