[Bf-committers] Failing addons/module load tests

Sergey Sharybin sergey.vfx at gmail.com
Mon Aug 28 10:00:41 CEST 2017


Hi,

Here is situation: in current master script_load_addons
and script_load_modules tests are failing. Report says it's due to attempt
to register class twice.

For example, in archimesh, ObjectProperties is registered from door,
venetian and window maker. This seems to be simple to solve: just
rename ObjectProperties to {Door, Venetian, Window}ObjectProperties. Is it
really a correct way of solving the issue?

But then, some addons are registering operators with quite generic names,
for example Splint in mesh_extra tools. Shall we just rename the class?

Other thing whicvh is failing is io_coat3d, but that one i didn't figure
out yet why exactly, code seems fine to me. Need someone's else eyes here.

So., questions goes are:
- Are my findings are correct about root cause of the failures?
- Why this didn't fail before? Was it just a dangerous code which was
allowing to override registered classes?
- Shall we just rename some classes for now, so there are no conflicts?

But bigger question is: how to avoid naming conflicts ins the future?
Especially for addons which are not distributed with Blender?

-- 
With best regards, Sergey Sharybin


More information about the Bf-committers mailing list