collectiveaccess / providence

Cataloguing and data/media management application
GNU General Public License v3.0
299 stars 167 forks source link

search_builder still showing in menu find for some occurrences when using ca_occurrences_breakout_find_by_type_in_submenu #1618

Open thCAgh opened 1 month ago

thCAgh commented 1 month ago

Whats the issue: Even disabling search builder for occurrences in app.conf, the search builder option is showing and when clicking on it, uncaught fatal error occurs. Is particularly showing the search builder option for occurrence_types without third "generation" in hierarchy (aka those occurrence_types without childs).

Version: Providence dev/php8

How to reproduce:

parent 1st hierarchy 2nd hierarchy
occurrence_types exhibitions
occurrence_types procedures Proc 1
occurrence_types procedures Proc 2
occurrence_types procedures Proc 3
occurrence_types procedures Proc 4
occurrence_types publications
Whats showing in the menu: menu submenu sub-submenu
Find exhibitions Basic search
Find exhibitions advanced search
Find exhibitions search builder
Find exhibitions explore
Find procedures Proc 1
Find procedures Proc 2
Find procedures Proc 3
Find procedures Proc 4
Find publications Basic search
Find publications advanced search
Find publications search builder
Find publications explore

Feedback in glitter I spoke with Monica and she suggested to fill an issue since it looks in her workds:

it's something to do with the dynamic menu creation for find when ca__breakout_by_typeinmenu is set to true. So most likely an bug in /app/lib/BaseSearchController.php which generates this.

collectiveaccess commented 2 weeks ago

Should be fixed with fa4c02a730e68c974b9878c6180dccb07254e15c