PCrudFilterPane can generate illegal queries

Bug #1218446 reported by Pasi Kallinen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.4
Fix Released
Undecided
Unassigned

Bug Description

When using some of the negating filters, such as "not like" or "not between", PCrudFilterPane can generate illegal queries with "undefined" values.

Fix in http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/paxed/pcrudfilterpane-fix

Revision history for this message
Pasi Kallinen (paxed) wrote :

Branch updated with another small fix, so the user-input text is remembered when changing operators.

Revision history for this message
Mike Rylander (mrylander) wrote :

Can the second fix get us into trouble when the value from the old widget doesn't make sense for the new operator?

Revision history for this message
Pasi Kallinen (paxed) wrote :

It checks the type of the input widget, and changing the field on which it operates clears the value. I haven't seen it break yet, but that proves nothing...

Dan Wells (dbw2)
tags: added: 2.5-beta-blocker
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Tested and works well. Pushed to master and rel_2_4. rel_2_3 has some differences preventing clean application of the patch, and those differences might even be such that the bug can't be reproduced in a 2.3.x release. Leaving rel_2_3 alone until/unless somebody reports a problem there.

Changed in evergreen:
assignee: nobody → Lebbeous Fogle-Weekley (lebbeous)
status: New → Fix Committed
Changed in evergreen:
assignee: Lebbeous Fogle-Weekley (lebbeous) → nobody
Dan Wells (dbw2)
Changed in evergreen:
status: Fix Committed → Fix Released
no longer affects: evergreen/2.3
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.