static Method Use

[Eric Ponce]

Hi!

I have two models: Articles and Events

In the index function of Articles Controller, i've tried to do this:

geddy.model.Article.all({},{sort: {createdAt:"desc"}},function(err, articles) {

    self.respond({params: params, articles: articles, events: geddy.model.Event.getFutureEvents()});

});

and in te Event Model

Event.getFutureEvents = function () {

var today = new Date();

Event.all({when:{gt: today}},function(err,events){
            //"when" is the Event date
            return events;
        });

}

But in the controller I don't recibe anything.

If I put console.log(events) in the Event Model I can see all events.

Can you help me?

Thanks! :D

[Matthew Eernisse]

You can't just `return` from an asynchronous function. You'll have to
use a callback, like this:

Event.getFutureEvents = (callback) {

var today = new Date();
Event.all({when:{gt: today}},function(err,events){
//"when" is the Event date

if (err) { callback(err, null); }
else {
callback(null, events);
}
});
};

> --
> The official community discussion group.
> website: geddyjs.org, source: https://github.com/mde/geddy, group:
> https://groups.google.com/d/forum/geddyjs?hl=en
>
>

[Eric Ponce]

thanks Matthew but your code give me an error:

Event.getFutureEvents = (callback) { 

                                   ^

SyntaxError: Unexpected token {

:O i've tried to solve it but i can't...

[Matthew Eernisse]

Ugh, stupid. Should have done it in a real text-editor. Should be:

Event.getFutureEvents = function (callback) {

You'll need to pass it a callback function that does your `respond`.

[Eric Ponce]

OK! :D it's working. I recive the events array. But i can't use. I have this:

In Articles Controller:

this.index = function (req, resp, params) {

    var self = this;

    geddy.model.Event.getFutureEvents(function(nothing,events){
        console.log(events);

    });

    geddy.model.Article.all({},{sort: {createdAt:"desc"}},function(err, articles) {

        self.respond({params: params, articles: articles, events: events});

    });

};

Console log prints all events BUT, how can I transfer the "events" var to the respond?

thanks a lot!!!

[Techwraith]

You'll need to make sure that your variables are all in the scope that you need them to be in:

this.index= function(req, resp, params){

  var self = this;

  geddy.model.Event.getFutureEvents(function(err, events) {
    geddy.model.Article.all({}, {sort: {createdAt: 'desc'}}, function(err, articles) {

      self.respond({params: params, articles: articles, events: events});
    });

  });

}

[Eric Ponce]

Thanks! it is what I thinking,....Now its working on 100%

But, Imagine that I want to pass a lot of data. For example:

geddy.model.Event.getFutureAvents(function(err, a) {
  geddy.model.Some.getFutureBvents(function(err, b) {
    geddy.model.Thing.getFutureCvents(function(err, c) {
      geddy.model.Mouse.getFutureDvents(function(err, d) {
        geddy.model.Keyboard.getFutureEEvents(function(err, e) {
          geddy.model.Ipod.getFutureFvents(function(err, f) {

            geddy.model.Article.all({}, {sort: {createdAt: 'desc'}}, function(err, articles) {

              self.respond({params: params, articles: articles, events: events, a:a,b:b,c:c,d:d.....[etc]});
            });
          });
        });
      });
    });
  });
});

This is the best method to do this?

thanks :D

[Techwraith]

So this is where I would start using named functions instead of nested anonymous functions:

var self = this
  , opts = {};

opts.params = params;

var gotThings = function (err, things) {
  opts.things = things;
  this.respond(opts);
}

var gotSomes = function (err, somes) {
  opts.somes = somes;
  geddy.model.Thing.getFutureThings(gotThings);
}

var gotEvents = function (err, events) {
  opts.events = events
  geddy.model.Some.getFutureSomes(gotSomes);
}

Does that make sense?

[Eric Ponce]

Oh nice! Thanks for this example. It's perfect!! ;)