[Koha-bugs] [Bug 27680] API DataTables Wrapper fails for ordering on multi-data-field columns

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

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

--- Comment #13 from Martin Renvoize <martin.renvoize at ptfs-europe.com> ---
As the final tests expose.. this raises a question.

What form of 'multi-param' sending do we actually want to support?  Prior to my
patches we were "in theory" limiting to pipe delimited strings.. but the tests
weren't passing any pipe delimited parameters and were passing for 'traditional
multiple params' (i.e ?param1=+thingy&param2=+thangy).

I've added tests to cover the following.. but clearly many of them currently
fail.

1. Multi param traditional `?param=+this&param=+that` - Passing
2. Multi param PHP `?param[]=+this&param[]=+that` - Failing
3. CSV param `?param=+this,+that` - Failing

I like option 1, but feel we should fail in a more formal way for any/all other
options.. right now we just explode horribly.

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