From 80b7a83940752f3647dd2974a112056e37438cef Mon Sep 17 00:00:00 2001
From: Go MAEDA <maeda@farend.jp>
Date: Wed, 24 Aug 2022 13:45:20 +0000
Subject: [PATCH] Use start_with? or end_with? to check the first or last
 character of a string (#37591).

Patch by Go MAEDA.


git-svn-id: https://svn.redmine.org/redmine/trunk@21774 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 lib/redmine/diff_table.rb                    | 6 +++---
 lib/redmine/export/pdf.rb                    | 2 +-
 lib/redmine/scm/adapters/abstract_adapter.rb | 6 +++---
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/lib/redmine/diff_table.rb b/lib/redmine/diff_table.rb
index 2976d2228..ca8c4b9d2 100644
--- a/lib/redmine/diff_table.rb
+++ b/lib/redmine/diff_table.rb
@@ -118,7 +118,7 @@ module Redmine
     end
 
     def parse_line(line, type="inline")
-      if line[0, 1] == "+"
+      if line.start_with?('+')
         diff = diff_for_added_line
         diff.line_right = line[1..-1]
         diff.nb_line_right = @line_num_r
@@ -126,7 +126,7 @@ module Redmine
         @line_num_r += 1
         @added += 1
         true
-      elsif line[0, 1] == "-"
+      elsif line.start_with?('-')
         diff = Diff.new
         diff.line_left = line[1..-1]
         diff.nb_line_left = @line_num_l
@@ -137,7 +137,7 @@ module Redmine
         true
       else
         write_offsets
-        if /\s/.match?(line[0, 1])
+        if line.start_with?(/\s/)
           diff = Diff.new
           diff.line_right = line[1..-1]
           diff.nb_line_right = @line_num_r
diff --git a/lib/redmine/export/pdf.rb b/lib/redmine/export/pdf.rb
index e586a2a0a..cdf8eca3c 100644
--- a/lib/redmine/export/pdf.rb
+++ b/lib/redmine/export/pdf.rb
@@ -120,7 +120,7 @@ module Redmine
         end
 
         def get_sever_url(url)
-          if !empty_string(url) and (url[0, 1] == '/')
+          if !empty_string(url) and url.start_with?('/')
             Setting.host_name.split('/')[0] + url
           else
             url
diff --git a/lib/redmine/scm/adapters/abstract_adapter.rb b/lib/redmine/scm/adapters/abstract_adapter.rb
index 5d92bb2d7..0ad3d7f45 100644
--- a/lib/redmine/scm/adapters/abstract_adapter.rb
+++ b/lib/redmine/scm/adapters/abstract_adapter.rb
@@ -158,12 +158,12 @@ module Redmine
 
         def with_leading_slash(path)
           path ||= ''
-          (path[0, 1]!="/") ? "/#{path}" : path
+          path.start_with?('/') ? path : "/#{path}"
         end
 
         def with_trailling_slash(path)
           path ||= ''
-          (path[-1, 1] == "/") ? path : "#{path}/"
+          path.end_with?('/') ? path : "#{path}/"
         end
 
         def without_leading_slash(path)
@@ -173,7 +173,7 @@ module Redmine
 
         def without_trailling_slash(path)
           path ||= ''
-          (path[-1, 1] == "/") ? path[0..-2] : path
+          path.end_with?('/') ? path[0..-2] : path
         end
 
         def valid_name?(name)
-- 
2.39.5