Remove unused function get_leaders_from_usergroup

[Remington Steed]

The perl function get_leaders_from_usergroup was found to be buggy and unused anywhere in the codebase. Let's remove unused code, rather than worry about maintenance and upkeep. Here's the branch:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/remove_unused_usergroup_leaders_function

working/user/rsteed/remove_unused_usergroup_leaders_function

From the commit message:

Remove buggy, unused function get_leaders_from_usergroup

Recent editing and testing of get_leaders_from_usergroup() uncovered an
unrelated bug which renders the function useless (it returns all group
members, not just the leader). It wasn't previously caught because
Evergreen has no apparent references to this function. This commit
removes the unused function.

[Dan Wells (dbw2)]

Just to add to the conversation, commit 2ea32b916f suggests that this was perhaps committed accidentally some 8 years ago. As Remington says, it isn't used anywhere. I support yanking it out, but want to give others time to comment.

Also, looks like LP#1281750 needs to go in first for this branch (as well it should).

[Dan Wells (dbw2)]

Dead code should die. Thanks, Remington.