Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Update url_generator.rb #2676

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update url_generator.rb #2676

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a75f49f
Update url_generator.rb
iqre8 Mar 14, 2021
22721c7
Update url_generator.rb
iqre8 Mar 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion lib/paperclip/url_generator.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,9 @@ def escape_url(url)
if url.respond_to?(:escape)
url.escape
else
URI.escape(url).gsub(escape_regex){|m| "%#{m.ord.to_s(16).upcase}" }
URI.encode_www_form_component(url).gsub(escape_regex) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/BlockDelimiters: Avoid using {...} for multi-line blocks.
Layout/TrailingWhitespace: Trailing whitespace detected.

|m| "%#{m.ord.to_s(16).upcase}"
iqre8 marked this conversation as resolved.
Show resolved Hide resolved
}
end
end

Expand Down