Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • O OS Core Common
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 22
    • Issues 22
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe SoftwareOpen Subsurface Data Universe Software
  • Platform
  • System
  • Lib
  • core
  • OS Core Common
  • Merge requests
  • !52

Fix multi record info

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Matt Wise requested to merge fix-multi-record-info into master Feb 02, 2021
  • Overview 7
  • Commits 4
  • Pipelines 9
  • Changes 4

No existing core service uses MultiRecordInfo that I have seen. Therefore, should be a non-breaking fix

The IStorageService implementation in core was returning Strings rather than Records for the query records route. Other Record query interfaces use the Record object.

The implementation was also throwing two different Exceptions that are resolved by this MR.

It appears the function was broken during the change to use the new BodyMapper, but went unnoticed until now.

Edited Feb 02, 2021 by Matt Wise
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix-multi-record-info