Project

General

Profile

Actions

Defect #37718

closed

Repository browser does not show "+" (plus sign) in filename

Added by Go MAEDA over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Category:
SCM
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

As reported in the forum post Cannot browse file contents that filename contains "+" i..., plus sings in filenames are not displayed in the repository browser when browsing a Subversion repository.


Files


Related issues

Related to Redmine - Defect #32752: Ruby 2.7: Remove deprecated URI.escape/unescapeClosedGo MAEDA

Actions
Actions #1

Updated by Go MAEDA over 2 years ago

This issue is appeared to be caused by r19455 that replaced URI.escape with CGI.escape at source:trunk/lib/redmine/scm/adapters/subversion_adapter.rb@19455#L111.

$ irb
irb(main):001:0> RUBY_VERSION
=> "2.6.9" 
irb(main):002:0> require 'cgi'
=> true
irb(main):003:0> CGI.unescape('+.txt')
=> " .txt" 
irb(main):004:0> URI.unescape('+.txt')
=> "+.txt" 
Actions #2

Updated by Go MAEDA over 2 years ago

  • Related to Defect #32752: Ruby 2.7: Remove deprecated URI.escape/unescape added
Actions #3

Updated by Go MAEDA over 2 years ago

  • Affected version set to 4.2.0
Actions #5

Updated by Go MAEDA over 2 years ago

  • Target version set to Candidate for next minor release
Actions #6

Updated by Go MAEDA over 2 years ago

  • Subject changed from Repository browser does not show plus sign in filename to Repository browser does not show "+" (plus sign) in filename
Actions #7

Updated by Go MAEDA about 2 years ago

  • Target version changed from Candidate for next minor release to 4.2.9

Setting the target version to 4.2.9.

Actions #8

Updated by Marius BĂLTEANU about 2 years ago

  • Assignee set to Marius BĂLTEANU
  • Target version changed from 4.2.9 to 4.2.8
Actions #9

Updated by Marius BĂLTEANU about 2 years ago

  • Status changed from New to Resolved
  • Resolution set to Fixed

I've committed and merged the fix to 5.0-stable and 4.2-stable branches with one mention: for 4.2-stable, I've backport only the fix because the changes to the tests do not apply cleanly (caused by #16849).

Actions #10

Updated by Marius BĂLTEANU about 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF