diff options
Diffstat (limited to 'app')
| -rw-r--r-- | app/models/search.rb | 27 | ||||
| -rw-r--r-- | app/views/tags/_taggable.html.erb | 2 |
2 files changed, 27 insertions, 2 deletions
diff --git a/app/models/search.rb b/app/models/search.rb index 7823a74..053e641 100644 --- a/app/models/search.rb +++ b/app/models/search.rb @@ -31,7 +31,7 @@ class Search < ApplicationRecord def parse_body res = { tag: [], plain: [] } - body.split.each do |part| + split_body.each do |part| if part.include?(':') parts = part.split(':', 2) parts[1] = clean_clause(part: parts[1]) @@ -47,6 +47,31 @@ class Search < ApplicationRecord res end + # Returns the body split into its constituents. Note that we cannot mindlessly + # do something like `body.split` because that would ignore quotes that are not + # to be split among other things. + def split_body + cur = '' + res = [] + co = '' + + body.each_char do |c| + if c.match?(/\s/) && co == '' + res << cur if cur.present? + cur = '' + else + # If it is a quote character, check whether it's being opened or + # closed. + co = c == co ? '' : c if c == '"' || c == "'" + + cur += c + end + end + + res << cur if cur.present? && co.blank? + res + end + # Returns the argument without any leading/trailing quotes. def clean_clause(part:) part.gsub(/^("|')+/, '').gsub(/("|')+$/, '') diff --git a/app/views/tags/_taggable.html.erb b/app/views/tags/_taggable.html.erb index 84e236b..5d848d0 100644 --- a/app/views/tags/_taggable.html.erb +++ b/app/views/tags/_taggable.html.erb @@ -3,7 +3,7 @@ <h5><%= I18n.t('tags.title') %> — <%= link_to I18n.t('tags.new-action'), new_tag_path %></h5> <div class="flex-list"> - <% Tag.find_each do |tag| %> + <% Tag.order(:name).find_each do |tag| %> <div class="flex-element"> <label> <%= form.check_box :tag_ids, { multiple: true, checked: tag_references&.include?(tag.id) }, tag.id, nil %> |