From e4022bf5bd9cec40bdeda47c81cda901b3506db2 Mon Sep 17 00:00:00 2001
From: febbweiss <fabrice.ecaille@gmail.com>
Date: Thu, 29 Oct 2015 13:02:18 +0000
Subject: [PATCH] Feature: use service to manage collection

---
 project-deployer.html |  4 +--
 project-deployer.js   | 65 ++++++++++++++++++++++++++++---------------
 2 files changed, 44 insertions(+), 25 deletions(-)

diff --git a/project-deployer.html b/project-deployer.html
index 7f9da54..06a24f2 100644
--- a/project-deployer.html
+++ b/project-deployer.html
@@ -68,10 +68,10 @@
     </div>
     <div class="form-group">
       <label for="git_url" class="col-sm-2 control-label">
-        <i class="fa fa-fw fa-github fa-2x" title="Git URL"></i>
+        <i class="fa fa-fw fa-github fa-2x" title="Git clone URL"></i>
       </label>
       <div class="col-sm-10">
-        <input type="url" class="form-control" name="git_url" placeholder="Git URL" value="{{project.git_url}}" required/>
+        <input type="url" class="form-control" name="git_url" placeholder="Git clone URL" value="{{project.git_url}}" required/>
       </div>
     </div>
     <div class="form-group">
diff --git a/project-deployer.js b/project-deployer.js
index 18a303b..a175367 100644
--- a/project-deployer.js
+++ b/project-deployer.js
@@ -1,11 +1,47 @@
 Projects = new Mongo.Collection('projects');
 
+ProjectService = {
+  insert: function(label, git_url, public_url, commands, callback) {
+    Projects.insert({
+      label: label,
+      git_url: git_url,
+      public_url: public_url,
+      commands: commands
+    }, callback);
+  },
+  
+  update: function(id, label, git_url, public_url ,commands) {
+    Projects.update(
+      id, 
+      { $set: { 
+        label: label,
+        git_url: git_url,
+        public_url: public_url,
+        commands: commands
+        } 
+      }
+    );
+  },
+  
+  delete: function(id) {
+    Projects.remove(id);
+  },
+  
+  get: function(id) {
+    return Projects.findOne({_id: id});
+  },
+  
+  list: function() {
+    return Projects.find({}, {sort: {label: 1}});
+  }
+};
+
 if (Meteor.isClient) {
   Meteor.subscribe('projects');
 
   Template.management.helpers({
     projects: function () {
-      return Projects.find({}, {sort: {label: 1}});
+      return ProjectService.list();
     }
   });
 
@@ -17,7 +53,6 @@ if (Meteor.isClient) {
     'submit .new-project': function (event) {
         event.preventDefault();
         var form = event.target;
-        console.log('handled', form.id.value);
         if( form.id.value ) {
           Meteor.call('editProject',form.id.value, form.label.value, form.git_url.value, form.public_url.value, form.commands.value);
           form.id.value = '';
@@ -79,38 +114,22 @@ if (Meteor.isServer) {
 
 Meteor.methods({
   listProjects: function() {
-    return Projects.find({}, {sort: {label: 1}});
+    return ProjectService.list();
   },
   
   getProject: function(id) {
-    return Projects.findOne({_id: id});
+    return ProjectService.get(id);
   },
   
   addProject: function(label, git_url, public_url ,commands) {
-    console.log('addProjects', arguments);
-    Projects.insert({
-      label: label,
-      git_url: git_url,
-      public_url: public_url,
-      commands: commands
-    });
+    ProjectService.insert(label, git_url, public_url ,commands);
   },
   
   editProject: function(id, label, git_url, public_url ,commands) {
-    console.log('editProjects', arguments);
-    Projects.update(
-      id, 
-      { $set: { 
-        label: label,
-        git_url: git_url,
-        public_url: public_url,
-        commands: commands
-        } 
-      }
-    );
+    ProjectService.update(id, label, git_url, public_url ,commands);
   },
   
   deleteProject: function(id) {
-    Projects.remove(id);
+    ProjectService.delete(id);
   }
 });