Browse Source

fixes for expressjs fortune test

Patrick Falls 12 years ago
parent
commit
0eb89f784e
2 changed files with 9 additions and 5 deletions
  1. 1 1
      config/create.js
  2. 8 4
      express/app.js

+ 1 - 1
config/create.js

@@ -10,7 +10,7 @@ db.world.ensureIndex({id: 1})
 db.fortune.drop()
 
 db.fortune.save( {id: 1, message: 'fortune: No such file or directory'} );
-db.fortune.save( {id: 2, message: 'A computer scientist is someone who fixes things that aren''t broken.'} );
+db.fortune.save( {id: 2, message: "A computer scientist is someone who fixes things that aren't broken."} );
 db.fortune.save( {id: 3, message: 'After enough decimal places, nobody gives a damn.'} );
 db.fortune.save( {id: 4, message: 'A bad random number generator: 1, 1, 1, 1, 1, 4.33e+67, 1, 1, 1'} );
 db.fortune.save( {id: 5, message: 'A computer program does what you tell it to do, not what you want it to do.'} );

+ 8 - 4
express/app.js

@@ -22,6 +22,11 @@ var cluster = require('cluster')
     randomNumber: Sequelize.INTEGER
   }, {
     freezeTableName: true
+  })
+  , Fortune      = sequelize.define('Fortune', {
+    message: Sequelize.STRING
+  }, {
+    freezeTableName: true
   });
 
 var Schema = mongoose.Schema
@@ -111,10 +116,9 @@ if (cluster.isMaster) {
 
   app.get('/fortune', function(req, res) {
     Fortune.findAll().success(function (fortunes) {
-      var newFortune = new Fortune();
-      newFortune.message = "Additional fortune added at request time.";
-      fortune.push(newFortune);
-      fortune.sort(sortFortunes);
+      var newFortune = Fortune.build({message: "Additional fortune added at request time."});
+      fortunes.push(newFortune);
+      fortunes.sort(sortFortunes);
 
       res.render('fortune', {fortunes: fortunes});
     });