search

Considerit / ConsiderIt

For deliberation and opinion visualization
GNU Affero General Public License v3.0
90 stars 14 forks source link

Update list.rb to check that customizations is non-null #188

Closed chadbrower0 closed 1 year ago

chadbrower0 commented 1 year ago

list_lists() sometimes accesses null subdomain.customizations

tkriplean commented 1 year ago

This should be unnecessary given https://github.com/Considerit/ConsiderIt/commit/098f999fd75bd969a5ed0ef17a53d67aaa3b2a68. Can you confirm?

chadbrower0 commented 1 year ago

I believe that's correct. I cannot verify until I go through setting up another dev instance, which may be some time. Additionally, I tend to practice "layers of defense" against crashing errors, which might be a reason to keep this check regardless.