Reviews backend, saving, etc.

Author: System Administrator

app/controllers/drafts_controller.rb

@@ -34,18 +34,32 @@ def save
 
   def review
     @drft = Draft.find_by_public_url(params[:id])
-    raw = HTMLEntities.new.decode(@drft.content)
-    preproc = StanfordParser::DocumentPreprocessor.new
-    sentences = preproc.getSentencesFromString(raw)
-    sentences.each_with_index do |sent, i|
-      chars = ["?", ",", ".", ":", ";", "...", "''", "'"]
-      sent = sentences[i] = sent.join(" ")
-      sent = sentences[i] = sent.gsub(/(``[^'']+$)/) {|s| "#{$1}\""}
-      sent = sentences[i] = sent.gsub("`` ", ' "').gsub("--", "—")
-      chars.each {|chr| sent = sentences[i] = sent.gsub(" " + chr, chr)}
-      sentences[i] = '<a class="sentence" href="#" id="sent-' + i.to_s + '">' + sent.strip + '</span>'
+    if session[:review_id].nil?
+      reviewified = @drft.reviewify
+      rev = Review.new(:draft_id => @drft.id, :n_sentences => reviewified[:n_sentences])
+      rev.save
+      session[:review_id] = rev.id
+      @content = reviewified[:content]
+      @n_zeros = reviewified[:n_sentences]
+    else
+      @content = HTMLEntities.new.decode(Review.find(session[:review_id]).content)
+      @n_zeros = Review.find(session[:review_id]).n_sentences
+      @general_comments = Review.find(session[:review_id]).general_comments
+      if @content.nil?
+        reviewified = @drft.reviewify
+        @content = reviewified[:content]
+        @n_zeros = reviewified[:n_sentences]
+      end
     end
-    @content = sentences.join(" ")
-    @n_zeros = '0' * sentences.length
+  end
+  
+  def save_review
+    rev = Review.find(session[:review_id])
+    rev.content = HTMLEntities.new.encode(params[:content])
+    rev.signature = params[:signature] unless params[:signature].empty?
+    rev.general_comments = params[:general_comments] unless params[:general_comments].empty?
+    rev.updated_at = Time.new
+    rev.save
+    render :json => Time.new
   end
 end

app/models/draft.rb

@@ -1,2 +1,29 @@
 class Draft < ActiveRecord::Base
+  has_many :reviews
+  
+  def reviewify
+    raw = HTMLEntities.new.decode(self.content)
+    preproc = StanfordParser::DocumentPreprocessor.new
+    sentences = preproc.getSentencesFromString(raw)
+    sentences.each_with_index do |sent, i|
+      strt, brk = false, false
+      chars = ["?", ",", ".", ":", ";", "...", "''", "'"]
+      sent = sentences[i] = sent.join(" ")
+      sent = sentences[i] = sent.gsub(/(``[^'']+$)/) {|s| "#{$1}\""}
+      sent = sentences[i] = sent.gsub("`` ", ' "').gsub("--", "&mdash;")
+      chars.each {|chr| sent = sentences[i] = sent.gsub(" " + chr, chr)}
+      if sent.index(/<\/p>\s+<p>/)
+        sent = sentences[i] = sent.gsub(/<\/p>\s+<p>/) {|s| ""}
+        brk = true
+      end
+      if sent.index("<p>")
+        sent = sentences[i] = sent.gsub("<p>", "")
+        strt = true
+      end
+      sentences[i] = '<a class="sentence" href="#" id="sent-' + i.to_s + '">' + sent.strip + '</a>'
+      if brk then sentences[i] = sentences[i].insert(0, "</p><p>") end
+      if strt then sentences[i] = sentences[i].insert(0, "<p>") end
+    end
+    return {:content => sentences.join(" "), :n_sentences => "0" * sentences.length}
+  end
 end

app/models/review.rb

@@ -1,2 +1,3 @@
 class Review < ActiveRecord::Base
+  belongs_to :draft
 end

app/views/drafts/review.html.erb

