Currently, a database query is run for each attendee in _minutes_translate_uids_to_display(). This patch makes this a single query, simplifies the function, and removes the recursive call.

CommentFileSizeAuthor
minutes_translate_uids.patch1.78 KBDavid Lesieur

Comments

pukku’s picture

pukku commented
Assigned: Unassigned » pukku

Hi! I'm about to go on a two-week vacation, and I'm wicked busy with getting stuff ready both at work and at home, so I don't have the time to review this right now. Once I get back (beginning of December), I will definitely see about applying this.

Thanks for the improvements!

Ricky

pukku’s picture

pukku commented
Status: Needs review » Fixed

Hi! I had some spare time that suddenly showed up, so I was able to look this over. I committed the patch, although I changed from using strtr() to t() just in case someone wants some kind of internationalization stuff to happen with it.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)