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
220
221
222
223
224
225
226
227
228
229
230
231
|
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: This is obviously super tedious. we will refactor later.
# 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...)
# TODO: Time.now + 2.days is to catch tickets created in future. shouldn't happen but does on my computer now, so this at least catches for now.
# TODO handle default values correctly:
options[:open_status] = 'open' if !options[:open_status] #hacky. redo this when handling defaults correctly
options[:sort_order] = 'updated_at_desc' if !options[:sort_order] #hacky. redo this when handling defaults correctly
options[:user_id] = user.id
options[:is_admin] = is_admin
@selection = TicketSelection.new(options)
#TODO: can this be more succinct?
if @selection.order
@tickets = Ticket.send(@selection.finder_method).startkey(@selection.startkey).endkey(@selection.endkey).send(@selection.order)
else
@tickets = Ticket.send(@selection.finder_method).startkey(@selection.startkey).endkey(@selection.endkey)
end
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
|
