erocs/JukeAlert — pull request #75 by MCL0stman (2013-09-01 05:09:15+00:00)
Title: Issue fix for Bug #74 and performance improvement (#events)

Issue fix for Bug #74 and performance improvement (#events)

Fixed issue listed in Bug #74 where we introduced a dependency between juke alert and citadel. Fixed by passing the group names a user has access to into a procedure. No dynamic sql was used and group name injection was safe.

Performance improvement by removing the Event count from the jalist metadata returned. While originally included as a 'nice to have metadata addition', I cannot justify the performance penalty at this time until users specifically request this data. yagni.

Minor change to sql logic was made in the order by - jalist order by statement now has a final order by statement that will order by pk asc. This is just to get a stable sort in the off chance that two snitches were last-visited recently at the same rough time.