<div dir="ltr"><div class="gmail_extra"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>On Mon, Sep 12, 2016 at 7:38 PM, David Cook <span dir="ltr"><<a href="mailto:dcook@prosentient.com.au" target="_blank">dcook@prosentient.com.au</a>></span> wrote:<br></div></div></div></div></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-AU"><div><p class="MsoNormal"><a name="m_4112599589801959118__MailEndCompose"><span style="font-size:11pt;font-family:calibri,sans-serif">I didn’t even know Koha::Exceptions was a thing! How does it work? Wasn’t Kyle going to work on some sort of developer handbook so there would be a central repository of guidelines/patterns?</span></a></p></div></div></blockquote><div><br></div><div>You can find it here: <a href="https://wiki.koha-community.org/wiki/Developers_Handbook">https://wiki.koha-community.org/wiki/Developers_Handbook</a></div><div><br></div><div>It doesn't have any documentation on using Exceptions, and that would definitely be a good addition. I feel like it's a bit "messy" as I started it by using the coding guidelines as a base, but it's not a bad start! If anyone wants to work on it, please do!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-AU"><div><p class="MsoNormal"><a name="m_4112599589801959118__MailEndCompose"><span style="font-size:11pt;font-family:calibri,sans-serif"> <u></u><u></u></span></a></p><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri,sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri,sans-serif">I read the Koha Coding Guidelines many moons ago, but it would be nice to know when there are updates. Since I’m not as active in the community these days, I don’t know if I have much of a say over things anymore, but it would be good to be informed of updates to the guidelines, so that a person can contribute more efficiently when one does contribute. <u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri,sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:calibri,sans-serif">TryCatch requires Moose while Try::Tiny is lightweight, but I guess we’re going down the Moose rabbit hole anyways these days? Doesn’t TryCatch just do try/catch/catch instead of if/elsif/elsif?</span></p></div></div></blockquote><div><br></div><div>I had thought TryCatch's use of Moose would be a deal-breaker ( at least until Koha is officially plack-only ), but after inspecting the code, I don't believe it actually uses Moose. It does use packages in the Moose namespace, but it doesn't appear to use Moose itself, though it's possible I didn't go far enough down the rabbit hole. At this point I think I'm in favor of TryCatch instead of Try::Tiny do to it's ability to catch multiple types of exceptions and it's avoidance of messing with error handlers.</div><div><br></div><div>Kyle</div></div></div></div>