the_merger 0.1.0 → 0.1.1

data/README.org

@@ -15,46 +15,48 @@ There is a demo app here: https://github.com/map7/mailout
 
 Add this line to your application's Gemfile:
 
-    gem 'the_merger'
+: gem 'the_merger'
 
 And then execute:
 
-    $ bundle
+: bundle
 
 Or install it yourself as:
 
-    $ gem install the_merger
+: gem install the_merger
 
 ** Configure
 
-When using TheMerger in a controller you have to add the following line to the top
+When using TheMerger in a controller you have to add the following line to the top. EG: the default would be the users controller as you are doing the mail merge there.
     
-    include TheMerger
+: include TheMerger
 
 Create a yaml file in your config directory called 'the_merger.yml' with something like this
 
-    ---
-    merge_model: "Client"
+: ---
+: merge_model: "Client"
 
 
 To use the javascript included in this gem put the following in your application.js
 
-    //= require the_merger/application
+: //= require the_merger/application
 
 
 Where you need to insert fields (I usually create a 'letters' table) put the following in the form, just above the text_area.
 
-    <%= field_selection %>
+: <%= field_selection %>
+
+Give the text_area you want to interact with the class 'mail_merge_body' so that the insert button knows where to insert the field selected.
 
 
 ** Usage
 
 Currently it's just one method and this is how you pass it information
 
-    TheMerger.mail_merge(
-		from: "from@example.com",
-		subject: "update_listing",
-		body: "Dear [firstname] [lastname], Please update your listing, from Mick")
+: TheMerger.mail_merge(
+:   from: "from@example.com",
+:   subject: "update_listing",
+:   body: "Dear [firstname] [lastname], Please update your listing, from Mick")
 
 
 ** Contributing
@@ -66,13 +68,14 @@ Currently it's just one method and this is how you pass it information
 5. Create new Pull Request
 
 ** Tasks
-*** DONE Get all users from the users table
-*** DONE Mail merge and output to the screen
-*** DONE Make the table a variable
-*** DONE Create a method to get all fields from the selected table.
-*** DONE Create a helper method to have a dropdown of fields
-*** DONE Turn this into an engine so that I can include asset pipeline javascript.
-*** DONE Get the Email all button to work
-
-*** TODO Write tests
-*** TODO Schedule repetitive mail outs
+- [X] Get all users from the users table
+- [X] Mail merge and output to the screen
+- [X] Make the table a variable
+- [X] Create a method to get all fields from the selected table.
+- [X] Create a helper method to have a dropdown of fields
+- [X] Turn this into an engine so that I can include asset pipeline javascript.
+- [X] Get the Email all button to work
+- [ ] Write tests
+- [ ] Schedule repetitive mail outs
+- [ ] Make compatible with tinymce
+

data/lib/the_merger/version.rb

@@ -1,3 +1,3 @@
 module TheMerger
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/lib/the_merger.rb

@@ -27,7 +27,7 @@ module TheMerger
   #
   def merge_fields(body,user)
     fields.each do |field|
-      body = body.gsub!("[#{field}]", user.send(field)) || body
+      body = body.gsub!("[#{field}]", user.send(field).to_s) || body
     end
     body
   end

data/spec/internal/db/schema.rb

@@ -10,6 +10,7 @@ ActiveRecord::Schema.define do
     t.string   "firstname"
     t.string   "lastname"
     t.string   "email"
+    t.integer  "age"
     t.datetime "created_at", :null => false
     t.datetime "updated_at", :null => false
   end

data/spec/lib/the_merger_spec.rb

@@ -2,13 +2,13 @@ require 'spec_helper'
 include TheMerger
 
 class User < ActiveRecord::Base
-  attr_accessible :email, :firstname, :lastname
+  attr_accessible :email, :firstname, :lastname, :age
 end
 
 describe TheMerger do
 
   before do
-    @user = User.create(firstname: "Michael", lastname: "Pope", email: "map7777@gmail.com")
+    @user = User.create(firstname: "Michael", lastname: "Pope", email: "map7777@gmail.com", age: 99)
   end
 
   describe "#merge_fields" do
@@ -18,6 +18,12 @@ describe TheMerger do
       end
     end
 
+    context "body includes a number" do
+      it "replaces [age] with 99" do
+        merge_fields("Are you over [age] years old?",@user).should eq("Are you over 99 years old?")
+      end
+    end
+    
     context "body is set to Dear [firstname] [lastname], [address]" do
       it "replaces fields which exist in the model" do
         merge_fields("Dear [firstname] [lastname], [address]",@user).should eq("Dear Michael Pope, [address]")
@@ -27,7 +33,7 @@ describe TheMerger do
 
   describe "#fields" do
     it "returns fields for the model excluding created_at, updated_at & id" do
-      fields.should eq(%w[firstname lastname email])
+      fields.should eq(%w[firstname lastname email age])
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: the_merger
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-02 00:00:00.000000000 Z
+date: 2013-07-04 00:00:00.000000000 Z
 dependencies: []
 description: Mail merge a table of fields into a standard letter
 email:
@@ -53,7 +53,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1235780733984697328
+      hash: -1055697086999734969
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -62,7 +62,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1235780733984697328
+      hash: -1055697086999734969
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23