Skip to content

Setting conversion: Fix UnboundLocalError on extract_review_intermediates variable - #355

Merged
iLLiCiTiT merged 1 commit into
developfrom
bugfix/fix_convert_extract_intermediate_files_unboundlocalerror
Aug 19, 2026
Merged

Setting conversion: Fix UnboundLocalError on extract_review_intermediates variable#355
iLLiCiTiT merged 1 commit into
developfrom
bugfix/fix_convert_extract_intermediate_files_unboundlocalerror

Conversation

@BigRoy

@BigRoy BigRoy commented Aug 19, 2026

Copy link
Copy Markdown
Member

Changelog Description

Setting conversion: Fix #354 UnboundLocalError on extract_review_intermediates variable

Additional review information

Settings conversion should work.

Testing notes:

  1. Copy settings from an older Nuke addon version that does not have overrides to ExtractReviewIntermediates but DOES have overrides to other plug-ins under publish section.
  2. Settings copy should work.

@BigRoy
BigRoy requested review from moonyuet and a lite review from Copilot August 19, 2026 14:02
@BigRoy BigRoy self-assigned this Aug 19, 2026
@BigRoy BigRoy added the type: bug Something isn't working label Aug 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes settings-conversion failure when copying older Nuke addon settings that don’t include publish.ExtractReviewIntermediates, preventing an UnboundLocalError during conversion and allowing the rest of the publish overrides to be copied successfully.

Changes:

  • Add an early return in _convert_extract_intermediate_files_0_2_3 when ExtractReviewIntermediates is not present.
  • Move extract_review_intermediates assignment outside the previous conditional so it’s always defined when used.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@iLLiCiTiT
iLLiCiTiT merged commit a8c0541 into develop Aug 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trouble copying production settings from 0.4.11 to 0.4.12

3 participants