abrain/RIDE — comment on commit be3e4e9334a617151c3dd8bd424085f432ffe78c
File:  line 0
By mkorpela:

I would use the inplace list.sort -method:
Your now replacing a public field object with another -- thus someone could have a stale reference to the old version.

something like:

resources = chief.external_resources
...
chief._insert_into_suite_structure(resource)
...
resources is not the one to use

IMHO this is a bug waiting to happen