summaryrefslogtreecommitdiff
path: root/help/app/models/ticket.rb
blob: fed2b8bf3f31953c0e02486ffe66411953061554 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
class Ticket < CouchRest::Model::Base
  #include ActiveModel::Validations

  use_database "tickets"
  require 'securerandom'
=begin
    title
    created_at
    updated_at
    email_address
    user
    user_verified?
    admins (list of admins who have commented on the ticket)
    code (secret url)
=end

  #belongs_to :user #from leap_web_users. doesn't necessarily belong to a user though
  property :created_by, String, :protected => true #Integer #nil unless user was authenticated for ticket creation, #THIS should not be changed after being set
  property :regarding_user, String#Integer # form cannot be submitted if they type in a username w/out corresponding ID. this field can be nil. for authenticated ticket creation by non-admins, should this just automatically be set to be same as created_by?  or maybe we don't use this field unless created_by is nil?
  #also, both created_by and regarding_user could be nil---say user forgets username, or has general question
  property :title, String
  property :email, String #verify

  #property :user_verified, TrueClass, :default => false #will be true exactly when user is set
  #admins
  #property :code, String, :protected => true # only should be set if created_by is nil #instead we will just use couchdb ID
  property :is_open, TrueClass, :default => true
  property :comments, [TicketComment]

  timestamps!

  #before_validation :set_created_by, :set_code, :set_email, :on => :create
  before_validation :set_email, :on => :create


  #named_scope :open, :conditions => {:is_open => true} #??

  design do
    #TODO--clean this all up
    #view :by_is_open
    #view :by_created_by

    view :by_updated_at
    view :by_created_at

    #view :by_is_open_and_created_by
    view :by_is_open_and_created_at
    view :by_is_open_and_updated_at


    #TODO: This view is only used in tests--should we keep it?
    view :by_includes_post_by,
      :map =>
      "function(doc) {
        var arr = {}
        if (doc['type'] == 'Ticket' && doc.comments) {
          doc.comments.forEach(function(comment){
          if (comment.posted_by && !arr[comment.posted_by]) {
             //don't add duplicates
             arr[comment.posted_by] = true;
             emit(comment.posted_by, 1);
          }
          });
        }
      }", :reduce => "function(k,v,r) { return sum(v); }"

    view :by_includes_post_by_and_is_open_and_updated_at,
      :map =>
      "function(doc) {
        var arr = {}
        if (doc['type'] == 'Ticket' && doc.comments) {
          doc.comments.forEach(function(comment){
          if (comment.posted_by && !arr[comment.posted_by]) {
            //don't add duplicates
            arr[comment.posted_by] = true;
            emit([comment.posted_by, doc.is_open, doc.updated_at], 1);
          }
          });
        }
      }", :reduce => "function(k,v,r) { return sum(v); }"

    view :by_includes_post_by_and_is_open_and_created_at,
      :map =>
      "function(doc) {
        var arr = {}
        if (doc['type'] == 'Ticket' && doc.comments) {
          doc.comments.forEach(function(comment){
          if (comment.posted_by && !arr[comment.posted_by]) {
            //don't add duplicates
            arr[comment.posted_by] = true;
            emit([comment.posted_by, doc.is_open, doc.created_at], 1);
          }
          });
        }
      }", :reduce => "function(k,v,r) { return sum(v); }"

    view :by_includes_post_by_and_updated_at,
      :map =>
      "function(doc) {
        var arr = {}
        if (doc['type'] == 'Ticket' && doc.comments) {
          doc.comments.forEach(function(comment){
          if (comment.posted_by && !arr[comment.posted_by]) {
            //don't add duplicates
            arr[comment.posted_by] = true;
            emit([comment.posted_by, doc.updated_at], 1);
          }
          });
        }
      }", :reduce => "function(k,v,r) { return sum(v); }"


    view :by_includes_post_by_and_created_at,
      :map =>
      "function(doc) {
        var arr = {}
        if (doc['type'] == 'Ticket' && doc.comments) {
          doc.comments.forEach(function(comment){
          if (comment.posted_by && !arr[comment.posted_by]) {
            //don't add duplicates
            arr[comment.posted_by] = true;
            emit([comment.posted_by, doc.created_at], 1);
          }
          });
        }
      }", :reduce => "function(k,v,r) { return sum(v); }"

  end

  validates :title, :presence => true
  #validates :comments, :presence => true #do we want it like this?


  # html5 has built-in validation which isn't ideal, as it says 'please enter an email address' for invalid email addresses, which implies an email address is required, and it is not.
  validates :email, :format => /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/, :if => :email #email address is optional

  #TODO:
  #def set_created_by
  #  self.created_by = User.current if User.current
  #end

  def self.for_user(user, options = {}, is_admin = false)

    # TODO: thought i  should reverse keys for descending, but that didn't work. look into whether that should be tweaked, and whether it works okay with pagination (seems to now...)

    options[:user_id] = user.id
    options[:is_admin] = is_admin

    @selection = TicketSelection.new(options)
    @selection.tickets
  end

  #def self.tickets_by_commenter(user_id)#, options = {})
  #  Ticket.includes_post_by_and_updated_at.startkey([user_id, 0]).endkey([user_id, Time.now])
  #end

  def is_creator_validated?
    !!created_by
  end

=begin
  def set_code #let's not use this---can use same show url
    # ruby 1.9 provides url-safe option---this is not necessarily url-safe
    self.code = SecureRandom.hex(8) if !is_creator_validated?
  end
=end


  def set_email
    self.email = nil if self.email == ""
    # in controller set to be current users email if that exists
  end

  #not saving with close and reopen, as we will save in update when they are called.
  #TODO: not sure if we should bother with these:
  def close
    self.is_open = false
    #save
  end

  def reopen
    self.is_open = true
    #save
  end

  def commenters
    commenters = []
    self.comments.each do |comment|
      if comment.posted_by
        if user = User.find(comment.posted_by)
          commenters << user.login if user and !commenters.include?(user.login)
        else
          commenters << 'unknown user' if !commenters.include?('unknown user') #todo don't hardcode string 'unknown user'
        end
      else
        commenters << 'unauthenticated user' if !commenters.include?('unauthenticated user') #todo don't hardcode string 'unauthenticated user'
      end
    end
    commenters.join(', ')
  end

  def comments_attributes=(attributes)
    if attributes # could be empty as we will empty if nothing was typed in
      comment = TicketComment.new(attributes.values.first) #TicketComment.new(attributes)
      #comment.posted_by = User.current.id if User.current #we want to avoid User.current, and current_user won't work here. instead will set in tickets_controller
      # what about: comment.posted_by = self.updated_by  (will need to add ticket.updated_by)
      comment.posted_at = Time.now
      comments << comment
    end
  end

=begin
  def validate
    if email_address and not email_address.strip =~ RFC822::EmailAddress
      errors.add 'email', 'contains an invalid address'
    end
  end
=end
end