From 63871baf6061668b162972193c55b5a8f7490797 Mon Sep 17 00:00:00 2001
From: elijah <elijah@riseup.net>
Date: Thu, 30 Apr 2015 00:32:33 -0700
Subject: added support for email notifications of ticket changes

---
 engines/support/app/models/ticket_comment.rb | 40 +++++++++-------------------
 1 file changed, 12 insertions(+), 28 deletions(-)

(limited to 'engines/support/app/models/ticket_comment.rb')

diff --git a/engines/support/app/models/ticket_comment.rb b/engines/support/app/models/ticket_comment.rb
index 2c5df41..1c04a1f 100644
--- a/engines/support/app/models/ticket_comment.rb
+++ b/engines/support/app/models/ticket_comment.rb
@@ -1,22 +1,17 @@
+#
+# elijah: the property 'posted_by' should have been a belongs_to() association
+#         or at least 'posted_by_id', but I am leaving it as is because I am
+#         not sure how to change it.
+#
 class TicketComment
   include CouchRest::Model::Embeddable
 
-  #belongs_to :ticket #is this best way to do it? will want to access all of a tickets comments, so maybe this isn't the way?
-  property :posted_by, String#, :protected => true #Integer#this should be current_user if that is set, meaning the user is logged in  #cannot have it be protected and set via comments_attributes=. also, if it is protected and we set in the tickets_controller, it gets unset. TODO---is this okay to have it not protected and manually check it? We do not users to be able to set this.
-  # if the current user is not set, then we could just say the comment comes from an 'unauthenticated user', which would be somebody with the secret URL
-  property :posted_at, Time#, :protected => true
-  #property :posted_verified, TrueClass, :protected => true #should be true if current_user is set when the comment is created
+  property :posted_by, String
+  property :posted_at, Time
   property :body, String
-  property :private, TrueClass # private comments are only viewable by admins  #this is checked when set, to make sure it was set by an admin
+  property :private, TrueClass
 
-  # ? timestamps!
   validates :body, :presence => true
-  #before_validation :set_time#, :set_posted_by
-
-  #design do
-  #  view :by_posted_at
-  #  view :by_body
-  #end
 
   # translations are in the same scope as those of a "proper" couchrest model
   def self.i18n_scope
@@ -28,21 +23,10 @@ class TicketComment
   end
 
   def posted_by_user
-    User.find(posted_by) if posted_by
-  end
-
-=begin
-  #TODO.
-  #this is resetting all comments associated with the ticket:
-  def set_time
-    self.posted_at = Time.now
-  end
-=end
-
-=begin
-  def set_posted_by
-    self.posted_by = User.current if User.current
+    if posted_by
+      @_posted_by_user ||= User.find(posted_by)
+    end
   end
-=end
+  alias user posted_by_user
 
 end
-- 
cgit v1.2.3