@@ -11,6 +11,7 @@
 	border-top: 1px solid #777;
 	border-bottom: 1px solid #777;
 	background-color: white;
+	min-height: 400px;
 	float: left;
 }
 li {
@@ -66,17 +67,35 @@ a.sentence.down {
 .note {
 	padding: 10px;
 	font-family: Courier;
-	
+	height: 90px;
 }
 </style>
 
 <script type="text/javascript">
 var times = '<%= @n_zeros %>'.split('');
+var t = null;
+
+var save_review = function(continuous) {
+	var content = $("#draft_content").html();
+	$.post("/drafts/save_review", {authenticity_token: AUTH_TOKEN, content: content, signature: $("#signature").attr("value"), general_comments: $("#general_comments").val()},
+		function(ret) {
+			var ret = JSON.parse(ret);
+			var now = new Date();
+			document.title = "(saved at " + now.format("h:MM:sstt") + " on " + now.format("dddd dS, yyyy") + ")";
+		}
+	)
+	if (continuous) {
+		setTimeout("save_review(true)", 1000 * 30) // Save every thirty seconds.
+	} else {
+		return false;
+	}
+};
 
 var hide_notes = function(affect) {
 	$(".note:visible[affect=" + affect + "]").each(function(e) {
 		if ($(this).val() == "") {
 			$(this).hide();
+			$("#sent-" + $(this).attr("id").split("-")[1]).css("text-decoration", "none");
 		};
 	});
 };
@@ -85,8 +104,16 @@ $(document).ready(function() {
 	$(".note").live("keydown", function(e) {
 		if (e.keyCode == 13) {
 			$(this).hide();
+			if ($(this).val() != "") {
+				$("#sent-" + $(this).attr("id").split("-")[1]).css("text-decoration", "underline");
+			} else {
+				$("#sent-" + $(this).attr("id").split("-")[1]).css("text-decoration", "none");
+			}
 		};
 	});
+	
+	setTimeout("save_review(true)", 3000);
+
 	var pop_note = function(id, action, ev) {
 		var $note = $("#note-" + id);
 		if ($note.length == 0) { 
@@ -110,13 +137,17 @@ $(document).ready(function() {
 		} else {
 			$note.attr("affect", "neutral");
 			$note.hide();
+			if ($note.val() == "") {
+				$("#sent-" + $note.attr("id").split("-")[1]).css("text-decoration", "none");
+			};
 		};
 		$note.focus();
-		setTimeout("hide_notes('" + $note.attr("affect") + "')", 2000);
+		t = setTimeout("hide_notes('" + $note.attr("affect") + "')", 2000);
 		return false;
 	}
 
 	$(".sentence").click(function(ev) {
+		clearTimeout(t);
 		$(".note").each(function(e) {
 			$(this).hide();
 		});
@@ -147,12 +178,22 @@ $(document).ready(function() {
 		<li>Click a sentence <strong>once</strong> if you like it and <strong>twice</strong> if you don't.</li>
 		<li>A window for <strong>quick notes</strong> will pop up in either case. <strong>Notes disappear</strong> if you don't type anything.</li>
 		<li>If you do leave comments, <strong>hit enter</strong> to close the note window. <strong>Everything will be saved automatically</strong>.</li>
+	</ol>
 </div>
 <div id="draft">
 	<h1 style="font-family: Arial; font-size: 22px; margin-top: 20px;"><%= @drft.title %></h1>
-	<%= @content %>
+	<div id="draft_content">
+	<% if @content.nil? %>
+		Whoops, it looks like something's gone horribly wrong and we've lost your work.
+	<% else %>
+		<%= @content %>
+	<% end %>
+	</div>
 </div>
 </a>
+<script language="javascript" src="/javascripts/simpleswap.js"></script>
 <div class="instructions">
-	<h3 style="margin: 2px 0px 5px 0px;">All done? Sign your name: <input type="text" style="font-family: Verdana; width: 180px; height: 25px; vertical-align: middle; padding-top: 2px; font-size: 13px; text-align: center; font-weight: bold; color: #555"/> and <a href="#">click here to send your feedback.</a></h3>
-</div>
+	<h3 style="margin: 2px 0px 5px 0px;">General Comments:</h3><textarea id="general_comments" style="width: 100%; height: 90px; padding: 10px; font-family: Courier"><%= @general_comments %></textarea>
+	<hr>
+	<div style="float: left; font-size: 12px; padding-top: 5px;"><h3 style="margin: 2px 0px 5px 0px;">All done? Sign your name: <input type="text" style="font-family: Verdana; width: 180px; height: 25px; vertical-align: middle; padding-top: 2px; font-size: 13px; text-align: center; font-weight: bold; color: #555" id="signature" value=""/> and</h3></div> <div style="float: right"><a href="#" id="submit-review"><img src="/images/button.png" oversrc="/images/button-hover.png"/></a></div>
+</div>

db/development.sqlite3

@@ -0,0 +1,15 @@
+class ReviewsBackend < ActiveRecord::Migration
+  def self.up
+    add_column :reviews, :draft_id, :integer
+    add_column :reviews, :content, :text
+    add_column :reviews, :signature, :string
+    add_column :drafts, :email, :string
+  end
+
+  def self.down
+    remove_column :reviews, :draft_id
+    remove_column :reviews, :content
+    remove_column :reviews, :signature
+    remove_column :drafts, :email
+  end
+end

db/migrate/20091027054208_reviews_backend.rb

@@ -0,0 +1,9 @@
+class AddNSentences < ActiveRecord::Migration
+  def self.up
+    add_column :reviews, :n_sentences, :string
+  end
+
+  def self.down
+    remove_column :reviews, :n_sentences
+  end
+end

db/migrate/20091027064137_add_n_sentences.rb

@@ -0,0 +1,9 @@
+class AddGeneralComments < ActiveRecord::Migration
+  def self.up
+    add_column :reviews, :general_comments, :text
+  end
+
+  def self.down
+    remove_column :reviews, :general_comments
+  end
+end

db/migrate/20091027064650_add_general_comments.rb

@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20091025225533) do
+ActiveRecord::Schema.define(:version => 20091027064650) do
 
   create_table "drafts", :force => true do |t|
     t.datetime "created_at"
@@ -18,11 +18,17 @@
     t.text     "title",      :default => ""
     t.text     "content",    :default => ""
     t.string   "public_url"
+    t.string   "email"
   end
 
   create_table "reviews", :force => true do |t|
     t.datetime "created_at"
     t.datetime "updated_at"
+    t.integer  "draft_id"
+    t.text     "content"
+    t.string   "signature"
+    t.string   "n_sentences"
+    t.text     "general_comments"
   end
 
 end

db/schema.rb

@@ -0,0 +1,38 @@
+// This is the implementation of SimpleSwap
+// by Jehiah Czebotar
+// Version 1.1 - June 10, 2005
+// Distributed under Creative Commons
+//
+// Include this script on your page
+// then make image rollovers simple like:
+// <img src="/images/ss_img.gif" oversrc="/images/ss_img_over.gif">
+//
+// http://jehiah.com/archive/simple-swap
+// 
+
+
+function SimpleSwap(el,which){
+  el.src=el.getAttribute(which || "origsrc");
+}
+
+function SimpleSwapSetup(){
+  var x = document.getElementsByTagName("img");
+  for (var i=0;i<x.length;i++){
+    var oversrc = x[i].getAttribute("oversrc");
+    if (!oversrc) continue;
+      
+    // preload image
+    // comment the next two lines to disable image pre-loading
+    x[i].oversrc_img = new Image();
+    x[i].oversrc_img.src=oversrc;
+    // set event handlers
+    x[i].onmouseover = new Function("SimpleSwap(this,'oversrc');");
+    x[i].onmouseout = new Function("SimpleSwap(this);");
+    // save original src
+    x[i].setAttribute("origsrc",x[i].src);
+  }
+}
+
+var PreSimpleSwapOnload =(window.onload)? window.onload : function(){};
+window.onload = function(){PreSimpleSwapOnload(); SimpleSwapSetup();}
+