PrismPipeline / Prism

Prism Pipeline
https://prism-pipeline.com
GNU Lesser General Public License v3.0
335 stars 76 forks source link

Wrong shotName if shotName = None in editShot() #24

Closed EmberLightVFX closed 3 years ago

EmberLightVFX commented 3 years ago

This piece of code makes the shotName variable end up being "no sequence-" if the shotName variable was None. This doesn't make sense why it should return "no sequence-". Alsoit breaks the editShot_open function in the shotgun plugin as it expects to get None back if the shotName is None.

RichardFrangenberg commented 3 years ago

These lines are required to autofill the sequence name of the Create Shot dialog when opened while a sequence item was selected. This whole section might need a refactor in the future to make it clearer what's happening. You are right that the Shotgun editShot_open function was broken. I fixed this now for the next update, while preserving the sequence autofill.