Comment 1 for bug 1745231

Revision history for this message
Anastasia (anastasia-macmood) wrote :

@Roger Peppe,

Thank you for investigating this failure and such a detailed analysis in this report \o/

I do completely agree with your reasoning and believe that more comprehensive fix would be to filter out dead models from the initial query. This will prevent State.Get trying to populate model details and will fix other places that potentially cannot handle dead models. The way we handle dead models, if at all, should be an exceptional, case-by-case handling: we should never get dead models in that list.

I'll propose against 2.3 branch first and forward port to develop (heading into 2.4) once the patch lands.