Bug 13204

Summary: Plugin housekeeping: Remove labs_theses.pl plugin
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: CatalogingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact:
Severity: trivial    
Priority: P5 - low CC: f.demians, jonathan.druart, m.de.rooy, paul.poulain, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 13204: Plugin housekeeping: Remove labs_theses.pl plugin
Bug 13204: Plugin housekeeping: Remove labs_theses.pl plugin
[PASSED QA] Bug 13204: Plugin housekeeping: Remove labs_theses.pl plugin

Description Marcel de Rooy 2014-11-05 13:40:39 UTC

    
Comment 1 Marcel de Rooy 2014-11-05 14:39:17 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2014-11-05 14:41:02 UTC
Instead of a signoff, a counterpatch is welcome too :)
Comment 3 Jonathan Druart 2014-11-27 11:05:11 UTC
Paul, this plugin (labs_theses.pl) comes from BibLibre.
It uses the (non-existent) LABTHE authorised values.
Any idea if it is still used somewhere?
Comment 4 Paul Poulain 2014-11-27 13:57:41 UTC
(In reply to Jonathan Druart from comment #3)
> Paul, this plugin (labs_theses.pl) comes from BibLibre.
> It uses the (non-existent) LABTHE authorised values.
> Any idea if it is still used somewhere?

Look at in2p3 staff interface (one of BibLibre customer), if it's not here, it must be frantiq (which is no more one of our customer, I think it's a Frederic Demian one, adding him)
Comment 5 Frédéric Demians 2014-12-01 08:14:33 UTC
I haven't any customer using this plugin, and I'm not aware of any library using it. For me, this patch could be signed-off without trouble. I can dot it if necessary.
Comment 6 Marcel de Rooy 2014-12-01 08:24:51 UTC
(In reply to Frédéric Demians from comment #5)
> I haven't any customer using this plugin, and I'm not aware of any library
> using it. For me, this patch could be signed-off without trouble. I can dot
> it if necessary.

If you can, thanks.
Comment 7 Frédéric Demians 2014-12-01 08:51:38 UTC Comment hidden (obsolete)
Comment 8 Jonathan Druart 2014-12-01 09:11:57 UTC
I have just contacted a library who use this plugin (or at least contain the LABTHE VA).
Comment 9 Marcel de Rooy 2014-12-01 09:29:37 UTC
(In reply to Jonathan Druart from comment #8)
> I have just contacted a library who use this plugin (or at least contain the
> LABTHE VA).
Could you please provide more details on how we can proceed now?
Comment 10 Jonathan Druart 2014-12-01 09:59:44 UTC
As soon as I get an answer from the customer I will let you know :)
Comment 11 Kyle M Hall (khall) 2014-12-05 15:34:41 UTC
Created attachment 34155 [details] [review]
[PASSED QA] Bug 13204: Plugin housekeeping: Remove labs_theses.pl plugin

Passing QA, pending response from Jonathan
Comment 12 Jonathan Druart 2014-12-05 15:43:05 UTC
Please don't push this for the moment.
Comment 13 Jonathan Druart 2014-12-08 14:23:23 UTC
The plugin did the same thing as when a field is linked to a list of AV.
It's useless and can be removed.
Switch back to Passed QA.
Comment 14 Marcel de Rooy 2014-12-08 14:32:33 UTC
(In reply to Jonathan Druart from comment #13)
> The plugin did the same thing as when a field is linked to a list of AV.
> It's useless and can be removed.
> Switch back to Passed QA.

Thanks for your attention.
Comment 15 Tomás Cohen Arazi (tcohen) 2015-01-15 00:37:42 UTC
Patch pushed to master.

Thanks Marcel!