[Libreoffice-commits] core.git: Branch 'libreoffice-4-1-5' - sc/source

[Eike Rathke]

 sc/source/core/tool/cellform.cxx |   10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

New commits:
commit b00aee5aa776c99940d858a8b73557612a558d01
Author: Eike Rathke <er...@redhat.com>
Date:   Mon Jan 20 19:21:55 2014 +0100

    resolved fdo#73836 do not apply format twice to string
    
    ScDocument::GetString() already returns the formatted string that then
    was formatted through number formatter. Format raw string instead.
    
    (cherry picked from commit f6b9b73a1f87d1815ed8ab8f2494e5526a710bb9)
    
    Change-Id: I806b4fc904d04169a6c1dbd4a3d00e2556eda7bd
    Reviewed-on: https://gerrit.libreoffice.org/7554
    Reviewed-by: Kohei Yoshida <libreoff...@kohei.us>
    Tested-by: Kohei Yoshida <libreoff...@kohei.us>
    Reviewed-by: David Tardon <dtar...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sc/source/core/tool/cellform.cxx b/sc/source/core/tool/cellform.cxx
index bc65f91..0893149 100644
--- a/sc/source/core/tool/cellform.cxx
+++ b/sc/source/core/tool/cellform.cxx
@@ -143,14 +143,16 @@ OUString ScCellFormat::GetString(
     {
         case CELLTYPE_STRING:
         {
-            OUString aCellString = rDoc.GetString(rPos);
-            rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor, 
bUseStarFormat);
+            ScRefCellValue aCell;
+            aCell.assign(rDoc, rPos);
+            rFormatter.GetOutputString(aCell.getString(&rDoc), nFormat, 
aString, ppColor, bUseStarFormat);
         }
         break;
         case CELLTYPE_EDIT:
         {
-            OUString aCellString = rDoc.GetString(rPos);
-            rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor);
+            ScRefCellValue aCell;
+            aCell.assign(rDoc, rPos);
+            rFormatter.GetOutputString(aCell.getString(&rDoc), nFormat, 
aString, ppColor);
         }
         break;
         case CELLTYPE_VALUE:

_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits