Fix Person.find_by_substring with frozen strings

95567904 · Benjamin Neff · 723be461 · 95567904
Unverified Commit 95567904 authored Aug 27, 2017 by Benjamin Neff
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

app/models/person.rb app/models/person.rb +1 -1

No files found.
--- a/app/models/person.rb
+++ b/app/models/person.rb
@@ -104,7 +104,7 @@ class Person < ApplicationRecord
  # @param [String] search substring
  # @return [Person::ActiveRecord_Relation]
  scope :find_by_substring, ->(search_str) {
-    search_str.strip!
+    search_str = search_str.strip
    if search_str.blank? || search_str.size < 2
      none
    else