Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
erp5 erp5
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 136
    • Merge requests 136
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • erp5erp5
  • Merge requests
  • !1938

Merged
Created May 16, 2024 by Jérome Perrin@jeromeOwner

accounting: fix error in Journal report with lines with an acquired node

  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 2

If a transaction is selected but instead of having accounts as source/destination on lines it acquires the organisation set as source/destination on the transaction, this causes an AttributeError getGapList when trying to use this organisation as an account.

To fix this, adjust the getMovementHistoryList parameters to select only lines using accounts, by passing node_uid, which is O(n) on the number of accounts, but the number of accounts is not supposed to be too large and other reports also do this assumption.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: fix/journal-acquired-node
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7