codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-24 08:09:13 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Ember wants a ListTopController, not a TopController

Browse source
This commit is contained in:
Régis Hanol 2014-01-07 01:36:30 +01:00
parent af3d262bf3
commit 53006d7778
2 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/assets/javascripts/discourse/controllers/top_controller.js → app/assets/javascripts/discourse/controllers/list_top_controller.js
View file

@ -1,12 +1,12 @@
/**
Controller of the top page
@class TopController
@class ListTopController
@extends Discourse.ObjectController
@namespace Discourse
@module Discourse
**/
Discourse.TopController = Discourse.ObjectController.extend({
Discourse.ListTopController = Discourse.ObjectController.extend({
redirectedToTopPageReason: function() {
// no need for a reason if the default homepage is "top"

11
app/assets/javascripts/discourse/routes/list_top_route.js
View file

@ -1,5 +1,9 @@
Discourse.ListTopRoute = Discourse.Route.extend({
model: function() {
return Discourse.TopList.find();
},
activate: function() {
this._super();
// will mark the "top" navigation item as selected
@ -9,13 +13,6 @@ Discourse.ListTopRoute = Discourse.Route.extend({
});
},
setupController: function() {
var topController = this.controllerFor("top");
Discourse.TopList.find().then(function (result) {
topController.set("model", result);
});
},
renderTemplate: function() {
this.render('top', { into: 'list', outlet: 'listView' });
},