Fix 309

[davidlatwe]

This PR will fix #309 , and UX bug related to #281 but on plugin section.

Before

• The instance Batman and David should not be toggled, but the restriction got bypassed via toggling the section.
• Plugin checkbox did not give immediate respond to section toggling.

After

• All fixed !

Implementation

1. Collect all optional items' state.
2. Override section state if all optional items are in the same state. This guaranteed the optional item be able to respond to every toggle.
3. Set check state.

[mottosso]

Nice work! Could you increment the version too? Then I'll make a new release for it.

[davidlatwe]

Done !

[BigRoy]

Good stuff again David! 👍

[tokejepsen]

Awesome work @davidlatwe

[davidlatwe]

Thanks :)
Merging this, and pinging @mottosso for release :P

[mottosso]

Yes, that is all fine! :)

One question though, that I noticed elsewhere as well, how come the instances are green, and yet checkable? :O I thought they were green only after having been processed, at which point they would not be checkable anymore?

[mottosso]

Oh and I remembered you are able to make releases too, you are most welcome to do that here as well!

[davidlatwe]

how come the instances are green, and yet checkable?

The reason it's green is that there's a plugin printing log message for every instances in collecting phase.

class ListHeros(pyblish.api.InstancePlugin):
    order = pyblish.api.CollectorOrder + 0.1

    def process(self, instance):
        self.log.info(instance)

So yeah, they did been processed :)

at which point they would not be checkable anymore?

I intentionally set data["optional"] = False to some instances at their creation, for the example ;D

Oh and I remembered you are able to make releases too

Haha, thanks :D

[BigRoy]

One question though, that I noticed elsewhere as well, how come the instances are green, and yet checkable? :O I thought they were green only after having been processed, at which point they would not be checkable anymore?

AND

The reason it's green is that there's a plugin printing log message for every instances in collecting phase.

Is this related to #283?