>From the code perspective it look fine (nice recursion), but there are redundant queries to database which are increase if there are more widgets on the page.
-- You received this bug notification because you are a member of Aiki Framework Developers, which is subscribed to aikiframework. https://bugs.launchpad.net/bugs/1006828 Title: Code that get widgets need to be refactored Status in Aiki Framework: New Bug description: Code that retrieve widgets is recursive and return widgets for every candidate widgets ($aiki->widgets->get_candidate_widgets) which is stupid becasue right now all widgets need to have the same url (in display_urls field) so they can be retrieve in one query (like in old aiki). The reason becasue this may be a problem is that on OCAL there is over 400 queries and there 100 queries to aiki_widgets table which can slow down the site, those queries are not expensive but if you're sum them up you can get few seconds delay. I optimize this code a bit (every query was executed twice one to get the id and one to get the row from aiki_widgets table) To manage notifications about this bug go to: https://bugs.launchpad.net/aikiframework/+bug/1006828/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~aikiframework-devel Post to : aikiframework-devel@lists.launchpad.net Unsubscribe : https://launchpad.net/~aikiframework-devel More help : https://help.launchpad.net/ListHelp
