Bug 13871: [TITLE_AMENDED] Additional changes
commit42e731857d53134f91ecf0099ae6b7c13393166c
authorMason James <mtj@kohaaloha.com>
Mon, 23 Mar 2015 01:30:26 +0000 (23 14:30 +1300)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Mon, 7 Mar 2016 17:22:19 +0000 (7 17:22 +0000)
treecb96722aa3ce9fc8a18de7ae0e093d90b35745c3
parenta5babb0effb760aa1d0d9de83ce8bb690099678c
 Bug 13871: [TITLE_AMENDED] Additional changes

The original perltidy patch from Mason has been amended.

The perltidy itself has been moved to a separate patch with the current
perltidyrc applied.

As noted on Bugzilla, the original perltidy patch included some extra
changes:
[1] You prefix timestamp with Sip
    This is not actually needed (it is imported), but if we should prefix it,
    we should prefix now with C4::SIP::Sip. But you only changed two
    occurrences (out of 26). So I remove these two changes.
[2] You remove the $server parameter from two calls of maybe_add:
    A closer look at the remaining code tells me that $server is always
    passed to maybe_add for FID_SCREEN_MSG. So this only left me the
    current whitespace change.
But at least we documented what we did or did not, and why..

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
C4/SIP/Sip/MsgType.pm