Skip to content
Snippets Groups Projects
Commit b6e26a12 authored by André Anjos's avatar André Anjos :speech_balloon:
Browse files

Merge branch 'quickfix_plotter_params' into 'master'

Don't assume plotter params exist

There seems to be an edge case when running the dev setup locally where there aren't any default plotter parameters available for the plotter. This crashes the search functionality. To reproduce:
- Go to an experiment
- Click on the blue magnifying glass to search for relevant experiments
- Django should crash in `search_tags.py` giving `'NoneType' object has no attribute 'fullname'`

See merge request !215
parents 3773a447 ddbf09bc
No related branches found
No related tags found
1 merge request!215Don't assume plotter params exist
Pipeline #
...@@ -210,7 +210,7 @@ def plot_details_for_result(r): ...@@ -210,7 +210,7 @@ def plot_details_for_result(r):
return dict( return dict(
plotter=plotting['default'][0].fullname(), plotter=plotting['default'][0].fullname(),
parameter=plotting['default'][1].fullname(), parameter=None if plotting['default'][1] == None else plotting['default'][1].fullname(),
parameters=json.dumps([k.fullname() for k in plotting['options'][plotting['default'][0]]]), parameters=json.dumps([k.fullname() for k in plotting['options'][plotting['default'][0]]]),
) )
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment