#121071#: [Regression]Item lists are incorrectly imported into AOO in DataPilot table

Reported by: Terry Yang
Reviewed by: sunying
patched by: Jianyuan Li

sc/source/filter/excel/xipivot.cxx

@@ -853,17 +853,20 @@ bool XclImpPivotCache::IsRefreshOnLoad() const
 // Pivot table
 // ============================================================================
 
-XclImpPTItem::XclImpPTItem( const XclImpPCField* pCacheField ) :
-    mpCacheField( pCacheField )
+XclImpPTItem::XclImpPTItem( const XclImpPTField& rPTField ) :
+    mrPTField( rPTField )
 {
 }
 
 const String* XclImpPTItem::GetItemName() const
 {
-    if( mpCacheField )
+    if( const XclImpPCField * mpCacheField = mrPTField.GetCacheField() )
+    {
         if( const XclImpPCItem* pCacheItem = mpCacheField->GetItem( maItemInfo.mnCacheIdx ) )
-            //! TODO: use XclImpPCItem::ConvertToText(), if all conversions are available
-            return pCacheItem->IsEmpty() ? &String::EmptyString() : pCacheItem->GetText();
+        {
+            return pCacheItem->GetItemName();
+        }
+    }
     return 0;
 }
 
@@ -948,7 +951,7 @@ void XclImpPTField::ReadSxvdex( XclImpStream& rStrm )
 
 void XclImpPTField::ReadSxvi( XclImpStream& rStrm )
 {
-    XclImpPTItemRef xItem( new XclImpPTItem( GetCacheField() ) );
+    XclImpPTItemRef xItem( new XclImpPTItem( *this ) );
     maItems.push_back( xItem );
     xItem->ReadSxvi( rStrm );
 }

sc/source/filter/excel/xlpivot.cxx

@@ -158,6 +158,37 @@ const bool* XclPCItem::GetBool() const
     return (meType == EXC_PCITEM_BOOL) ? &mbValue : 0;
 }
 
+const String* XclPCItem::GetItemName() const
+{
+    //! TODO: use XclImpPCItem::ConvertToText(), if all conversions are available
+    if( EXC_PCITEM_BOOL == this->GetType() )
+    {
+        static String szBool[] = { String::CreateFromAscii("FALSE"), String::CreateFromAscii("TRUE") };
+
+        if( const bool* pBool = this->GetBool() )
+        {
+            return &(!!*pBool)[szBool];
+        }
+        else
+        {
+            return &0[szBool];
+        }
+    }
+    else
+    {
+        if( this->IsEmpty())
+        {
+            static String aEmptyString( String::EmptyString() );
+            return &aEmptyString;
+        }
+        else
+            return this->GetText();
+    }
+
+    return NULL;
+}
+
+
 // Field settings =============================================================
 
 XclPCFieldInfo::XclPCFieldInfo() :

sc/source/filter/inc/xipivot.hxx

@@ -199,13 +199,14 @@ typedef ScfRef< XclImpPivotCache > XclImpPivotCacheRef;
 // ============================================================================
 
 class XclImpPivotTable;
+class XclImpPTField;
 
 // ============================================================================
 
 class XclImpPTItem
 {
 public:
-    explicit            XclImpPTItem( const XclImpPCField* pCacheField );
+    explicit            XclImpPTItem( const XclImpPTField & );
 
     /** Returns the internal name of the item or 0, if no name could be found. */
     const String*       GetItemName() const;
@@ -220,7 +221,8 @@ public:
 
 private:
     XclPTItemInfo       maItemInfo;         /// General data for this item.
-    const XclImpPCField* mpCacheField;      /// Corresponding pivot cache field.
+    //const XclImpPCField* mpCacheField;      /// Corresponding pivot cache field.
+    const XclImpPTField & mrPTField;
 };
 
 typedef ScfRef< XclImpPTItem > XclImpPTItemRef;

sc/source/filter/inc/xlpivot.hxx

@@ -447,6 +447,7 @@ public:
     const sal_uInt16*   GetError() const;
     /** Returns pointer to Boolean value, if the item type is 'boolean', otherwise 0. */
     const bool*         GetBool() const;
+    const String *      GetItemName() const;
 
 private:
     XclPCItemType       meType;         /// Type of the item.