<p>I have a hazy memory that the code was originally written to make ways with large numbers of nodes invalid, but such ways exist in the history and <em>were</em> valid when they were created. So the code was changed to validate new versions of ways only with <code>preconditions_ok?</code>.</p>
<p>I think it would be better to do it with <code>valid?</code> on the model, as that seems more likely to give us universal coverage of all code paths. It's possible that, by now, no current version of any way would have an invalid number of nodes, so this could be on <code>Way</code> and left off <code>OldWay</code>?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/issues/1516#issuecomment-291852059">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLW5Fw7wQhzNPidhAHBJ08JlyJ8n2ks5rs47rgaJpZM4M0Fa-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLb5clI51NWNYRPzuwd30yAPF4uteks5rs47rgaJpZM4M0Fa-.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/issues/1516#issuecomment-291852059"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@zerebubuth in #1516: I have a hazy memory that the code was originally written to make ways with large numbers of nodes invalid, but such ways exist in the history and _were_ valid when they were created. So the code was changed to validate new versions of ways only with `preconditions_ok?`.\r\n\r\nI think it would be better to do it with `valid?` on the model, as that seems more likely to give us universal coverage of all code paths. It's possible that, by now, no current version of any way would have an invalid number of nodes, so this could be on `Way` and left off `OldWay`?"}],"action":{"name":"View Issue","url":"https://github.com/openstreetmap/openstreetmap-website/issues/1516#issuecomment-291852059"}}}</script>