Comment 3 for bug 1296937

Following up discussion today in IRC regarding the non-typical addition of the $ids_only parameter to ->charged_items() to the middle rather than the end of the parameter list, the patch at the tip of the user/gmcharlt/lp1296937_split_public_and_private_interfaces branch in the working/Evergreen repository splits the public and implementation interfaces for the charged items lookup. As the commit message indicates, this is meant to be tested with a reversion of SIPServer commit c97d64412.

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

Consider this a pullrequest for this follow-up.