69

I'm having some trouble binding a function defined in a controller with a callback function in a directive. My code looks like the following:

In my controller:

$scope.handleDrop = function ( elementId, file ) {
    console.log( 'handleDrop called' );
}

Then my directive:

.directive( 'myDirective', function () {
    return {
      scope: {
        onDrop: '&'
      },
      link: function(scope, elem, attrs) {
        var myFile, elemId = [...]

        scope.onDrop(elemId, myFile);
      }
    } );

And in my html page:

<my-directive on-drop="handleDrop"></my-directive>

Having no luck with the code above. From what I've read in various tutorials I understand I'm supposed to specify the arguments in the HTML page?

79

There is one small mistake in your code, please try the code below and it should work for you

<!doctype html>
<html ng-app="test">
  <head>
    <script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.0.8/angular.js"></script>

  </head>
 <body ng-controller="test" >    


<!-- tabs -->
<div my-directive on-drop="handleDrop(elementId,file)"></div>

 <script>
     var app = angular.module('test', []);

     app.directive('myDirective', function () {
         return {
             scope: {
                 onDrop: '&'
             },
             link: function (scope, elem, attrs) {
                 var elementId = 123;
                 var file = 124;
                 scope.onDrop({elementId:'123',file:'125'});

             }
         }
     });

     app.controller('test', function ($scope) {
         alert("inside test");
         $scope.handleDrop = function (elementId, file) {
             alert(file);
         }
     });

   </script>
</body>


</html>
|improve this answer|||||
  • 2
    Where is this behavior defined in the Angular documentation? – Nick Radford Feb 7 '14 at 21:55
  • 1
    There doesn't seem to be an actual topic in the docs but the topic is the linking function of directives, docs.angularjs.org/guide/directive – tommybananas Mar 6 '14 at 18:14
  • 3
    Apparently the parameter names have to match the markup exactly; I wonder if this would survive minification? – TrueWill Jul 2 '14 at 19:40
  • @TrueWill I think AngularJS struggles with traditional minification altogether, but it does have an ngmin module that does the same thing – Joseph Paterson Aug 8 '14 at 5:21
  • 3
    @JosephPaterson According to the web page ngmin is deprecated in favor of ng-annotate. Thanks! – TrueWill Aug 8 '14 at 16:11
119

Alternative method that will survive minification

leave your html as it was:

<my-directive on-drop="handleDrop"></my-directive>

change the call to:

scope.onDrop()('123','125')

notice the extra opening and closing parenthesis given to onDrop. This will instantiate the function instead of injecting the function's code.

Why is it better

  1. changing the parameters' names in the handleDrop() definition (or even adding some more, if you handle it correctly) will not make you change each of the directives injections in the html. much DRYer.

  2. as @TrueWill suggested, i'm almost sure the other solutions will not survive minification, while this way code stays with maximum flexibility and is name agnostic.

another personal reason is the object syntax, which makes me write much more code:

functionName({xName: x, yName: y})

(and adding the function signature in every directive call)

as oppose to

functionName()(x,y)

(zero maintenance to your html)

i found this great solution here.

hope that helps!

|improve this answer|||||
  • This worked for me as well, but I had an issue using object notation. Explicitly using classes in Typescript. this comment helped me fix it. – Justin Boyson Aug 4 '16 at 21:57
  • This works. I was getting "TypeError: Cannot use 'in' operator to search for x in y" by doing what the Joe was. – nilloc Jul 26 '18 at 16:53

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.