Skip to content

Commit f7f4981

Browse files
committed
Use better approach to create an object.
1 parent 5b16511 commit f7f4981

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

model.gb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ class ListItem
6464
if result[:error].nil?
6565
title = params[:title]
6666
checked = params[:checked].to_i == 1
67-
resultID = self.db.exec(String.fmt("INSERT INTO list_items (title, checked) VALUES ('%s', %s)", title, checked))
67+
resultID = self.db.exec("INSERT INTO list_items (title, checked) VALUES ($1, $2)", title, checked)
6868
new({ id: resultID, title: title, checked: checked })
6969
else
7070
new({ error: result[:error] })
@@ -77,8 +77,6 @@ class ListItem
7777
yield({ error: 'Title cannot be empty' })
7878
elsif params[:title].empty?
7979
yield({ error: 'Title cannot be empty' })
80-
elsif params[:title].include?('"')
81-
yield({ error: 'Invalid title' })
8280
else
8381
if params[:title].length > 40
8482
yield({ error: 'Title too long (should less than 40 characters)' })

0 commit comments

Comments
 (0)