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 140
    • Merge requests 140
  • 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
  • !928

Merged
Created Sep 06, 2019 by Jérome Perrin@jeromeOwner

hal_json_style: Tolerate documents with non-utf8 properties

  • Overview 4
  • Commits 1
  • Changes 2

HAL JSON uses JSON, which uses UTF-8, but properties of ERP5 documents does not enforce any specific encoding, so if we happen to have documents with string properties that are not UTF-8 compatible, they could not be queried with HAL JSON (json.dumps was raising UnicodeError)

This pre-process field default properties and document title to make sure they can be serialized as json. This assumes that configuration (form definition, actions, ...) will be valid UTF-8, but just does extra effort to make sure "broken" data can be displayed without error.

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