Bug 12845

Summary: C4::Output::FormatData is unused
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: m.de.rooy, nick, tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 12845: C4::Output::FormatData can be removed
Bug 12845: C4::Output::FormatData can be removed
[SIGNED OFF] Bug 12845: C4::Output::FormatData can be removed
[SIGNED OFF] Bug 12845: C4::Output::FormatData can be removed
Bug 12845: C4::Output::FormatData can be removed
Bug 12845: C4::Output::FormatData can be removed

Description Jonathan Druart 2014-08-28 15:03:23 UTC
... and can be removed.
Comment 1 Jonathan Druart 2014-08-28 15:04:28 UTC Comment hidden (obsolete)
Comment 2 Nick Clemens 2014-09-03 03:29:42 UTC
I am still getting one result:

nick@debian:~/koha$ git grep FormatData
C4/Output.pm:        &output_html_with_http_headers &output_ajax_with_http_headers &output_with_http_headers FormatData FormatNumber
Comment 3 Jonathan Druart 2014-09-03 07:23:55 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-09-03 07:24:23 UTC
(In reply to Nick Clemens from comment #2)
> I am still getting one result:

Oops :) Fixed in the last patch.
Comment 5 Nick Clemens 2014-09-03 10:33:33 UTC Comment hidden (obsolete)
Comment 6 Nick Clemens 2014-09-03 10:37:04 UTC
Sorry, looked at wrong bug for signoff, will retest and sign off asap
Comment 7 Nick Clemens 2014-09-03 10:55:31 UTC Comment hidden (obsolete)
Comment 8 Marcel de Rooy 2014-09-03 14:09:57 UTC Comment hidden (obsolete)
Comment 9 Jonathan Druart 2015-01-15 08:56:30 UTC
Created attachment 35289 [details] [review]
Bug 12845: C4::Output::FormatData can be removed

Test plan:
  git grep FormatData
should not return any result in pl and pm files.

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Tomás Cohen Arazi 2015-01-17 13:44:12 UTC
Patch pushed to master.

Thanks Jonathan!