Allow customization of rendering version diff values
This commit is contained in:
parent
00615bea97
commit
949b9d64bf
|
@ -11,6 +11,10 @@
|
||||||
overflow: auto;
|
overflow: auto;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.versions-wrapper {
|
||||||
|
margin-left: 2rem;
|
||||||
|
}
|
||||||
|
|
||||||
</style>
|
</style>
|
||||||
</%def>
|
</%def>
|
||||||
|
|
||||||
|
@ -45,6 +49,7 @@
|
||||||
|
|
||||||
</div><!-- form-wrapper -->
|
</div><!-- form-wrapper -->
|
||||||
|
|
||||||
|
<div class="versions-wrapper">
|
||||||
% for version in versions:
|
% for version in versions:
|
||||||
|
|
||||||
<h2>${title_for_version(version)}</h2>
|
<h2>${title_for_version(version)}</h2>
|
||||||
|
@ -62,7 +67,7 @@
|
||||||
% for field in fields_for_version(version):
|
% for field in fields_for_version(version):
|
||||||
<tr>
|
<tr>
|
||||||
<td class="field">${field}</td>
|
<td class="field">${field}</td>
|
||||||
<td class="value old-value">${repr(getattr(version.previous, field))}</td>
|
<td class="value old-value">${render_old_value(version, field)}</td>
|
||||||
<td class="value new-value"> </td>
|
<td class="value new-value"> </td>
|
||||||
</tr>
|
</tr>
|
||||||
% endfor
|
% endfor
|
||||||
|
@ -81,8 +86,8 @@
|
||||||
% for field in fields_for_version(version):
|
% for field in fields_for_version(version):
|
||||||
<tr${' class="diff"' if getattr(version, field) != getattr(version.previous, field) else ''|n}>
|
<tr${' class="diff"' if getattr(version, field) != getattr(version.previous, field) else ''|n}>
|
||||||
<td class="field">${field}</td>
|
<td class="field">${field}</td>
|
||||||
<td class="value old-value">${repr(getattr(version.previous, field))}</td>
|
<td class="value old-value">${render_old_value(version, field)}</td>
|
||||||
<td class="value new-value">${repr(getattr(version, field))}</td>
|
<td class="value new-value">${render_new_value(version, field, 'dirty')}</td>
|
||||||
</tr>
|
</tr>
|
||||||
% endfor
|
% endfor
|
||||||
</tbody>
|
</tbody>
|
||||||
|
@ -101,7 +106,7 @@
|
||||||
<tr>
|
<tr>
|
||||||
<td class="field">${field}</td>
|
<td class="field">${field}</td>
|
||||||
<td class="value old-value"> </td>
|
<td class="value old-value"> </td>
|
||||||
<td class="value new-value">${repr(getattr(version, field))}</td>
|
<td class="value new-value">${render_new_value(version, field, 'new')}</td>
|
||||||
</tr>
|
</tr>
|
||||||
% endfor
|
% endfor
|
||||||
</tbody>
|
</tbody>
|
||||||
|
@ -109,6 +114,7 @@
|
||||||
% endif
|
% endif
|
||||||
|
|
||||||
% endfor
|
% endfor
|
||||||
|
</div>
|
||||||
</%def>
|
</%def>
|
||||||
|
|
||||||
|
|
||||||
|
|
|
@ -1169,6 +1169,8 @@ class MasterView(View):
|
||||||
'title_for_version': self.title_for_version,
|
'title_for_version': self.title_for_version,
|
||||||
'fields_for_version': self.fields_for_version,
|
'fields_for_version': self.fields_for_version,
|
||||||
'continuum': continuum,
|
'continuum': continuum,
|
||||||
|
'render_old_value': self.render_version_old_field_value,
|
||||||
|
'render_new_value': self.render_version_new_field_value,
|
||||||
})
|
})
|
||||||
|
|
||||||
def title_for_version(self, version):
|
def title_for_version(self, version):
|
||||||
|
@ -1198,6 +1200,12 @@ class MasterView(View):
|
||||||
versions.extend(query.all())
|
versions.extend(query.all())
|
||||||
return versions
|
return versions
|
||||||
|
|
||||||
|
def render_version_old_field_value(self, version, field):
|
||||||
|
return repr(getattr(version.previous, field))
|
||||||
|
|
||||||
|
def render_version_new_field_value(self, version, field, typ):
|
||||||
|
return repr(getattr(version, field))
|
||||||
|
|
||||||
def configure_common_form(self, form):
|
def configure_common_form(self, form):
|
||||||
"""
|
"""
|
||||||
Configure the form in whatever way is deemed "common" - i.e. where
|
Configure the form in whatever way is deemed "common" - i.e. where
|
||||||
|
|
Loading…
Reference in a new issue