JS multiple expression in ng-click AngularJS

Question

Why doesn't this work?

<a ng-click="if(item.lp_iframe=='') ? href='{{item.lp_url}}' :
             playVideo(item.lp_iframe_id)" target="_blank">

I suspect I have to escape the ''??

what is the error statement that you getting in the console? — V31
– V31, Commented Feb 26, 2014 at 4:11

Tim Green · Accepted Answer · 2014-02-26 04:14:04Z

2

move this to a function in scope, like this

$scope.foo = function() ｛
  // ...
｝

ng-click="foo()"

According to the doc.

ng-click="expression"

Expression doesn't mean javascript, see http://docs.angularjs.org/guide/expression.

answered Feb 26, 2014 at 4:14

Tim Green

3,6492 gold badges25 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Tim Green Over a year ago

@user3346088 why playVideo will cause problem? call it like playVideo(...) or $scope.playVideo(...).

Ramesh Rajendran · Accepted Answer · 2014-02-26 07:34:10Z

0

Your problem is, your expression look like

<a>ng-click="href='url'"></a> // is the syntax is correct? Not

So try this

<a ng-click="if(item.lp_iframe=='') ? redirect(item.lp_url) :
             playVideo(item.lp_iframe_id)" target="_blank">

   $scope.redirect= function(url) ｛ 
      window.open(url)  
    ｝

or,

Try this

<a ng-click="clickevent()" target="_blank">

$scope.clickevent= function() ｛
  if(item.lp_iframe=='') 
       {   
        window.open(url)
       }
        playVideo(item.lp_iframe_id)
    ｝

edited Feb 26, 2014 at 7:34

answered Feb 26, 2014 at 5:15

Ramesh Rajendran

38.9k49 gold badges159 silver badges241 bronze badges

15 Comments

user3346088 Over a year ago

$location is not defined

Ramesh Rajendran Over a year ago

you need assign the $location in your controller

user3346088 Over a year ago

Object #<Object> has no method 'reload'

user3346088 Over a year ago

it make perfect sense but I gt an erorr, what does the reload do? and return true?

Ramesh Rajendran Over a year ago

Sorry change that to $route.reload();

|

Collectives™ on Stack Overflow

JS multiple expression in ng-click AngularJS

2 Answers 2

1 Comment

15 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

15 Comments

Your Answer

Sign up or log in

Post as a guest

Related