Adds a sub to allow getting parameters that aren't required, but may be sent
Created attachment 14651 [details] [review]
bug 9412 Add optional_params sub to C4::Service
Similar to require_params, but just returns undef for any param that
wasn't sent by request
Signed-off-by: Hugh Davenport <firstname.lastname@example.org>
I think this needs a test plan.
Failed QA for the following reasons:
1) No test plan
2) Adds a subroutine that is not used
Can you please justify the addition of this new subroutine when it isn't used by any new code?
Sorry, when I git bz'd the patch up here, it didn't take into account dependencies which I guess I kinda assumed it would. This bug is actually building blocks for bug 4439 which then use this method. I've updated this bug to show this dependency.