Question

62

AngularJS: How to pass values from Controller to Service Method?

rated 0 times [ 66] [ 4] / answers: 1 / hits: 31667 / 11 Years ago, wed, june 5, 2013, 12:00:00

I have a controller which is dependent on a TransactionService. One of the method is

$scope.thisMonthTransactions = function () {

    $scope.resetTransactions();

    var today = new Date();

    $scope.month = (today.getMonth() + 1).toString();

    $scope.year = today.getFullYear().toString();

    $scope.transactions = Transaction.getForMonthAndYear();

};

The TransactionService looks like

angular.module('transactionServices', ['ngResource']).factory('Transaction', function ($resource, $rootScope) {

    return $resource('/users/:userId/transactions/:transactionId',

        // todo: default user for now, change it

        {userId: 'bd675d42-aa9b-11e2-9d27-b88d1205c810', transactionId: '@uuid'},

        {

            getRecent: {method: 'GET', params: {recent: true}, isArray: true},

            getForMonthAndYear: {method: 'GET', params: {month: 5, year: 2013}, isArray: true}

        });

});

As you can see the method getForMonthAndYear depends on two parameters month and year, which are hardcoded right now as params: {month: 5, year: 2013}. How can I pass this data from my controller?

I tried injecting rootScope in TransactionService, but that did not help(meaning I do not know how to use it probably).

Also Angular ngResource documentation doesn't recommend any way to perform this.

Can someone please guide here?

UPDATE

My Controller looks like

function TransactionsManagerController($scope, Transaction) {



    $scope.thisMonthTransactions = function () {

        $scope.resetTransactions();

        var today = new Date();

        $scope.month = (today.getMonth() + 1).toString();

        $scope.year = today.getFullYear().toString();



        var t = new Transaction();

        $scope.transactions = t.getForMonthAndYear({month: $scope.month});

    };

}

and I change service method to

getForMonthAndYear: {method: 'GET', params: {month: @month, year: 2013}, isArray: true}

I look at the console.log and it says

Uncaught SyntaxError: Unexpected token ILLEGAL transaction.js:11

Uncaught Error: No module: transactionServices

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

quentinaveryb

Add To Favorites

Follow

Total Points: 102

Total Questions: 100

Total Answers: 93

Location: Colombia

Member since Mon, May 2, 2022

2 Years ago

quentinaveryb questions

1 How to use annotation layer in PDF.js?

Thu, Aug 6, 20, 00:00, 4 Years ago

1 Cannot assign to read only property of #< Object> using React with Recoil

Sat, Jul 18, 20, 00:00, 4 Years ago

1 Strapi JWT token lifetime?

Fri, Jul 17, 20, 00:00, 4 Years ago

1 What is the difference between async await and a regular fetch?

Tue, Feb 4, 20, 00:00, 4 Years ago

1 How do I include a JavaScript variable in an HTML header?

Mon, Aug 12, 19, 00:00, 5 Years ago

View All

answered 11 Years ago reedmustafam · Accepted Answer

Defining params in the resource constructor are only necessary when the call needs to have a default when none is supplied. Any param passed in to the method is appended as a query param, regardless of whether it has a default value defined. The '@' means that the params value is replaced by what gets returned in the JSON response, so your @uuid makes sense, but not your @month.

Personally, I would just create the resource like so:

$resource('/users/:userId/transactions/:transactionId',

    // todo: default user for now, change it

    {userId: 'bd675d42-aa9b-11e2-9d27-b88d1205c810', transactionId: '@uuid'},

    {

        getRecent: {method: 'GET', params: {recent: true}, isArray: true},

        getForMonthAndYear: {method: 'GET', isArray: true}

    });

Then add the query variables as needed by passing them in. (Creating special method names is fine but not necessary so showing both ways below.)

var t = new Transaction();

$scope.recentTransactions = t.$get({recent:true}) //results in /users/bd675d42-aa9b-11e2-9d27-b88d1205c810/transactions/?recent=true

$scope.recentTransactions = t.$getRecent(); //same thing as above

$scope.transactions = t.$get({month: $scope.month, year: $scope.year}); //results in /users/bd675d42-aa9b-11e2-9d27-b88d1205c810/transactions/?month=5&year=2013

$scope.transactions = t.$getForMonthAndYear({month: $scope.month, year: $scope.year}); //same as above... since no defaults in constructor, always pass in the params needed