jblowe / infrared

interim ucb blacklight customizations
Apache License 2.0
0 stars 1 forks source link

solr prefix keeps defaulting to `get` #2

Open mcampos-quinn opened 2 years ago

mcampos-quinn commented 2 years ago

In app/controllers/catalog_controller.rb line 58 you can specify the solr prefix: config.solr_path = 'select'. Uncommenting this allows a search to complete, but not showing individual documents which still defaults to get which returns a 404 from the public solr endpoint.

Looking at the BL7 gem lib/blacklight/configuration.rb line 54: property :document_solr_path, default: 'get' leads me to think that the default might be more efficient for single documents using get but this setting seems to gunk up our existing routes.

Not sure if this should be refactored? Or just patched to work for now? I also can't tell why the live QA version works as-is.

mcampos-quinn commented 2 years ago

Adding config.document_solr_path = 'select' to catalog_controller.rb fixes the above. We should probably investigate if this is "correct."