Actions
Defect #7794
closedInternal server error on formatting an issue as a PDF in Japanese
Start date:
2011-03-06
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
IssuesController#show fails with internal server error when
- params[:format] is pdf
- AcceptLanguage: ja
- @issue.description contains an ASCII control character.
This is because the map SJIS_widths does not have a mapping for control characters and it leads concatenation of a String and nil at vendor/plugins/rfpdf/lib/rfpdf/japanese.rb:182.
The attached patch fixes this problem.
Files
Related issues
Updated by Toshi MARUYAMA over 13 years ago
- Status changed from New to 7
- Assignee set to Toshi MARUYAMA
Updated by Toshi MARUYAMA over 13 years ago
- Tracker changed from Patch to Defect
- Subject changed from Fix for an internal server error on formatting an issue as a PDF in Japanese to Internal server error on formatting an issue as a PDF in Japanese
Updated by Toshi MARUYAMA over 13 years ago
- Target version set to 1.1.3
- Affected version (unused) set to 1.1.2
- Affected version set to 1.1.2
I confirmed Japanese locale has problem.
Korean, Simplified and Traditional Chinese don't have problem.
Updated by Toshi MARUYAMA over 13 years ago
- Status changed from 7 to Closed
- Resolution set to Fixed
Updated by Jun NAITOH over 13 years ago
- File test-1-org.pdf test-1-org.pdf added
- File test-1-ja-pdf.patch.pdf test-1-ja-pdf.patch.pdf added
- File test-1-ja-pdf_fix.patch.pdf test-1-ja-pdf_fix.patch.pdf added
- File ja-pdf_fix.patch ja-pdf_fix.patch added
- Status changed from Closed to Reopened
When I tested #61 in trunk, now. I found ja-pdf.patch problem.
I retested in Redmine 1.1.2 + ja-pdf.patch.
Test string is "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z a b c d e f g h i j k l m n o p q r s t u v w x y z 1 2 3 4 5 6 7 8 9 0"
- Redmine 1.1.2 : test-1-org.pdf
- Redmine 1.1.2 + ja-pdf.patch : test-1-ja-pdf.patch.pdf
The width of the ASCII character has changed by ja-pdf.patch.
I rewrite ja-pdf_fix.patch for Redmine 1.1.2.
- Redmine 1.1.2 + ja-pdf_fix.patch : test-1-ja-pdf_fix.patch.pdf
Updated by Toshi MARUYAMA over 13 years ago
- Status changed from Reopened to Closed
I have committed r5271 in 1.1-stable.
Thanks.
Updated by Toshi MARUYAMA over 13 years ago
- Status changed from Closed to Reopened
Updated by Toshi MARUYAMA over 13 years ago
- Status changed from Reopened to Closed
Actions