https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19234

Martin Renvoize <martin.renvo...@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.renvoize@ptfs-europe
                   |                            |.com

--- Comment #8 from Martin Renvoize <martin.renvo...@ptfs-europe.com> ---
This somehow feels like a step too far in trying to generalise a mapping
between api query parameters and database fields.. perhaps I'm wrong though and
we're aiming to try to keep that link strongly bound.. perhaps some examples of
how you're expecting to use this outside of just the pagination merging.

On the pagination side.. I think I'd probably add that slice of code to
::Plugin::Pagination (to keep it with the pagination stuff) and name it
something along the lines of 'dbic_merge_pagination_attributes'?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to