After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 586580 - Orca repeats the coordinates of Calc cells
Orca repeats the coordinates of Calc cells
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: speech
unspecified
Other All
: Normal normal
: 2.28.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404411
 
 
Reported: 2009-06-21 23:01 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2009-11-09 21:35 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26


Attachments
patch (578 bytes, patch)
2009-06-21 23:03 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2009-06-21 23:01:58 UTC
Steps to reproduce:

1. Launch Calc
2. Navigate amongst cells

Expected results: Orca would speak the coordinates one time.

Actual results: Orca speaks the coordinates one time per digit in the coordinates.

From debug.out:

generateSpeech for {'formatType': 'unfocused', 'role': ROLE_TABLE_CELL, 'mode': 'speech', 'recursing': True} using 'newAncestors + newRowHeader + newColumnHeader + newRadioButtonGroup + endOfTableIndicator + tableCellRow + newNodeLevel + (tutorial and (pause + tutorial) or [])'
newAncestors=[]
newRowHeader=[]
newColumnHeader=[]
newRadioButtonGroup=[]
endOfTableIndicator=[]
tableCellRow=['', 'A1009', 'A1009', 'A1009', 'A1009']
newNodeLevel=[]
GENERATOR: _getTutorialForTableCell
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
GENERATOR: _getTutorialForTableCellRow
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
GENERATOR: getTutorial
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
               ()
tutorial=['']
pause=[<orca.speech_generator.Pause instance at 0x85bf46c>]
generateSpeech generated '['', 'A1009', 'A1009', 'A1009', 'A1009', <orca.speech_generator.Pause instance at 0x85bf46c>, '']'
SPEECH OUTPUT: 'A1009 A1009 A1009 A1009.'
Comment 1 Joanmarie Diggs (IRC: joanie) 2009-06-21 23:03:29 UTC
Created attachment 137143 [details] [review]
patch

With this patch:
~~~~~~~~~~~
generateSpeech for {'formatType': 'unfocused', 'role': ROLE_TABLE_CELL, 'mode': 'speech', 'recursing': True} using 'newAncestors + newRowHeader + newColumnHeader + newRadioButtonGroup + endOfTableIndicator + tableCellRow + newNodeLevel + (tutorial and (pause + tutorial) or [])'
newAncestors=[]
newRowHeader=[]
newColumnHeader=[]
newRadioButtonGroup=[]
endOfTableIndicator=[]
tableCellRow=['', 'A1009']
newNodeLevel=[]
GENERATOR: _getTutorialForTableCell
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
GENERATOR: _getTutorialForTableCellRow
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
GENERATOR: getTutorial
           obj             = Cell A1009 
           role            = table cell
           alreadyFocused  = False
           utterances:
               ()
tutorial=['']
pause=[<orca.speech_generator.Pause instance at 0x85bf46c>]
generateSpeech generated '['', 'A1009', <orca.speech_generator.Pause instance at 0x85bf46c>, '']'
SPEECH OUTPUT: 'A1009.'
~~~~~~~~~~~

Will, please review. Thanks! (And this only seems to occur post refactor.)
Comment 2 Willie Walker 2009-06-21 23:17:41 UTC
Yep - looks like the 'break' accomplishes the equivalent of the 'return utterances' line in gnome-2-26: http://git.gnome.org/cgit/orca/tree/src/orca/scripts/apps/soffice/speech_generator.py?h=gnome-2-26#n289

Thanks for finding this.
Comment 3 Joanmarie Diggs (IRC: joanie) 2009-06-21 23:52:58 UTC
Thanks Will. Committed to master.