Fixed some memory leaks
[pspp] / src / data / ods-reader.c
1 /* PSPP - a program for statistical analysis.
2    Copyright (C) 2011, 2012, 2013 Free Software Foundation, Inc.
3
4    This program is free software: you can redistribute it and/or modify
5    it under the terms of the GNU General Public License as published by
6    the Free Software Foundation, either version 3 of the License, or
7    (at your option) any later version.
8
9    This program is distributed in the hope that it will be useful,
10    but WITHOUT ANY WARRANTY; without even the implied warranty of
11    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
12    GNU General Public License for more details.
13
14    You should have received a copy of the GNU General Public License
15    along with this program.  If not, see <http://www.gnu.org/licenses/>. */
16
17 #include <config.h>
18
19 #include "libpspp/message.h"
20 #include "libpspp/misc.h"
21 #include "libpspp/assertion.h"
22
23 #include "data/data-in.h"
24
25 #include "gl/c-strtod.h"
26 #include "gl/minmax.h"
27
28 #include "gettext.h"
29 #define _(msgid) gettext (msgid)
30 #define N_(msgid) (msgid)
31
32 #include "ods-reader.h"
33 #include "spreadsheet-reader.h"
34
35 #if !ODF_READ_SUPPORT
36
37 struct casereader *
38 ods_open_reader (const struct spreadsheet_read_options *opts, 
39                  struct dictionary **dict)
40 {
41   msg (ME, _("Support for %s files was not compiled into this installation of PSPP"), "OpenDocument");
42
43   return NULL;
44 }
45
46 #else
47
48 #include "libpspp/zip-reader.h"
49
50
51 #include <assert.h>
52 #include <stdbool.h>
53 #include <errno.h>
54 #include <libxml/xmlreader.h>
55 #include <zlib.h>
56
57 #include "data/format.h"
58 #include "data/case.h"
59 #include "data/casereader-provider.h"
60 #include "data/dictionary.h"
61 #include "data/identifier.h"
62 #include "data/value.h"
63 #include "data/variable.h"
64 #include "libpspp/i18n.h"
65 #include "libpspp/str.h"
66
67 #include "gl/xalloc.h"
68
69 static void ods_file_casereader_destroy (struct casereader *, void *);
70
71 static struct ccase *ods_file_casereader_read (struct casereader *, void *);
72
73 static const struct casereader_class ods_file_casereader_class =
74   {
75     ods_file_casereader_read,
76     ods_file_casereader_destroy,
77     NULL,
78     NULL,
79   };
80
81 struct sheet_detail
82 {
83   /* The name of the sheet (utf8 encoding) */
84   char *name;
85
86   int start_col;
87   int stop_col;
88   int start_row;
89   int stop_row;
90 };
91
92
93 enum reader_state
94   {
95     STATE_INIT = 0,        /* Initial state */
96     STATE_SPREADSHEET,     /* Found the start of the spreadsheet doc */
97     STATE_TABLE,           /* Found the sheet that we actually want */
98     STATE_ROW,             /* Found the start of the cell array */
99     STATE_CELL,            /* Found a cell */
100     STATE_CELL_CONTENT     /* Found a the text within a cell */
101   };
102
103 struct ods_reader
104 {
105   struct spreadsheet spreadsheet;
106   struct zip_reader *zreader;
107   xmlTextReaderPtr xtr;
108
109   enum reader_state state;
110   int row;
111   int col;
112   int node_type;
113   int current_sheet;
114   xmlChar *current_sheet_name;
115
116   const xmlChar *target_sheet_name;
117   int target_sheet_index;
118
119
120   int start_row;
121   int start_col;
122   int stop_row;
123   int stop_col;
124
125   int col_span;
126
127   struct sheet_detail *sheets;
128   int n_allocated_sheets;
129
130   struct caseproto *proto;
131   struct dictionary *dict;
132   struct ccase *first_case;
133   bool used_first_case;
134   bool read_names;
135
136   struct string ods_errs;
137 };
138
139
140 static bool
141 reading_target_sheet (const struct ods_reader *r)
142 {
143   if (r->target_sheet_name != NULL)
144     {
145       if ( 0 == xmlStrcmp (r->target_sheet_name, r->current_sheet_name))
146         return true;
147     }
148   
149   if (r->target_sheet_index == r->current_sheet + 1)
150     return true;
151
152   return false;
153 }
154
155
156 static void process_node (struct ods_reader *r);
157
158
159 const char *
160 ods_get_sheet_name (struct spreadsheet *s, int n)
161 {
162   struct ods_reader *or = (struct ods_reader *) s;
163   
164   assert (n < s->n_sheets);
165
166   while ( 
167           (or->n_allocated_sheets <= n)
168           || or->state != STATE_SPREADSHEET
169           )
170     {
171       int ret = xmlTextReaderRead (or->xtr);
172       if ( ret != 1)
173         break;
174
175       process_node (or);
176     }
177
178   return or->sheets[n].name;
179 }
180
181 char *
182 ods_get_sheet_range (struct spreadsheet *s, int n)
183 {
184   struct ods_reader *or = (struct ods_reader *) s;
185   
186   assert (n < s->n_sheets);
187
188   while ( 
189           (or->n_allocated_sheets <= n)
190           || (or->sheets[n].stop_row == -1) 
191           || or->state != STATE_SPREADSHEET
192           )
193     {
194       int ret = xmlTextReaderRead (or->xtr);
195       if ( ret != 1)
196         break;
197
198       process_node (or);
199     }
200
201   return create_cell_ref (
202                           or->sheets[n].start_col,
203                           or->sheets[n].start_row,
204                           or->sheets[n].stop_col,
205                           or->sheets[n].stop_row);
206 }
207
208
209 static void
210 ods_file_casereader_destroy (struct casereader *reader UNUSED, void *r_)
211 {
212   struct ods_reader *r = r_;
213   if ( r == NULL)
214     return ;
215
216   if (r->xtr)
217     xmlFreeTextReader (r->xtr);
218   r->xtr = NULL;
219
220   if ( ! ds_is_empty (&r->ods_errs))
221     msg (ME, "%s", ds_cstr (&r->ods_errs));
222
223   ds_destroy (&r->ods_errs);
224
225   if ( ! r->used_first_case )
226     case_unref (r->first_case);
227
228   caseproto_unref (r->proto);
229
230   //  free (r);
231 }
232
233 static void
234 process_node (struct ods_reader *r)
235 {
236   xmlChar *name = xmlTextReaderName (r->xtr);
237   if (name == NULL)
238     name = xmlStrdup (_xml ("--"));
239
240
241   r->node_type = xmlTextReaderNodeType (r->xtr);
242
243   switch (r->state)
244     {
245     case STATE_INIT:
246       if (0 == xmlStrcasecmp (name, _xml("office:spreadsheet")) &&
247           XML_READER_TYPE_ELEMENT  == r->node_type)
248         {
249           r->state = STATE_SPREADSHEET;
250           r->current_sheet = -1;
251           r->current_sheet_name = NULL;
252         }
253       break;
254     case STATE_SPREADSHEET:
255       if (0 == xmlStrcasecmp (name, _xml("table:table"))
256           && 
257           (XML_READER_TYPE_ELEMENT == r->node_type))
258         {
259           xmlFree (r->current_sheet_name);
260           r->current_sheet_name = xmlTextReaderGetAttribute (r->xtr, _xml ("table:name"));
261
262           ++r->current_sheet;
263
264           if (r->current_sheet >= r->n_allocated_sheets)
265             {
266               assert (r->current_sheet == r->n_allocated_sheets);
267               r->sheets = xrealloc (r->sheets, sizeof (*r->sheets) * ++r->n_allocated_sheets);
268               r->sheets[r->n_allocated_sheets - 1].start_col = -1;
269               r->sheets[r->n_allocated_sheets - 1].stop_col = -1;
270               r->sheets[r->n_allocated_sheets - 1].start_row = -1;
271               r->sheets[r->n_allocated_sheets - 1].stop_row = -1;
272               r->sheets[r->n_allocated_sheets - 1].name = CHAR_CAST (char *, xmlStrdup (r->current_sheet_name));
273             }
274
275           r->col = 0;
276           r->row = 0;
277
278           r->state = STATE_TABLE;
279         }
280       else if (0 == xmlStrcasecmp (name, _xml("office:spreadsheet")) &&
281                XML_READER_TYPE_ELEMENT  == r->node_type)
282         {
283           r->state = STATE_INIT;
284         }
285       break;
286     case STATE_TABLE:
287       if (0 == xmlStrcasecmp (name, _xml("table:table-row")) && 
288           (XML_READER_TYPE_ELEMENT  == r->node_type))
289         {
290           xmlChar *value =
291             xmlTextReaderGetAttribute (r->xtr,
292                                        _xml ("table:number-rows-repeated"));
293           
294           int row_span = value ? _xmlchar_to_int (value) : 1;
295
296           r->row += row_span;
297           r->col = 0;
298           
299           if (! xmlTextReaderIsEmptyElement (r->xtr))
300             r->state = STATE_ROW;
301         }
302       else if (0 == xmlStrcasecmp (name, _xml("table:table")) && 
303                (XML_READER_TYPE_END_ELEMENT  == r->node_type))
304         {
305           r->state = STATE_SPREADSHEET;
306         }
307       break;
308     case STATE_ROW:
309       if ( (0 == xmlStrcasecmp (name, _xml ("table:table-cell")))
310            && 
311            (XML_READER_TYPE_ELEMENT  == r->node_type))
312         {
313           xmlChar *value =
314             xmlTextReaderGetAttribute (r->xtr,
315                                        _xml ("table:number-columns-repeated"));
316           
317           r->col_span = value ? _xmlchar_to_int (value) : 1;
318           r->col += r->col_span;
319
320           if (! xmlTextReaderIsEmptyElement (r->xtr))
321             r->state = STATE_CELL;
322
323           xmlFree (value);
324         }
325       else if ( (0 == xmlStrcasecmp (name, _xml ("table:table-row")))
326                 &&
327                 (XML_READER_TYPE_END_ELEMENT  == r->node_type))
328         {
329           r->state = STATE_TABLE;
330         }
331       break;
332     case STATE_CELL:
333       if ( (0 == xmlStrcasecmp (name, _xml("text:p")))
334             &&
335            ( XML_READER_TYPE_ELEMENT  == r->node_type))
336         {
337           if (! xmlTextReaderIsEmptyElement (r->xtr))
338             r->state = STATE_CELL_CONTENT;
339         }
340       else if
341         ( (0 == xmlStrcasecmp (name, _xml("table:table-cell")))
342           &&
343           (XML_READER_TYPE_END_ELEMENT  == r->node_type)
344           )
345         {
346           r->state = STATE_ROW;
347         }
348       break;
349     case STATE_CELL_CONTENT:
350       assert (r->current_sheet >= 0);
351       assert (r->current_sheet < r->n_allocated_sheets);
352
353       if (r->sheets[r->current_sheet].start_row == -1)
354         r->sheets[r->current_sheet].start_row = r->row - 1;
355
356       if ( 
357           (r->sheets[r->current_sheet].start_col == -1)
358           ||
359           (r->sheets[r->current_sheet].start_col >= r->col - 1)
360            )
361         r->sheets[r->current_sheet].start_col = r->col - 1;
362
363       r->sheets[r->current_sheet].stop_row = r->row - 1;
364
365       if ( r->sheets[r->current_sheet].stop_col <  r->col - 1)
366         r->sheets[r->current_sheet].stop_col = r->col - 1;
367
368       if (XML_READER_TYPE_END_ELEMENT  == r->node_type)
369         r->state = STATE_CELL;
370       break;
371     default:
372       NOT_REACHED ();
373       break;
374     };
375
376   xmlFree (name);
377 }
378
379 /* 
380    A struct containing the parameters of a cell's value 
381    parsed from the xml
382 */
383 struct xml_value
384 {
385   xmlChar *type;
386   xmlChar *value;
387   xmlChar *text;
388 };
389
390 struct var_spec
391 {
392   char *name;
393   struct xml_value firstval;
394 };
395
396
397 /* Determine the width that a xmv should probably have */
398 static int
399 xmv_to_width (const struct xml_value *xmv, int fallback)
400 {
401   int width = SPREADSHEET_DEFAULT_WIDTH;
402
403   /* Non-strings always have zero width */
404   if (xmv->type != NULL && 0 != xmlStrcmp (xmv->type, _xml("string")))
405     return 0;
406
407   if ( fallback != -1)
408     return fallback;
409
410   if ( xmv->value )
411     width = ROUND_UP (xmlStrlen (xmv->value),
412                       SPREADSHEET_DEFAULT_WIDTH);
413   else if ( xmv->text)
414     width = ROUND_UP (xmlStrlen (xmv->text),
415                       SPREADSHEET_DEFAULT_WIDTH);
416
417   return width;
418 }
419
420 /*
421    Sets the VAR of case C, to the value corresponding to the xml data
422  */
423 static void
424 convert_xml_to_value (struct ccase *c, const struct variable *var,
425                       const struct xml_value *xmv)
426 {
427   union value *v = case_data_rw (c, var);
428
429   if (xmv->value == NULL && xmv->text == NULL)
430     value_set_missing (v, var_get_width (var));
431   else if ( var_is_alpha (var))
432     /* Use the text field, because it seems that there is no
433        value field for strings */
434     value_copy_str_rpad (v, var_get_width (var), xmv->text, ' ');
435   else
436     {
437       const struct fmt_spec *fmt = var_get_write_format (var);
438       enum fmt_category fc  = fmt_get_category (fmt->type);
439
440       assert ( fc != FMT_CAT_STRING);
441
442       if ( 0 == xmlStrcmp (xmv->type, _xml("float")))
443         {
444           v->f = c_strtod (CHAR_CAST (const char *, xmv->value), NULL);
445         }
446       else
447         {
448           const char *text = xmv->value ?
449             CHAR_CAST (const char *, xmv->value) : CHAR_CAST (const char *, xmv->text);
450
451
452           free (data_in (ss_cstr (text), "UTF-8",
453                          fmt->type,
454                          v,
455                          var_get_width (var),
456                          "UTF-8"));
457         }
458     }
459 }
460
461
462 /* Try to find out how many sheets there are in the "workbook" */
463 static int
464 get_sheet_count (struct zip_reader *zreader)
465 {
466   xmlTextReaderPtr mxtr;
467   struct zip_member *meta = NULL;
468   meta = zip_member_open (zreader, "meta.xml");
469
470   if ( meta == NULL)
471     return -1;
472
473   mxtr = xmlReaderForIO ((xmlInputReadCallback) zip_member_read,
474                          (xmlInputCloseCallback) zip_member_finish,
475                          meta,   NULL, NULL, 0);
476
477   while (1 == xmlTextReaderRead (mxtr))
478     {
479       xmlChar *name = xmlTextReaderName (mxtr);
480       if ( 0 == xmlStrcmp (name, _xml("meta:document-statistic")))
481         {
482           xmlChar *attr = xmlTextReaderGetAttribute (mxtr, _xml ("meta:table-count"));
483
484           if ( attr != NULL)
485             {
486               int s = _xmlchar_to_int (attr);
487               xmlFreeTextReader (mxtr);
488               xmlFree (name);
489               xmlFree (attr);      
490               return s;
491             }
492           xmlFree (attr);      
493         }
494       xmlFree (name);      
495     }
496
497   xmlFreeTextReader (mxtr);
498   return -1;
499 }
500
501 static void
502 ods_error_handler (void *ctx, const char *mesg,
503                         UNUSED xmlParserSeverities sev, xmlTextReaderLocatorPtr loc)
504 {
505   struct ods_reader *r = ctx;
506        
507   msg (MW, _("There was a problem whilst reading the %s file `%s' (near line %d): `%s'"),
508        "ODF",
509        r->spreadsheet.file_name,
510        xmlTextReaderLocatorLineNumber (loc),
511        mesg);
512 }
513
514
515 static bool
516 init_reader (struct ods_reader *r, bool report_errors)
517 {
518   struct zip_member *content = zip_member_open (r->zreader, "content.xml");
519   xmlTextReaderPtr xtr;
520
521   if ( content == NULL)
522     return false;
523
524   zip_member_ref (content);
525
526   if (r->xtr)
527     xmlFreeTextReader (r->xtr);
528
529   xtr = xmlReaderForIO ((xmlInputReadCallback) zip_member_read,
530                         (xmlInputCloseCallback) zip_member_finish,
531                         content,   NULL, NULL,
532                         report_errors ? 0 : (XML_PARSE_NOERROR | XML_PARSE_NOWARNING) );
533
534   if ( xtr == NULL)
535     return false;
536
537   r->xtr = xtr;
538   r->spreadsheet.type = SPREADSHEET_ODS;
539   r->row = 0;
540   r->col = 0;
541   r->current_sheet = 0;
542   r->state = STATE_INIT;
543
544   if (report_errors) 
545     xmlTextReaderSetErrorHandler (xtr, ods_error_handler, r);
546
547   return true;
548 }
549
550
551 struct spreadsheet *
552 ods_probe (const char *filename, bool report_errors)
553 {
554   struct ods_reader *r;
555   struct string errs = DS_EMPTY_INITIALIZER;
556   int sheet_count;
557   struct zip_reader *zr = zip_reader_create (filename, &errs);
558
559   if (zr == NULL)
560     {
561       if (report_errors)
562         {
563           msg (ME, _("Cannot open %s as a OpenDocument file: %s"),
564                filename, ds_cstr (&errs));
565         }
566       return NULL;
567     }
568
569   sheet_count = get_sheet_count (zr);
570
571   r = xzalloc (sizeof *r);
572   r->zreader = zr;
573
574   if (! init_reader (r, report_errors))
575     {
576       goto error;
577     }
578
579   r->spreadsheet.n_sheets = sheet_count;
580   r->n_allocated_sheets = 0;
581   r->sheets = NULL;
582
583   ds_destroy (&errs);
584
585   r->spreadsheet.file_name = filename;
586   return &r->spreadsheet;
587
588  error:
589   zip_reader_destroy (r->zreader);
590   ds_destroy (&errs);
591   free (r);
592   return NULL;
593 }
594
595 struct casereader *
596 ods_make_reader (struct spreadsheet *spreadsheet, 
597                  const struct spreadsheet_read_options *opts)
598 {
599   intf ret = 0;
600   xmlChar *type = NULL;
601   unsigned long int vstart = 0;
602   casenumber n_cases = CASENUMBER_MAX;
603   int i;
604   struct var_spec *var_spec = NULL;
605   int n_var_specs = 0;
606
607   struct ods_reader *r = (struct ods_reader *) spreadsheet;
608   xmlChar *val_string = NULL;
609
610   assert (r);
611   r->read_names = opts->read_names;
612   ds_init_empty (&r->ods_errs);
613
614
615   if ( !init_reader (r, true))
616     goto error;
617
618   if ( opts->cell_range )
619     {
620       if ( ! convert_cell_ref (opts->cell_range,
621                                &r->start_col, &r->start_row,
622                                &r->stop_col, &r->stop_row))
623         {
624           msg (SE, _("Invalid cell range `%s'"),
625                opts->cell_range);
626           goto error;
627         }
628     }
629   else
630     {
631       r->start_col = 0;
632       r->start_row = 0;
633       r->stop_col = -1;
634       r->stop_row = -1;
635     }
636
637   r->state = STATE_INIT;
638   r->target_sheet_name = BAD_CAST opts->sheet_name;
639   r->target_sheet_index = opts->sheet_index;
640   r->row = r->col = 0;
641
642
643 #if 0
644   printf ("%s:%d %d,%d %d,%d\n", __FILE__, __LINE__, 
645           r->start_col,
646           r->start_row,
647           r->stop_col,
648           r->stop_row);
649 #endif
650
651
652   /* Advance to the start of the cells for the target sheet */
653   while ( ! reading_target_sheet (r)  
654           || r->state != STATE_ROW || r->row <= r->start_row )
655     {
656       if (1 != (ret = xmlTextReaderRead (r->xtr)))
657            break;
658
659       process_node (r);
660     }
661
662   if (ret < 1)
663     {
664       msg (MW, _("Selected sheet or range of spreadsheet `%s' is empty."),
665            spreadsheet->file_name);
666       goto error;
667     }
668
669   if ( opts->read_names)
670     {
671       while (1 == (ret = xmlTextReaderRead (r->xtr)))
672         {
673           int idx;
674
675           process_node (r);
676
677           /* If the row is finished then stop for now */
678           if (r->state == STATE_TABLE && r->row > r->start_row)
679             break;
680
681           idx = r->col - r->start_col -1 ;
682
683           if ( idx < 0)
684             continue;
685
686           if (r->stop_col != -1 && idx > r->stop_col - r->start_col)
687             continue;
688
689           if (r->state == STATE_CELL_CONTENT 
690               &&
691               XML_READER_TYPE_TEXT  == r->node_type)
692             {
693               xmlChar *value = xmlTextReaderValue (r->xtr);
694
695               if ( idx >= n_var_specs)
696                 {
697                   var_spec = xrealloc (var_spec, sizeof (*var_spec) * (idx + 1));
698
699                   /* xrealloc (unlike realloc) doesn't initialise its memory to 0 */
700                   memset (var_spec + n_var_specs,
701                           0, 
702                           (idx - n_var_specs + 1) * sizeof (*var_spec));
703                   n_var_specs = idx + 1;
704                 }
705               var_spec[idx].firstval.text = 0;
706               var_spec[idx].firstval.value = 0;
707               var_spec[idx].firstval.type = 0;
708
709               var_spec [idx].name = strdup (CHAR_CAST (const char *, value));
710
711               xmlFree (value);
712             }
713         }
714     }
715
716   /* Read in the first row of data */
717   while (1 == xmlTextReaderRead (r->xtr))
718     {
719       int idx;
720       process_node (r);
721
722       if ( ! reading_target_sheet (r) )
723         break;
724
725       /* If the row is finished then stop for now */
726       if (r->state == STATE_TABLE &&
727           r->row > r->start_row + (opts->read_names ? 1 : 0))
728         break;
729
730       idx = r->col - r->start_col - 1;
731       if (idx < 0)
732         continue;
733
734       if (r->stop_col != -1 && idx > r->stop_col - r->start_col)
735         continue;
736
737       if ( r->state == STATE_CELL &&
738            XML_READER_TYPE_ELEMENT  == r->node_type)
739         {
740           type = xmlTextReaderGetAttribute (r->xtr, _xml ("office:value-type"));
741           val_string = xmlTextReaderGetAttribute (r->xtr, _xml ("office:value"));
742         }
743
744       if ( r->state == STATE_CELL_CONTENT &&
745            XML_READER_TYPE_TEXT  == r->node_type)
746         {
747 #if 0
748           printf ("%s:%d Idx %d n_var_specs %d\n", __FILE__, __LINE__,
749                   idx, n_var_specs);
750
751           printf ("%s:%d Idx %d r_col %d\n", __FILE__, __LINE__,
752                   idx, r->col);
753 #endif
754
755           if (idx >= n_var_specs)
756             {
757               var_spec = xrealloc (var_spec, sizeof (*var_spec) * (idx + 1));
758               memset (var_spec + n_var_specs,
759                       0, 
760                       (idx - n_var_specs + 1) * sizeof (*var_spec));
761
762               var_spec [idx].name = NULL;
763               n_var_specs = idx + 1;
764             }
765
766           var_spec [idx].firstval.type = type;
767           var_spec [idx].firstval.text = xmlTextReaderValue (r->xtr);
768           var_spec [idx].firstval.value = val_string;
769
770           val_string = NULL;
771           type = NULL;
772         }
773     }
774
775
776   /* Create the dictionary and populate it */
777   r->spreadsheet.dict = r->dict = dict_create (
778     CHAR_CAST (const char *, xmlTextReaderConstEncoding (r->xtr)));
779
780   for (i = 0; i < n_var_specs ; ++i )
781     {
782       struct fmt_spec fmt;
783       struct variable *var = NULL;
784       char *name = dict_make_unique_var_name (r->dict, var_spec[i].name, &vstart);
785       int width  = xmv_to_width (&var_spec[i].firstval, opts->asw);
786       dict_create_var (r->dict, name, width);
787       free (name);
788
789       var = dict_get_var (r->dict, i);
790
791       if ( 0 == xmlStrcmp (var_spec[i].firstval.type, _xml("date")))
792         {
793           fmt.type = FMT_DATE;
794           fmt.d = 0;
795           fmt.w = 20;
796         }
797       else
798         fmt = fmt_default_for_width (width);
799
800       var_set_both_formats (var, &fmt);
801     }
802
803   /* Create the first case, and cache it */
804   r->used_first_case = false;
805
806   if ( n_var_specs ==  0 )
807     {
808       msg (MW, _("Selected sheet or range of spreadsheet `%s' is empty."),
809            spreadsheet->file_name);
810       goto error;
811     }
812
813   r->proto = caseproto_ref (dict_get_proto (r->dict));
814   r->first_case = case_create (r->proto);
815   case_set_missing (r->first_case);
816
817   for (i = 0 ; i < n_var_specs; ++i)
818     {
819       const struct variable *var = dict_get_var (r->dict, i);
820
821       convert_xml_to_value (r->first_case, var,  &var_spec[i].firstval);
822     }
823
824   /* Read in the first row of data */
825   while (1 == xmlTextReaderRead (r->xtr))
826     {
827       process_node (r);
828
829       if (r->state == STATE_ROW)
830         break;
831     }
832
833   // zip_reader_destroy (zreader);
834
835   for ( i = 0 ; i < n_var_specs ; ++i )
836     {
837       free (var_spec[i].firstval.type);
838       free (var_spec[i].firstval.value);
839       free (var_spec[i].firstval.text);
840       free (var_spec[i].name);
841     }
842
843   free (var_spec);
844
845
846   return casereader_create_sequential
847     (NULL,
848      r->proto,
849      n_cases,
850      &ods_file_casereader_class, r);
851
852  error:
853   
854   //zip_reader_destroy (zreader);
855
856   for ( i = 0 ; i < n_var_specs ; ++i )
857     {
858       free (var_spec[i].firstval.type);
859       free (var_spec[i].firstval.value);
860       free (var_spec[i].firstval.text);
861       free (var_spec[i].name);
862     }
863
864   free (var_spec);
865
866   dict_destroy (r->spreadsheet.dict);
867   r->spreadsheet.dict = NULL;
868   ods_file_casereader_destroy (NULL, r);
869
870
871   return NULL;
872 }
873
874
875 /* Reads and returns one case from READER's file.  Returns a null
876    pointer on failure. */
877 static struct ccase *
878 ods_file_casereader_read (struct casereader *reader UNUSED, void *r_)
879 {
880   struct ccase *c = NULL;
881   xmlChar *val_string = NULL;
882   xmlChar *type = NULL;
883   struct ods_reader *r = r_;
884
885   if (!r->used_first_case)
886     {
887       r->used_first_case = true;
888       return r->first_case;
889     }
890
891
892   /* Advance to the start of a row. (If there is one) */
893   while (r->state != STATE_ROW 
894          && 1 == xmlTextReaderRead (r->xtr)
895          )
896     {
897       process_node (r);
898     }
899
900
901   if ( ! reading_target_sheet (r)  
902        ||  r->state < STATE_TABLE
903        ||  (r->stop_row != -1 && r->row > r->stop_row + 1)
904        )
905     {
906       return NULL;
907     }
908
909   c = case_create (r->proto);
910   case_set_missing (c);
911   
912   while (1 == xmlTextReaderRead (r->xtr))
913     {
914       process_node (r);
915
916       if ( r->stop_row != -1 && r->row > r->stop_row + 1)
917         break;
918
919       if (r->state == STATE_CELL &&
920            r->node_type == XML_READER_TYPE_ELEMENT)
921         {
922           type = xmlTextReaderGetAttribute (r->xtr, _xml ("office:value-type"));
923           val_string = xmlTextReaderGetAttribute (r->xtr, _xml ("office:value"));
924         }
925
926       if (r->state == STATE_CELL_CONTENT && 
927            r->node_type == XML_READER_TYPE_TEXT)
928         {
929           int col;
930           struct xml_value *xmv = xzalloc (sizeof *xmv);
931           xmv->text = xmlTextReaderValue (r->xtr);
932           xmv->value = val_string;       
933           xmv->type = type;
934           val_string = NULL;
935
936           for (col = 0; col < r->col_span; ++col)
937             {
938               const struct variable *var;
939               const int idx = r->col - col - r->start_col - 1;
940               if (idx < 0)
941                 continue;
942               if (r->stop_col != -1 && idx > r->stop_col - r->start_col )
943                 break;
944
945               var = dict_get_var (r->dict, idx);
946               convert_xml_to_value (c, var, xmv);
947             }
948
949           xmlFree (xmv->text);
950           xmlFree (xmv->value);
951           xmlFree (xmv->type);
952           free (xmv);
953         }
954       if ( r->state <= STATE_TABLE)
955         break;
956     }
957
958   return c;
959 }
960 #endif