Remove hard-coded limit of 50 results in search

Hello,

When searching for tickets belonging to a certain tag, the search results are limited to 50 results:

This makes it impossible to browse further tickets when there are more than 50 belonging to a certain tag. Is it possible to configure that limit or just remove it?

Thank you.

4 Likes

below you can add paging. but I think causes performance problems, that is why it was removed.

zammad/app/controllers/tickets_controller.rb

Thanks for your reply. I don’t think modifying the Ruby controller is much help, since it actually obeys to the params[:limit] it is sent, so there’s no trouble there. The actual caller of the endpoint (the snippet I posted in the first post) should offer a way to customise the limit instead.

Also I’ve looked at your snippet but that particular commit (cf16264) is 4 years old (13 Apr 2018). But the latest revision still honours params[:limit].

It seems the hardcoding was introduced in 2016: Improved shown search result of extended search. · zammad/zammad@05cf8c2 · GitHub, so maybe it is time we revisit a more flexible solution?

2 Likes

You can modify the values yourself. I have tested limit and paging. (not excluded that can cause problems.)

I find 50 sufficient, everything above goes to generate reports.

I’m aware I can modify the values myself, but I think it would be much better to offer a user preference, or remove that limit altogether; not all organisations have development resources to make such amends, let alone run the Zammad stack themselves, so this feels like a valid bug fix/feature req to ask into the mainstream branch.

If the system enforces 50 results despite the user never explicitly asking for that limit, then the system is not behaving as expected. If the interface is not warning the user that their search is limited to 50 results, then the interface is not behaving as expected, but in both scenarios the user is left confused as to why it should be this way, and why it cannot be changed.

I’m happy to provide the code to customise this limit, if the team is willing to agree and help me merge it in.

Thanks for your time.

4 Likes