Bug 173293

Summary: Editing formula twice causes wrong circle error
Product: [Applications] calligrasheets Reporter: Sven Langkamp <sven.langkamp>
Component: generalAssignee: Calligra Sheets (KSpread) Bugs <calligra-sheets-bugs-null>
Status: RESOLVED FIXED    
Severity: grave CC: gassauer
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Sven Langkamp 2008-10-22 12:51:34 UTC
Version:           2.0 beta-2 (using 4.1.1 (KDE 4.1.0 (4.1 >= 20080722)) "release 40.1", KDE:KDE4:Factory:Desktop / openSUSE_11.0)
Compiler:          gcc
OS:                Linux (i686) release 2.6.25.16-0.1-pae

To reproduce:
-insert e.g. -5 into Cell A1
-insert formula =ABS(A1) into A2
-select A2 again, but don't change anything
-deselect A2

Result:
A1 and A2 show the correct values after the first editing, but #CIRCLE! after the second
Comment 1 Sven Langkamp 2008-10-25 10:11:48 UTC
*** Bug 173496 has been marked as a duplicate of this bug. ***
Comment 2 Tomas Mecir 2008-10-26 17:59:53 UTC
SVN commit 876167 by mecir:

Improving and sanitizing the reference selection code. This fixes
problems where formula references got incorrectly remembered as a
selection upon editing a cell, changing a wrong cell and causing many
problems.

Furthermore, enabled signal calling when doing reference selection list
updates (no idea why Stefan decided to disable them). This means that
the colored boxes around referenced cells within a formula are actually
displayed again.

Also unified the two editing widgets - each of them was using a
different code to update cells and such, which led to a mess. Now one of
the widgets simply relays all keyboard events to the other, who updates
the text in both. This greatly improves KSpread's behaviour when editing
cells.

Some issues still remain - most notably, selecting the text in the
toolbox editor will work oddly. I will fix this soon.

So few lines of code, so many improvements. Yes, I'm good ;)

BUG: 173293



 M  +5 -0      Selection.cpp  
 M  +26 -37    ui/Editors.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=876167