From cebec27e8053e5f9acebc48a084aadc9cddac300 Mon Sep 17 00:00:00 2001 From: Brenda Wallace Date: Sun, 2 Aug 2020 13:57:20 +1200 Subject: [PATCH] Prefer ary[n..] over ary[n..-1]. --- app/models/post.rb | 2 +- lib/haml/filters/growstuff_markdown.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/post.rb b/app/models/post.rb index d84e2dc5d..057402187 100644 --- a/app/models/post.rb +++ b/app/models/post.rb @@ -82,7 +82,7 @@ class Post < ApplicationRecord end body.scan(Haml::Filters::GrowstuffMarkdown::MEMBER_AT_REGEX) do |_m| # find member case-insensitively and add to list of recipients - member = Member.case_insensitive_login_name(Regexp.last_match(1)[1..-1]).first + member = Member.case_insensitive_login_name(Regexp.last_match(1)[1..]).first recipients << member if member && recipients.exclude?(member) end # don't send notifications to yourself diff --git a/lib/haml/filters/growstuff_markdown.rb b/lib/haml/filters/growstuff_markdown.rb index 5295140e4..e84465be1 100644 --- a/lib/haml/filters/growstuff_markdown.rb +++ b/lib/haml/filters/growstuff_markdown.rb @@ -65,7 +65,7 @@ module Haml::Filters def find_member(login_name) # Remove @ if present - login_name = login_name[1..-1] if login_name.start_with?('@') + login_name = login_name[1..] if login_name.start_with?('@') Member.case_insensitive_login_name(login_name).first end end