AngularJS syntax error on expression when using add/subtract operators

Question

I have an element as below :

<a class="previous i-previous  fa fa-caret-left" href=""
    ng-click="setPage(currentPage-1)"
    ng-hide="currentPage == 0">
</a>

I had this code working but now I have this exception :

Error: [$parse:syntax] http://errors.angularjs.org/1.2.26/$parse/syntax?p0=undefined&p1=is%20unexpected%2C%20expecting%20%5B)%5D&p2=null&p3=setPage(currentPage&p4=setPage(currentPage

The code that causes the exception is :

ng-click="setPage(currentPage-1)"

When i removed -1 it works so it seems the problem is about that part.

What might be the cause of this exception?

EDIT :

I have this code in my controller :

$scope.currentPage = 0;
$scope.setPage = function (page) { 
        $scope.currentPage = page;
    };

try setPage(--currentPage)

Vineet
– Vineet

2015-06-25 09:34:59 +00:00
Commented Jun 25, 2015 at 9:34 — Vineet
– Vineet, Commented Jun 25, 2015 at 9:34
It's running correctly. jsfiddle.net/U3pVM/16666

Sunil Madan
– Sunil Madan

2015-06-25 09:44:28 +00:00
Commented Jun 25, 2015 at 9:44 — Sunil Madan
– Sunil Madan, Commented Jun 25, 2015 at 9:44

Community · Accepted Answer · 2017-05-23 10:29:18Z

2

It is caused by batarang in Chrome, disable it or use other browser to test, or use incognito mode.

https://stackoverflow.com/a/31006278/5039495

edited May 23, 2017 at 10:29

CommunityBot

11 silver badge

answered Jun 25, 2015 at 9:59

Icycool

7,1991 gold badge27 silver badges33 bronze badges

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

4 Comments

Vineet Over a year ago

What do you want to say ?

cuneyttyler Over a year ago

Wow, it caused me for 2 hours. Thanks.

Icycool Over a year ago

@Vineet setPage(currentPage-1) is valid angular syntax, but it get parsed incorrectly on chrome with batarang. You can call it a false positive.

Icycool Over a year ago

@cuneytyvz Yea I spent few hours on it as well, hope they will fix it soon

Vineet · Accepted Answer · 2015-06-25 10:02:05Z

1

Just try

ng-click="currentPage = currentPage - 1; setPage(currentPage)"

edited Jun 25, 2015 at 10:02

answered Jun 25, 2015 at 9:47

Vineet

4,7153 gold badges25 silver badges42 bronze badges

4 Comments

cuneyttyler Over a year ago

I have this method in other parts of my code and use it for pagination view like "currentpage + 1" also. I'm going to try this but i think "currentPage-1" is also needs to be working.

cuneyttyler Over a year ago

great thanks, it worked. But i still didn't get why the other one does not.

Vineet Over a year ago

Simply, your ng-click function sends the value first to controller and then decreasing it by 1.

cuneyttyler Over a year ago

i mean i didn't get why the version in the question does not work. And also currentPage-- also didn't work but 'ng-click="currentPage = currentPage - 1; setPage(currentPage)"' did work.

Sunil Madan · Accepted Answer · 2015-06-25 10:13:32Z

0

It's running correctly. check fiddle

<div ng-app>

  <div ng-controller="TodoCtrl">
    <a  href=""
    ng-click="setPage(currentPage-1)"
    ng-hide="currentPage == 0">test
</a>
  </div>
</div>



function TodoCtrl($scope) {
    $scope.currentPage = 5;
$scope.setPage = function (page) { 
        $scope.currentPage = page;
    };
}

http://jsfiddle.net/U3pVM/16666/

answered Jun 25, 2015 at 10:13

Sunil Madan

4571 gold badge5 silver badges19 bronze badges

Collectives™ on Stack Overflow

AngularJS syntax error on expression when using add/subtract operators

3 Answers 3

4 Comments

4 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related