PDF: fix 0x5c(backslash) escape processing in FPDF (#61).

Japanese CP932(Shift_JIS) and Traditional Chinese Big5 have 0x5c(backslash) problem.

Contributed Jun NAITOH.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@5565 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Toshi MARUYAMA 2011-04-29 05:35:26 +00:00
parent ed3efeb960
commit f62605c636
3 changed files with 3 additions and 51 deletions

View File

@ -163,8 +163,6 @@ module Redmine
end end
txt = txtar txt = txtar
end end
# 0x5c char handling
txt.gsub(/\\/, "\\\\\\\\")
end end
def RDMCell(w,h=0,txt='',border=0,ln=0,align='',fill=0,link='') def RDMCell(w,h=0,txt='',border=0,ln=0,align='',fill=0,link='')

View File

@ -27,46 +27,6 @@ class PdfTest < ActiveSupport::TestCase
assert_equal '', pdf.fix_text_encoding(nil) assert_equal '', pdf.fix_text_encoding(nil)
end end
def test_fix_text_encoding_backslash_ascii
set_language_if_valid 'ja'
pdf = Redmine::Export::PDF::IFPDF.new('ja')
assert pdf
assert_equal '\\\\abcd', pdf.fix_text_encoding('\\abcd')
assert_equal 'abcd\\\\', pdf.fix_text_encoding('abcd\\')
assert_equal 'ab\\\\cd', pdf.fix_text_encoding('ab\\cd')
assert_equal '\\\\abcd\\\\', pdf.fix_text_encoding('\\abcd\\')
assert_equal '\\\\abcd\\\\abcd\\\\',
pdf.fix_text_encoding('\\abcd\\abcd\\')
end
def test_fix_text_encoding_double_backslash_ascii
set_language_if_valid 'ja'
pdf = Redmine::Export::PDF::IFPDF.new('ja')
assert pdf
assert_equal '\\\\\\\\abcd', pdf.fix_text_encoding('\\\\abcd')
assert_equal 'abcd\\\\\\\\', pdf.fix_text_encoding('abcd\\\\')
assert_equal 'ab\\\\\\\\cd', pdf.fix_text_encoding('ab\\\\cd')
assert_equal 'ab\\\\\\\\cd\\\\de', pdf.fix_text_encoding('ab\\\\cd\\de')
assert_equal '\\\\\\\\abcd\\\\\\\\', pdf.fix_text_encoding('\\\\abcd\\\\')
assert_equal '\\\\\\\\abcd\\\\\\\\abcd\\\\\\\\',
pdf.fix_text_encoding('\\\\abcd\\\\abcd\\\\')
end
def test_fix_text_encoding_backslash_ja_cp932
pdf = Redmine::Export::PDF::IFPDF.new('ja')
assert pdf
assert_equal "\x83\\\\\x98A",
pdf.fix_text_encoding("\xe3\x82\xbd\xe9\x80\xa3")
assert_equal "\x83\\\\\x98A\x91\xe3\x95\\\\",
pdf.fix_text_encoding("\xe3\x82\xbd\xe9\x80\xa3\xe4\xbb\xa3\xe8\xa1\xa8")
assert_equal "\x91\xe3\x95\\\\\\\\",
pdf.fix_text_encoding("\xe4\xbb\xa3\xe8\xa1\xa8\\")
assert_equal "\x91\xe3\x95\\\\\\\\\\\\",
pdf.fix_text_encoding("\xe4\xbb\xa3\xe8\xa1\xa8\\\\")
assert_equal "\x91\xe3\x95\\\\a\\\\",
pdf.fix_text_encoding("\xe4\xbb\xa3\xe8\xa1\xa8a\\")
end
def test_fix_text_encoding_cannot_convert_ja_cp932 def test_fix_text_encoding_cannot_convert_ja_cp932
pdf = Redmine::Export::PDF::IFPDF.new('ja') pdf = Redmine::Export::PDF::IFPDF.new('ja')
assert pdf assert pdf

View File

@ -642,10 +642,7 @@ class FPDF
def Text(x, y, txt) def Text(x, y, txt)
# Output a string # Output a string
txt.gsub!(')', '\\)') s=sprintf('BT %.2f %.2f Td (%s) Tj ET',x*@k,(@h-y)*@k, escape(txt));
txt.gsub!('(', '\\(')
txt.gsub!('\\', '\\\\')
s=sprintf('BT %.2f %.2f Td (%s) Tj ET',x*@k,(@h-y)*@k,txt);
s=s+' '+dounderline(x,y,txt) if @underline and txt!='' s=s+' '+dounderline(x,y,txt) if @underline and txt!=''
s='q '+@TextColor+' '+s+' Q' if @ColorFlag s='q '+@TextColor+' '+s+' Q' if @ColorFlag
out(s) out(s)
@ -719,14 +716,11 @@ class FPDF
else else
dx=@cMargin dx=@cMargin
end end
txt = txt.gsub(')', '\\)')
txt.gsub!('(', '\\(')
txt.gsub!('\\', '\\\\')
if @ColorFlag if @ColorFlag
s=s+'q '+@TextColor+' ' s=s+'q '+@TextColor+' '
end end
s=s+sprintf('BT %.2f %.2f Td (%s) Tj ET', s=s+sprintf('BT %.2f %.2f Td (%s) Tj ET',
(@x+dx)*@k,(@h-(@y+0.5*h+0.3*@FontSize))*@k,txt) (@x+dx)*@k,(@h-(@y+0.5*h+0.3*@FontSize))*@k,escape(txt))
s=s+' '+dounderline(@x+dx,@y+0.5*h+0.3*@FontSize,txt) if @underline s=s+' '+dounderline(@x+dx,@y+0.5*h+0.3*@FontSize,txt) if @underline
s=s+' Q' if @ColorFlag s=s+' Q' if @ColorFlag
if link and link != '' if link and link != ''
@ -1538,7 +1532,7 @@ class FPDF
def escape(s) def escape(s)
# Add \ before \, ( and ) # Add \ before \, ( and )
s.gsub('\\','\\\\').gsub('(','\\(').gsub(')','\\)') s.gsub('\\','\\\\\\').gsub('(','\\(').gsub(')','\\)')
end end
def putstream(s) def putstream(s)