<div dir="ltr">Hi, nice overview again, Jonathan!<br><br><div class="gmail_quote"><div dir="ltr">El mié., 14 sept. 2016 a las 6:40, Jonathan Druart (<<a href="mailto:jonathan.druart@bugs.koha-community.org">jonathan.druart@bugs.koha-community.org</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
= RESTful API =<br>
Last month I have asked some general questions about the RESTful API<br>
and the different patches waiting for QA/SO.<br>
I did not receive any replies from people directly involved in the<br>
RESTful API patches.<br>
I have put this part aside last month (and certainly for the next<br>
month as well).<br>
As Marcel said: "If the community decides to push it, the community<br>
will have to maintain it".<br>
Ok so let's define "the community" :)<br></blockquote><div><br></div><div>Last month I replied to this, and I now notice I answered partially. Here it goes:</div><div><br></div><div>I find no reason to think this 'RESTful API' code in a different way than the rest of the codebase. In fact, it is only a small piece of code that routes URLs into the current business classes. The routes can be wrongly defined (wording, convenience, etc), the data types can be wrongly specified, the required permissions to access those endpoints can be wrong. And that's all of it. No important business logic on this code.</div><div>Anyone should be able to fill bugs for it if problems are found. And I have no problem writing patches for that and I'm sure others will join.</div><div><br></div><div>The sample Jonathan gave on "What's on #3", about wrong function called for deleting patrons, it is just a piece of code that needs to be failed-qa, and that's it. It should probably be made dependent on the C4::Members rewrite into Koha::Patron(s) classes because it makes more sense. But that's it. I don't really understand what's the big deal about the REST api implementation. Pretty cool features could be written on top of it if we had more endpoints pushed, And there are several already in BZ, even with patches attached:</div><div><br></div><div><a href="https://bugs.koha-community.org/bugzilla3/buglist.cgi?quicksearch=REST%20api%20routes">https://bugs.koha-community.org/bugzilla3/buglist.cgi?quicksearch=REST%20api%20routes</a><br></div><div><br></div><div>Regards</div><div><br></div><div><br></div><div><br></div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr"><div style="color:rgb(117,117,117);font-family:'helvetica neue',helvetica,arial,sans-serif;font-size:12.8px">Tomás Cohen Arazi</div><div style="color:rgb(117,117,117);font-family:'helvetica neue',helvetica,arial,sans-serif;font-size:12.8px">Theke Solutions (<a href="http://theke.io/">https://theke.io</a>)<br>✆ +54 9351 3513384<br>GPG: B2F3C15F</div></div></div>