Open obydog002 opened 5 months ago
Any ideas where to look so I can have a shot at fixing this for myself?
question is what do expect to be fix. in the end it is the limit of api framework translation of request to sql. so from client point of view, you can either prevent such a combination of filtering/sort or handle error in more meaningful way.
On Mon, 15 Jan 2024, 05:49 obydog002, @.***> wrote:
Any ideas where to look so I can have a shot at fixing this for myself?
— Reply to this email directly, view it on GitHub https://github.com/FAForever/downlords-faf-client/issues/3115#issuecomment-1891298008, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBYJJPX4TOAV4EGGWFM77TYOSYOLAVCNFSM6AAAAABB2VS6LWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJRGI4TQMBQHA . You are receiving this because you are subscribed to this thread.Message ID: @.***>
The fix is to remove the illegal sorting option that doesn't work with filtering.
Ahh yes, the sort by option. I just didn't know how to use the filter panel We can close this then
Well there still shouldn't be a combo that results in an invalid request. I had thought I got all of them before
Sort by: Average Reviews score
Describe the bug Trying to search in the Player Name field on replays gives the exception. Client 2024.1.1 but I've seen this bug at least since the 2023.12.1 client.
To Reproduce Write something and search
Log or error message
Expected behavior Filtered replays to appear
Screenshots
OS Linux