493354 - Incorrect results returned by getTextBeforeOffset

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Joanmarie Diggs]

Attachment: test case

Steps to reproduce:

1. Load the attached test case.

2. In Accerciser, select the accessible associated with the text object from the tree on the left.

3. In Accerciser's IPython Console, use getTextBeforeOffset to obtain the word and the character at any specified offset within the text.

Expected results: A string of the specified boundary type (i.e. word or character) would be returned along with the correct start and end offsets for that string. For example: ('silly ', 17, 23) or ('s', 17, 18).

Actual results: The start offset appears to be correct, but the string and end offset are bogus:

In [1]:text = acc.queryText()
In [2]:for i in range(text.characterCount): text.getTextBeforeOffset(i, TEXT_BOUNDARY_WORD_START)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('This is another, silly test. ', 0, -39238599)
Out[2]: ('is another, silly test. ', 5, -39238599)
Out[2]: ('is another, silly test. ', 5, -39238599)
Out[2]: ('is another, silly test. ', 5, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('another, silly test. ', 8, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('silly test. ', 17, -39238599)
Out[2]: ('test. ', 23, -39238599)
Out[2]: ('test. ', 23, -39238599)
Out[2]: ('test. ', 23, -39238599)
Out[2]: ('test. ', 23, -39238599)
Out[2]: ('test. ', 23, -39238599)

In [3]:for i in range(text.characterCount): text.getTextBeforeOffset(i, TEXT_BOUNDARY_CHAR)
Out[3]: ('This is another, silly test. ', 0, -39238599)
Out[3]: ('This is another, silly test. ', 0, -39238599)
Out[3]: ('his is another, silly test. ', 1, -39238599)
Out[3]: ('is is another, silly test. ', 2, -39238599)
Out[3]: ('s is another, silly test. ', 3, -39238599)
Out[3]: (' is another, silly test. ', 4, -39238599)
Out[3]: ('is another, silly test. ', 5, -39238599)
Out[3]: ('s another, silly test. ', 6, -39238599)
Out[3]: (' another, silly test. ', 7, -39238599)
Out[3]: ('another, silly test. ', 8, -39238599)
Out[3]: ('nother, silly test. ', 9, -39238599)
Out[3]: ('other, silly test. ', 10, -39238599)
Out[3]: ('ther, silly test. ', 11, -39238599)
Out[3]: ('her, silly test. ', 12, -39238599)
Out[3]: ('er, silly test. ', 13, -39238599)
Out[3]: ('r, silly test. ', 14, -39238599)
Out[3]: (', silly test. ', 15, -39238599)
Out[3]: (' silly test. ', 16, -39238599)
Out[3]: ('silly test. ', 17, -39238599)
Out[3]: ('illy test. ', 18, -39238599)
Out[3]: ('lly test. ', 19, -39238599)
Out[3]: ('ly test. ', 20, -39238599)
Out[3]: ('y test. ', 21, -39238599)
Out[3]: (' test. ', 22, -39238599)
Out[3]: ('test. ', 23, -39238599)
Out[3]: ('est. ', 24, -39238599)
Out[3]: ('st. ', 25, -39238599)
Out[3]: ('t. ', 26, -39238599)
Out[3]: ('. ', 27, -39238599)

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

This is probably a dupe of bug 433667, but I'm marking 433667 a dependancy until we are sure. Besides, this is a great report. Thanks Joanmarie. Is this bug causing lots of pain? (Feel free to tweak Importance fields)

Comment 2

[Fernando Herrera]

Just to be sure. The proper output should be something like:
- for TEXT_BOUNDARY_WORD_START

Out[5]: ('', 0, 0)
Out[5]: ('', 0, 0)
Out[5]: ('', 0, 0)
Out[5]: ('', 0, 0)
Out[5]: ('', 0, 0)
Out[5]: ('This ', 0, 5)
Out[5]: ('This ', 0, 5)
Out[5]: ('This ', 0, 5)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('is ', 5, 8)
Out[5]: ('another, ', 8, 17)
Out[5]: ('another, ', 8, 17)
Out[5]: ('another, ', 8, 17)
Out[5]: ('another, ', 8, 17)
Out[5]: ('another, ', 8, 17)
Out[5]: ('another, ', 8, 17)
Out[5]: ('silly ', 17, 23)
Out[5]: ('silly ', 17, 23)
Out[5]: ('silly ', 17, 23)
Out[5]: ('silly ', 17, 23)
Out[5]: ('silly ', 17, 23)
Out[5]: ('silly ', 17, 23)

- for TEXT_BOUNDARY_CHAR:
Out[6]: ('', 0, 0)
Out[6]: ('T', 0, 1)
Out[6]: ('h', 1, 2)
Out[6]: ('i', 2, 3)
Out[6]: ('s', 3, 4)
Out[6]: (' ', 4, 5)
Out[6]: ('i', 5, 6)
Out[6]: ('s', 6, 7)
Out[6]: (' ', 7, 8)
Out[6]: ('a', 8, 9)
Out[6]: ('n', 9, 10)
Out[6]: ('o', 10, 11)
Out[6]: ('t', 11, 12)
Out[6]: ('h', 12, 13)
Out[6]: ('e', 13, 14)
Out[6]: ('r', 14, 15)
Out[6]: (',', 15, 16)
Out[6]: (' ', 16, 17)
Out[6]: ('s', 17, 18)
Out[6]: ('i', 18, 19)
Out[6]: ('l', 19, 20)
Out[6]: ('l', 20, 21)
Out[6]: ('y', 21, 22)
Out[6]: (' ', 22, 23)
Out[6]: ('t', 23, 24)
Out[6]: ('e', 24, 25)
Out[6]: ('s', 25, 26)
Out[6]: ('t', 26, 27)
Out[6]: ('.', 27, 28)

Comment 3

[alexander :surkov (:asurkov)]

all blocking were fixed (moz24), marking this one as fixed