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
  • !188

Merged
Created Oct 28, 2016 by Georgios Dagkakis@georgios.dagkakisDeveloper

erp5_trade, OrderModule_activateGetOrderStatList: make at_date inclusive

  • Overview 5
  • Commits 3
  • Changes 2

The way it was in reports involving OrderModule_activateGetOrderStatList, at_date was not included. There even was a mis-match of assigning:

to_date = request.get('at_date', None)

One commit to fix this including also some formatting fixups.

Second commit adding a test-case in testTradeReports, where a report is run having at_date as the start_date of a Sale Order. Previous code would fail in this, since the Sale Order was not considered in the report.

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