Created an attachment (id=17803) [details]
Spreadsheet with some data and a chart from OOo Calc

SVN commit 589745 by ingwa:

ODF: Create support for data in rows or columns

Yay!  This is the first ODF commit for kchart in a loooong time.  Viva
      aKademy for giving me time to actually do this.

CCBUGS: 134195

Btw, Bug 134195 is a great testcase because it is simple and yet
contains a number of things that don't work yet.



 M  +5 -0      ChangeLog  
 M  +8 -2      kchart_params.cc  


--- branches/koffice/1.6/koffice/kchart/ChangeLog #589744:589745
@@ -1,3 +1,8 @@
+2006-09-28  Inge Wallin  <inge@lysator.liu.se>
+
+	* kchart_params.cc (loadOasisPlotarea): Add support for
+	chart:series-source (row or column):
+
 2006-08-11  Inge Wallin  <inge@lysator.liu.se>
 
 	Only show checkboxes in config dialog if applicable.
--- branches/koffice/1.6/koffice/kchart/kchart_params.cc #589744:589745
@@ -572,7 +572,11 @@
     // chart:stock-with-volume      - boolean
     // chart:japanese-candle-sticks - boolean
 
-    // chart:series-source     - "row" or "columns
+    // chart:series-source     - "rows" or "columns"
+    // "columns" is the default
+    if ( styleStack.attributeNS( KoXmlNS::chart, "series-source" ) == "rows" ) {
+      setDataDirection( DataRows );
+    }
 
     // chart:data-label-number - "value" / "percentage" / "none" (def: none)
 
@@ -957,6 +961,9 @@
 	break;
     }
 
+    // chart:series-source
+    plotAreaStyle.addProperty( "chart:series-source",
+			       ( dataDirection() == DataRows ) ? "rows" : "columns" );
     // Register the style, and get back its auto-generated name
     const QString styleName = mainStyles.lookup( plotAreaStyle, "ch" );
 
@@ -965,7 +972,6 @@
     saveOasisAxis( bodyWriter, mainStyles, KDChartAxisParams::AxisPosBottom, "x" );
     saveOasisAxis( bodyWriter, mainStyles, KDChartAxisParams::AxisPosLeft, "y" );
 
-    // TODO chart:series
     // TODO chart:wall
     // TODO chart:floor
 }

Just found out by looking inside the file that it actually is invalid.  The
cells without value are given as "1.#NAN", which should be "NAN".  This does
not mean that kchart doesn't load the file wrong in other ways, of course. I am
on those, but you should also bug report the file to OpenOffice.org as invalid.

Hi Peter,

2.1 is coming out soon.  Do you have the same problem with the current version
of oocalc?  The file in the bugreport is actually invalid, as I wrote above.

Inge,

I wish I could help you, but KSpread in beta 3 packages from Fedora freezes for
me when attempting to open or create any document. Unfortunately I don't have
time to investigate the cause of that.

Ok, that's serious in itself. But that's more likely a packaging problem, since
I have it working here.

Created an attachment (id=37936) [details]
New test document

The chart in the attached file does not render at all in KSpread 2.1 RC1. File
created in OOo 3.1.1.

Thanks, Peter.  I'll look into that.

SVN commit 1046921 by ingwa:

* Make data direction load from the correct place.  That place is the
  plot-area style, not the plot-area element.

* Add some comments.

CCBUG: 134195



 M  +47 -25    PlotArea.cpp  


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

SVN commit 1046926 by ingwa:

Backport r1046921:

* Make data direction load from the correct place.  That place is the
  plot-area style, not the plot-area element.

* Add some comments.

BUG: 134195


 M  +46 -24    PlotArea.cpp  


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

The original bug that was shown in the attached file is now fixed.  A new bug
is entered in bug 213917.