Merge pull request #2366 from rhyzx/fix-terminal-directives-with-global-mixin

Fix custom terminal directive with global mixin

Author: yyx990803

src/compiler/compile.js

@@ -627,7 +627,8 @@ function makeTerminalNodeLinkFn (el, dirName, value, options, def) {
     filters: parsed.filters,
     raw: value,
     // either an element directive, or if/for
-    def: def || publicDirectives[dirName]
+    // #2366 or custom terminal directive
+    def: def || resolveAsset(options, 'directives', dirName)
   }
   // check ref for v-for and router-view
   if (dirName === 'for' || dirName === 'router-view') {

test/unit/specs/compiler/compile_spec.js

@@ -634,4 +634,17 @@ describe('Compile', function () {
     expect(el.textContent).toBe('worked!')
     expect(getWarnCount()).toBe(0)
   })
+
+  it('allow custom terminal directive', function () {
+    Vue.mixin({}) // #2366 conflict with custom terminal directive
+    Vue.compiler.terminalDirectives.push('foo')
+    Vue.directive('foo', {})
+
+    new Vue({
+      el: el,
+      template: '<div v-foo></div>'
+    })
+
+    expect(getWarnCount()).toBe(0)
+  })
 })