<div dir="ltr">Jeremy,<div><br></div><div>That’s correct except for one small detail: version_added and version_removed are two fields on a single model called RepositoryContent[0]. So it’s really two records total and not four.</div><div><br></div><div>It kind of brings up another point: when you remove a version where content was removed but then re-added later, you end up with two RepositoryContent records pointing to the same version. It’s an odd result but not only minor in terms of severity[1]. We should look at perhaps fixing it though if we adopt proposal #2.</div><div><br></div><div>[0] <a href="https://git.io/vbExy">https://git.io/vbExy</a> (note that they’re called vadded and vremoved in the PR)</div><div>[1] <a href="https://git.io/vbEpI">https://git.io/vbEpI</a></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Mon, Dec 11, 2017 at 5:18 PM, Jeremy Audet <span dir="ltr"><<a href="mailto:jaudet@redhat.com" target="_blank">jaudet@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Regarding the second option: What happens if I (as a user) add a content unit to a repo, later remove it, later add it again, and later remove it again? Would this result in two "version_added" and two "version_removed" records?<br></div>
</blockquote></div><br></div>