Page MenuHome

Remove "_base*" from context API
ClosedPublic

Authored by Dalai Felinto (dfelinto) on May 21 2019, 10:43 PM.

Details

Summary

We are not exposing RNA_ObjectBase in the 2.80 API.

Thus we can't have operators relying on it (e.g, CTX_data_visible_bases,
CTX_data_active_base, ...). Otherwise users won't be able to override
context for these operators.

This commit keep the CTX_data_.*bases() functions around so we don't
need to change the operators and potentially break things that late into
2.80. However as far as the Python scripters are concerned there is no
base to be overriden, ever.

That also simplify the guessing game addon developers have to play when
trying to override an operatori context. They still need to find whether an
operator requires editables, visibles, selected, ... objects. But at
least they don't need to find out whether the operators need base or
object.

Diff Detail

Repository
rB Blender
Branch
rip-base (branched from master)
Build Status
Buildable 3680
Build 3680: arc lint + arc unit

Event Timeline

Committed on rB0910932e71d actually. Too small of a change to hold for review

Don't forget to update the add-ons. Only 2-3 seem to be using it.

This revision is now accepted and ready to land.May 22 2019, 12:33 AM

There is actually no Addon directly affected by this commit. But there are a few references of "base" there in commented out or broken code. I plan to do a pass on them, sure thing.