I want to popup an alert message from an angularJS function, but it's not working.

Script

app.controller("APIController", function ($scope, $http) {
$scope.saveSubs = function () {
    var sub = {
        UserName: $scope.username,
        Password: $scope.password
    };
    if ($scope.username === 'admin' && $scope.password === 'admin') {
        window.location.href = '/Home/HotelSearchRedirect';
    } else {
        $window.alert("User name or password wrong. please enter correct username or password.");
    }
};});

What is wrong with this code?

I want to popup an alert message from an angularJS function, but it's not working.

Script

app.controller("APIController", function ($scope, $http) {
$scope.saveSubs = function () {
    var sub = {
        UserName: $scope.username,
        Password: $scope.password
    };
    if ($scope.username === 'admin' && $scope.password === 'admin') {
        window.location.href = '/Home/HotelSearchRedirect';
    } else {
        $window.alert("User name or password wrong. please enter correct username or password.");
    }
};});

What is wrong with this code?

• javascript
• angularjs

• Use window instead of $window – user688 Commented Jun 2, 2017 at 5:58

4 Answers 4

$window needs to be injected.

app.controller("APIController", function ($scope, $http,$window) {
$scope.saveSubs = function () {
    var sub = {
        UserName: $scope.username,
        Password: $scope.password
    };
    if ($scope.username === 'admin' && $scope.password === 'admin') {
        window.location.href = '/Home/HotelSearchRedirect';
    } else {
        $window.alert("User name or password wrong. please enter correct username or password.");
    }
};});

Just replace this code.

Only alert("User name or password wrong. please enter correct username or password.") will be suffice.

There is no need of window or $window.

You should use window instead of $window,

  window.alert("User name or password wrong. please enter correct username or password.");

Inject $window.

app.controller("APIController", function ($window, $scope, $http) { 
   // Code
});