From cec21966579a09cab20876c4266ad6d227cf0233 Mon Sep 17 00:00:00 2001
From: Evan You <yyx990803@gmail.com>
Date: Sat, 16 Apr 2016 16:14:45 -0400
Subject: [PATCH] warn against duplicate keys

---
 benchmarks/reorder-list/index.html | 8 +++++---
 src/runtime/vdom/patch.js          | 8 +++++++-
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/benchmarks/reorder-list/index.html b/benchmarks/reorder-list/index.html
index 93fca382..2c0d269d 100644
--- a/benchmarks/reorder-list/index.html
+++ b/benchmarks/reorder-list/index.html
@@ -8,6 +8,7 @@
 
 <script type="text/x-template" id="t">
   <div>
+    <h1>{{ total }} Components</h1>
     <p>{{ action }} took {{time}}ms.</p>
     <button @click="shuffle">shuffle</button>
     <button @click="add">add</button>
@@ -34,13 +35,13 @@
   </tr>
 </script>
 
-<h1>1000 Components</h1>
 <div id="el">
 </div>
 
 <script>
+var total = 1000
 var items = []
-for (var i = 0; i < 1000; i++) {
+for (var i = 0; i < total; i++) {
   items.push({
     id: i,
     label: String(Math.random()).slice(0, 5)
@@ -52,6 +53,7 @@ var vm = new Vue({
   el: '#el',
   template: '#t',
   data: {
+    total: total,
     time: 0,
     action: 'Render',
     items: items,
@@ -63,7 +65,7 @@ var vm = new Vue({
     }),
     add: monitor('add', function () {
       this.items.push({
-        id: this.items.length,
+        id: total++,
         label: String(Math.random()).slice(0, 5)
       })
     }),
diff --git a/src/runtime/vdom/patch.js b/src/runtime/vdom/patch.js
index 51fa74d9..4b87290c 100644
--- a/src/runtime/vdom/patch.js
+++ b/src/runtime/vdom/patch.js
@@ -1,6 +1,6 @@
 import VNode from './vnode'
 import * as dom from './dom'
-import { isPrimitive } from '../util/index'
+import { isPrimitive, warn } from '../util/index'
 
 const emptyNode = VNode('', {}, [])
 const hooks = ['create', 'update', 'remove', 'destroy', 'pre', 'post']
@@ -178,6 +178,12 @@ export default function createPatchFunction (modules, api) {
           newStartVnode = newCh[++newStartIdx]
         } else {
           elmToMove = oldCh[idxInOld]
+          if (process.env.NODE_ENV !== 'production' && !elmToMove) {
+            warn(
+              'Duplicate track-by key: ' + idxInOld + '. ' +
+              'Make sure each v-for item has a unique track-by key.'
+            )
+          }
           patchVnode(elmToMove, newStartVnode, insertedVnodeQueue)
           oldCh[idxInOld] = undefined
           api.insertBefore(parentElm, getElm(elmToMove), getElm(oldStartVnode))