This is a change that needs careful review. It was causing a bug in the scopes, where it was loading each scope twice. This particular line of code has been there for 3 years, so it's clearly not the cause of any immediate crashes or problems. So I'm a little cautious to suddenly say that this is a bug.

However, it doesn't make much sense to me right now why loadConfiguration(0) should be called twice (once here and once below in UAVGadgetDecorator::loadConfiguration() ) so it does kind of seem unnecessary. It also doesn't make much sense to me why we should load only the first configuration if there are multiple configurations to load.