[Koha-bugs] [Bug 8202] Add authorities to export tool

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8202

--- Comment #7 from Jared Camins-Esakov <jcamins at cpbibliography.com> ---
(In reply to comment #5)
> QA comments:
> 
> Hi Jared,
> 
> Your modifications seem good and they are close to mine (for export.pl).
> About your patch, I think you can repropose it modifying the indentation of
> export.tt (ex. 
> +        <ol><li>
> 
> +        </li></ol>
> 
> +[% ELSE %]
> 
> +<div id="exporttype" class="toptabs">
> +<ul>
> )
> 
> I agree with you, I quibble...

I would be happy to submit a follow-up fixing indentation. I was very tempted
to reindent export.tt, actually, but section 1.3 in the coding guidelines says
don't refactor needlessly, and don't spend time changing someone else's style
to yours. :)

-- 
You are receiving this mail because:
You are watching all bug changes.