<div dir="ltr">+1!<div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-18 10:57 GMT-03:00 Jonathan Druart <span dir="ltr"><<a href="mailto:jonathan.druart@bugs.koha-community.org" target="_blank">jonathan.druart@bugs.koha-community.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Bug 14544 is ready to test :)<br>
<br>
RIP C4::VirtualShelves*<br>
<br>
2015-07-16 18:51 GMT+01:00 Jonathan Druart<br>
<<a href="mailto:jonathan.druart@bugs.koha-community.org">jonathan.druart@bugs.koha-community.org</a>>:<br>
> Hello developers,<br>
><br>
> For much of the last 2 days, I have tried to get rid of<br>
> C4::VirtualShelves::Page, which is, in my opinion, the worst module of<br>
> Koha [1]<br>
><br>
> I am a bit frustrated because I will be afk for the next 2 weeks and I<br>
> have not finished what I intended to do. I am still far of removing<br>
> this module, but I prepared the ground:<br>
> I have added the package Koha::Virtualshelf, Koha::Virtualshelves,<br>
> Koha::Virtualshelfshare[s] and Koha::Virtualshelfcontent[s] to take<br>
> advantage of Dbix::Class.<br>
> I have also introduced a Koha::Exceptions module to throw exceptions<br>
> when needed.<br>
><br>
> Of course I have added tests to cover the changes.<br>
><br>
> I did not test widely the interface, so there are certainly some bugs.<br>
> But for what I tested, it works like as before.<br>
> It would be interesting to do some benchmarks also.<br>
><br>
> I really would like to get feedbacks on these patches. For those<br>
> interested in following the evolution of the code and want to share<br>
> their idea, have a look :)<br>
> Compare both code (before and after), you will understand why we<br>
> really need to move/rewrite our code :)<br>
><br>
> I won't be far from my mailbox, don't hesitate to ask [relevant]<br>
> questions and comment the patch set.<br>
><br>
> Be care, the last patch is a wip ("release early, release often"),<br>
> don't apply it if you want to test. You will certainly need to update<br>
> the Dbix::Class schema files.<br>
> The next steps are: Get rid of GetShelves, GetShelfContents and the<br>
> other subroutines of C4::Virtualshelves. Then introduce a normal<br>
> structure to the perl script (<a href="http://shelves.pl" rel="noreferrer" target="_blank">shelves.pl</a> and <a href="http://opac-shelves.pl" rel="noreferrer" target="_blank">opac-shelves.pl</a>). Finally<br>
> remove C4::VirtualShelves::Page.<br>
><br>
> Cheers,<br>
> Jonathan<br>
><br>
> [1] Actually it's not true, there are C4::Context, C4::Auth and<br>
> C4::Search before :)<br>
_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a><br>
</blockquote></div><br></div>