Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1704

Fix workflow bug "Local variable 'value' referenced before assignment"

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Eteri requested to merge Sokhoyan/erp5_fork:master into master Dec 06, 2022
  • Overview 6
  • Commits 2
  • Pipelines 0
  • Changes 2

In lines 862-892 the local variable "value" is getting assigned in the "if..else.." statement. But there is a path that leaves the "value" unassigned, hence raising UnboundLocalError error. I think we either should define a default value for the "value" variable, or make sure that there is no path that could leave the "value" unassigned. I'm not familiar with the code, so my suggestion might be wrong. @klaus , @jerome can you please check?

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: master
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7