[jira] [Commented] (FOP-2514) [PATCH] Empty hyphenation-character leads to "String index out of range: 0"

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (FOP-2514) [PATCH] Empty hyphenation-character leads to "String index out of range: 0"

Andreas Joseph Krogh (Jira)

    [ https://issues.apache.org/jira/browse/FOP-2514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16803046#comment-16803046 ]

Chris Bowditch commented on FOP-2514:
-------------------------------------

I still can't commit your change. Looks like it breaks a unit test:

[junit] Testcase: runTest[6](org.apache.fop.intermediate.IFTestCase):
FAILED
 [junit] Expected XPath expression to evaluate to 'true', but got '' (XPath:
/descendant::if:text[1]/@hyphenated)
 [junit] junit.framework.AssertionFailedError: Expected XPath expression to e
valuate to 'true', but got '' (XPath: /descendant::if:text[1]/@hyphenated)
 [junit] at org.apache.fop.layoutengine.EvalCheck.doCheck(EvalCheck.java:
86)
 [junit] at org.apache.fop.layoutengine.EvalCheck.check(EvalCheck.java:65
)

> [PATCH] Empty hyphenation-character leads to "String index out of range: 0"
> ---------------------------------------------------------------------------
>
>                 Key: FOP-2514
>                 URL: https://issues.apache.org/jira/browse/FOP-2514
>             Project: FOP
>          Issue Type: Bug
>    Affects Versions: 2.0
>            Reporter: Bj√∂rn Kautler
>            Assignee: Chris Bowditch
>            Priority: Major
>         Attachments: hyphen.fo, issue-2514.patch
>
>
> If you set hyphenation-character to the empty value, then you get the error message "String index out of range: 0".
> If there is no hyphenation-character set, there should simply nothing be used.
> This is useful in cases where you want hyphenation but no hyphenation character like in program listings or similar



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)