Differential D42 Diff 140 swh/web/ui/converters.py

Changeset View

Standalone View

swh/web/ui/converters.py

Show First 20 Lines • Show All 152 Lines • ▼ Show 20 Lines	Args:
- type: revision's type (git, tar or dsc at the moment)		- type: revision's type (git, tar or dsc at the moment)
- metadata: if the revision is synthetic, this can reference dynamic		- metadata: if the revision is synthetic, this can reference dynamic
properties.		properties.

Returns:		Returns:
Revision dictionary with the same keys as inputs, only:		Revision dictionary with the same keys as inputs, only:
- sha1s are in hexadecimal strings (id, directory)		- sha1s are in hexadecimal strings (id, directory)
- bytes are decoded in string (author_name, committer_name,		- bytes are decoded in string (author_name, committer_name,
author_email, committer_email, message)		author_email, committer_email)
- remaining keys are left as is		- remaining keys are left as is

"""		"""
return from_swh(revision,		revision = from_swh(revision,
hashess=set(['id', 'directory', 'parents', 'children']),		hashess=set(['id',
		'directory',
		'parents',
		'children']),
bytess=set(['name',		bytess=set(['name',
'fullname',		'fullname',
'email',		'email']),
'message']),
dates={'date', 'committer_date'})		dates={'date', 'committer_date'})

		if revision:
		if 'message' in revision:
		try:
		revision['message'] = revision['message'].decode('utf-8')
		except UnicodeDecodeError:
		revision['message_decoding_failed'] = True
		revision['message'] = None
		olasdUnsubmitted Done Inline Actions Please use `message_url` instead of `msg_url` for consistency. olasd: Please use `message_url` instead of `msg_url` for consistency.
		ardumontUnsubmitted Done Inline Actions The converter and the service layer modules are api/view agnostic. To be separated as the existing code is doing, the decode call should be done in the service layer. If it breaks, then the message is None. It should be the concern of the api layer to test for the message. If none exists, then the message_url is populated with the message url of the raw api. ardumont: The converter and the service layer modules are api/view agnostic. To be separated as the…

		return revision


def from_content(content):		def from_content(content):
"""Convert swh content to serializable content dictionary.		"""Convert swh content to serializable content dictionary.

"""		"""
return from_swh(content,		return from_swh(content,
hashess={'sha1', 'sha1_git', 'sha256'},		hashess={'sha1', 'sha1_git', 'sha256'},
bytess={},		bytess={},
Show All 28 Lines