fast_excel 0.2.1 → 0.3.0

data/libxlsxwriter/src/xmlwriter.c

@@ -3,7 +3,7 @@
  *
  * Used in conjunction with the libxlsxwriter library.
  *
- * Copyright 2014-2017, John McNamara, jmcnamara@cpan.org. See LICENSE.txt.
+ * Copyright 2014-2019, John McNamara, jmcnamara@cpan.org. See LICENSE.txt.
  *
  */
 
@@ -140,6 +140,15 @@ lxw_xml_data_element(FILE * xmlfile,
     fprintf(xmlfile, "</%s>", tag);
 }
 
+/*
+ * Write an XML <si> element for rich strings, without encoding.
+ */
+void
+lxw_xml_rich_si_element(FILE * xmlfile, const char *string)
+{
+    fprintf(xmlfile, "<si>%s</si>", string);
+}
+
 /*
  * Escape XML characters in attributes.
  */
@@ -153,19 +162,19 @@ _escape_attributes(struct xml_attribute *attribute)
     while (*p_attr) {
         switch (*p_attr) {
             case '&':
-                strncat(p_encoded, LXW_AMP, sizeof(LXW_AMP) - 1);
+                memcpy(p_encoded, LXW_AMP, sizeof(LXW_AMP) - 1);
                 p_encoded += sizeof(LXW_AMP) - 1;
                 break;
             case '<':
-                strncat(p_encoded, LXW_LT, sizeof(LXW_LT) - 1);
+                memcpy(p_encoded, LXW_LT, sizeof(LXW_LT) - 1);
                 p_encoded += sizeof(LXW_LT) - 1;
                 break;
             case '>':
-                strncat(p_encoded, LXW_GT, sizeof(LXW_GT) - 1);
+                memcpy(p_encoded, LXW_GT, sizeof(LXW_GT) - 1);
                 p_encoded += sizeof(LXW_GT) - 1;
                 break;
             case '"':
-                strncat(p_encoded, LXW_QUOT, sizeof(LXW_QUOT) - 1);
+                memcpy(p_encoded, LXW_QUOT, sizeof(LXW_QUOT) - 1);
                 p_encoded += sizeof(LXW_QUOT) - 1;
                 break;
             default:
@@ -195,15 +204,15 @@ lxw_escape_data(const char *data)
     while (*data) {
         switch (*data) {
             case '&':
-                strncat(p_encoded, LXW_AMP, sizeof(LXW_AMP) - 1);
+                memcpy(p_encoded, LXW_AMP, sizeof(LXW_AMP) - 1);
                 p_encoded += sizeof(LXW_AMP) - 1;
                 break;
             case '<':
-                strncat(p_encoded, LXW_LT, sizeof(LXW_LT) - 1);
+                memcpy(p_encoded, LXW_LT, sizeof(LXW_LT) - 1);
                 p_encoded += sizeof(LXW_LT) - 1;
                 break;
             case '>':
-                strncat(p_encoded, LXW_GT, sizeof(LXW_GT) - 1);
+                memcpy(p_encoded, LXW_GT, sizeof(LXW_GT) - 1);
                 p_encoded += sizeof(LXW_GT) - 1;
                 break;
             default:
@@ -349,7 +358,7 @@ lxw_new_attribute_dbl(const char *key, double value)
     struct xml_attribute *attribute = malloc(sizeof(struct xml_attribute));
 
     LXW_ATTRIBUTE_COPY(attribute->key, key);
-    lxw_snprintf(attribute->value, LXW_MAX_ATTRIBUTE_LENGTH, "%.16g", value);
+    lxw_sprintf_dbl(attribute->value, value);
 
     return attribute;
 }

data/libxlsxwriter/third_party/minizip/Makefile

@@ -9,7 +9,7 @@ ifdef V
 Q=
 endif
 
-UNAME := $(shell uname)
+UNAME := $(shell uname -sr)
 
 # Check for MinGW/MinGW64/Cygwin environments.
 ifneq (,$(findstring MINGW, $(UNAME)))
@@ -32,6 +32,11 @@ CC = gcc
 CFLAGS += -DUSE_FILE32API
 endif
 
+# Fix for modified zconf.h on Gentoo.
+ifneq (,$(findstring gentoo, $(UNAME)))
+CFLAGS += -DOF=_Z_OF
+endif
+
 all: ioapi.o zip.o  ioapi.so zip.so
 
 %.o  : %.c

data/libxlsxwriter/third_party/minizip/ioapi.c

@@ -96,6 +96,7 @@ static voidpf ZCALLBACK fopen_file_func (voidpf opaque, const char* filename, in
 {
     FILE* file = NULL;
     const char* mode_fopen = NULL;
+    (void) opaque;
     if ((mode & ZLIB_FILEFUNC_MODE_READWRITEFILTER)==ZLIB_FILEFUNC_MODE_READ)
         mode_fopen = "rb";
     else
@@ -114,6 +115,7 @@ static voidpf ZCALLBACK fopen64_file_func (voidpf opaque, const void* filename,
 {
     FILE* file = NULL;
     const char* mode_fopen = NULL;
+    (void) opaque;
     if ((mode & ZLIB_FILEFUNC_MODE_READWRITEFILTER)==ZLIB_FILEFUNC_MODE_READ)
         mode_fopen = "rb";
     else
@@ -132,6 +134,7 @@ static voidpf ZCALLBACK fopen64_file_func (voidpf opaque, const void* filename,
 static uLong ZCALLBACK fread_file_func (voidpf opaque, voidpf stream, void* buf, uLong size)
 {
     uLong ret;
+    (void) opaque;
     ret = (uLong)fread(buf, 1, (size_t)size, (FILE *)stream);
     return ret;
 }
@@ -139,6 +142,7 @@ static uLong ZCALLBACK fread_file_func (voidpf opaque, voidpf stream, void* buf,
 static uLong ZCALLBACK fwrite_file_func (voidpf opaque, voidpf stream, const void* buf, uLong size)
 {
     uLong ret;
+    (void) opaque;
     ret = (uLong)fwrite(buf, 1, (size_t)size, (FILE *)stream);
     return ret;
 }
@@ -146,6 +150,7 @@ static uLong ZCALLBACK fwrite_file_func (voidpf opaque, voidpf stream, const voi
 static long ZCALLBACK ftell_file_func (voidpf opaque, voidpf stream)
 {
     long ret;
+    (void) opaque;
     ret = ftell((FILE *)stream);
     return ret;
 }
@@ -154,6 +159,7 @@ static long ZCALLBACK ftell_file_func (voidpf opaque, voidpf stream)
 static ZPOS64_T ZCALLBACK ftell64_file_func (voidpf opaque, voidpf stream)
 {
     ZPOS64_T ret;
+    (void) opaque;
     ret = FTELLO_FUNC((FILE *)stream);
     return ret;
 }
@@ -162,6 +168,7 @@ static long ZCALLBACK fseek_file_func (voidpf  opaque, voidpf stream, uLong offs
 {
     int fseek_origin=0;
     long ret;
+    (void) opaque;
     switch (origin)
     {
     case ZLIB_FILEFUNC_SEEK_CUR :
@@ -185,6 +192,7 @@ static long ZCALLBACK fseek64_file_func (voidpf  opaque, voidpf stream, ZPOS64_T
 {
     int fseek_origin=0;
     long ret;
+    (void) opaque;
     switch (origin)
     {
     case ZLIB_FILEFUNC_SEEK_CUR :
@@ -210,6 +218,7 @@ static long ZCALLBACK fseek64_file_func (voidpf  opaque, voidpf stream, ZPOS64_T
 static int ZCALLBACK fclose_file_func (voidpf opaque, voidpf stream)
 {
     int ret;
+    (void) opaque;
     ret = fclose((FILE *)stream);
     return ret;
 }
@@ -217,6 +226,7 @@ static int ZCALLBACK fclose_file_func (voidpf opaque, voidpf stream)
 static int ZCALLBACK ferror_file_func (voidpf opaque, voidpf stream)
 {
     int ret;
+    (void) opaque;
     ret = ferror((FILE *)stream);
     return ret;
 }

data/libxlsxwriter/third_party/minizip/zip.c

@@ -519,12 +519,14 @@ local ZPOS64_T zip64local_SearchCentralDir(const zlib_filefunc64_32_def* pzlib_f
       break;
 
     for (i=(int)uReadSize-3; (i--)>0;)
+    {
       if (((*(buf+i))==0x50) && ((*(buf+i+1))==0x4b) &&
         ((*(buf+i+2))==0x05) && ((*(buf+i+3))==0x06))
       {
         uPosFound = uReadPos+i;
         break;
       }
+    }
 
       if (uPosFound!=0)
         break;

data/libxlsxwriter/third_party/tmpfileplus/tmpfileplus.c

@@ -169,7 +169,7 @@ static char *getenv_save(const char *varname, char *buf, size_t bufsize)
 	buf[0] = '\0';
 	if (ptr)
 	{
-		strncpy(buf, ptr, bufsize);
+		strncpy(buf, ptr, bufsize-1);
 		buf[bufsize-1] = '\0';
 		return buf;
 	}
@@ -189,7 +189,7 @@ static FILE *mktempfile_internal(const char *tmpdir, const char *pfx, char **tmp
  */
 {
 	FILE *fp;
-	int fd;
+	int fd = 0;
 	char randpart[] = "1234567890";
 	size_t lentempname;
 	int i;

data/libxlsxwriter/version.txt

@@ -1 +1 @@
-2017-02-28 20:35:27 +0800 c8064ee
+2019-06-17 01:19:43 +0800 56b73bb

data/test/auto_width_test.rb

@@ -0,0 +1,19 @@
+require_relative 'test_helper'
+
+describe "FastExcel text_width" do
+
+  it "should calculate text width" do
+    workbook = FastExcel.open(constant_memory: false)
+    sheet = workbook.add_worksheet
+    sheet.auto_width = true
+
+    sheet.append_row([
+      "tini",
+      "Longer",
+      "Some longer text!",
+      "This gem is FFI binding for libxlsxwriter C library"
+    ])
+
+    assert_equal(sheet.calculated_column_widths, {0 => 3.52, 1 => 5.28, 2 => 14.96, 3 => 44.88})
+  end
+end

data/test/date_test.rb

@@ -20,3 +20,37 @@ describe "FastExcel.date_num" do
   end
 
 end
+
+describe "FastExcel.write_value" do
+
+  it "should save correct datetime" do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet
+
+    format = workbook.number_format("yyyy-mm-dd hh:mm:ss")
+    value = DateTime.parse('2017-03-01 15:15:15 +0000')
+
+    worksheet.write_value(0, 0, value, format)
+    workbook.close
+
+    data = parse_xlsx_as_matrix(workbook.filename)
+
+    assert_equal(data[0][0], value)
+  end
+
+  it "should save correct date" do
+    workbook = FastExcel.open(constant_memory: true)
+    worksheet = workbook.add_worksheet
+
+    format = workbook.number_format("yyyy-mm-dd")
+    value = Date.parse('2017-03-01')
+
+    worksheet.write_value(0, 0, value, format)
+    workbook.close
+
+    data = parse_xlsx_as_matrix(workbook.filename)
+
+    assert_equal(data[0][0], value)
+  end
+
+end

data/test/format_test.rb

@@ -0,0 +1,179 @@
+require_relative 'test_helper'
+
+describe "FastExcel::FormatExt align" do
+
+  before do
+    @workbook = FastExcel.open(constant_memory: true)
+    @format = @workbook.add_format
+  end
+
+  it "should give default aligns" do
+    assert_equal({horizontal: :align_none, vertical: :align_none}, @format.align)
+  end
+
+  it "should set align by full name" do
+    @format.align = :align_center
+    assert_equal({horizontal: :align_center, vertical: :align_none}, @format.align)
+  end
+
+  it "should set by string" do
+    @format.align = "align_center"
+    assert_equal({horizontal: :align_center, vertical: :align_none}, @format.align)
+  end
+
+  it "should set by short name" do
+    @format.align = :center
+    assert_equal({horizontal: :align_center, vertical: :align_none}, @format.align)
+  end
+
+  it "should set by hash" do
+    @format.align = {v: "center", h: "center"}
+    assert_equal({horizontal: :align_center, vertical: :align_vertical_center}, @format.align)
+  end
+
+  it "should raise exception for unknown value" do
+    error = assert_raises(ArgumentError) do
+      @format.align = :aaa
+    end
+
+    assert_equal(error.message, "Can not set align = :aaa, possible values are: [:align_none, :align_left, "\
+      ":align_center, :align_right, :align_fill, :align_justify, :align_center_across, :align_distributed, "\
+      ":align_vertical_top, :align_vertical_bottom, :align_vertical_center, :align_vertical_justify, :align_vertical_distributed]")
+  end
+
+  it "should raise exception for unknown hash key" do
+    error = assert_raises(ArgumentError) do
+      @format.align = {aaa: 1}
+    end
+
+    assert_equal(error.message, "Not allowed keys for align: [:aaa], possible keys: [:horizontal, :h, :vertical, :v]")
+  end
+
+  it "should get and set" do
+    @format.align = {h: :center, v: :center}
+    format2 = @workbook.add_format(align: @format.align)
+
+    assert_equal({horizontal: :align_center, vertical: :align_vertical_center}, format2.align)
+  end
+
+end
+
+
+describe "FastExcel::FormatExt colors" do
+
+  before do
+    workbook = FastExcel.open(constant_memory: true)
+    @format = workbook.add_format
+  end
+
+  it "should set font color as hex num" do
+    @format.font_color = 0xFF0000
+    assert_equal(0xFF0000, @format.font_color)
+  end
+
+  it "should set font color as hex string" do
+    @format.font_color = '0xFF0000'
+    assert_equal(0xFF0000, @format.font_color)
+  end
+
+  it "should set font color as css hex string" do
+    @format.font_color = '#FF0000'
+    assert_equal(0xFF0000, @format.font_color)
+  end
+
+  it "should set font color as short hex string" do
+    @format.font_color = 'FF0000'
+    assert_equal(0xFF0000, @format.font_color)
+  end
+
+  it "should set font color as name" do
+    @format.font_color = 'red'
+    assert_equal(0xFF0000, @format.font_color)
+  end
+
+  it "should set font css color" do
+    @format.font_color = 'alice_blue'
+    assert_equal(0xF0F8FF, @format.font_color)
+  end
+
+  it "should allow to use symbol" do
+    @format.font_color = :alice_blue
+    assert_equal(0xF0F8FF, @format.font_color)
+  end
+
+  it "should have long method for border colors" do
+    @format.border_bottom_color = :alice_blue
+    assert_equal(0xF0F8FF, @format.border_bottom_color)
+    assert_equal(0xF0F8FF, @format.bottom_color)
+  end
+
+  it "should raise for unexpected type" do
+    error = assert_raises(ArgumentError) do
+      @format.font_color = {aaa: 1}
+    end
+
+    assert_equal(error.message, "Can not use Hash ({:aaa=>1}) for color value, expected String or Hex Number")
+  end
+
+  it "should raise for unexpected color" do
+    error = assert_raises(ArgumentError) do
+      @format.font_color = :aaa
+    end
+
+    assert_equal(error.message, "Unknown color value :aaa, expected hex string or color name")
+  end
+
+end
+
+
+describe "FastExcel::FormatExt border" do
+
+  before do
+    workbook = FastExcel.open(constant_memory: true)
+    @format = workbook.add_format
+  end
+
+  it "should set border as symbol" do
+    @format.bottom = :border_thin
+    assert_equal(:border_thin, @format.bottom)
+  end
+
+  it "should set border as short symbol" do
+    @format.bottom = :thin
+    assert_equal(:border_thin, @format.bottom)
+  end
+
+  it "should set border as string" do
+    @format.bottom = "thin"
+    assert_equal(:border_thin, @format.bottom)
+  end
+
+  it "should set border as number" do
+    @format.bottom = 1
+    assert_equal(:border_thin, @format.bottom)
+  end
+
+  it "should set border with long prop name" do
+    error = assert_raises(ArgumentError) do
+      @format.border_bottom = :aaa
+    end
+
+    assert_equal(error.message, "Unknown value :aaa for border. Possible values: "\
+      "[:none, :thin, :medium, :dashed, :dotted, :thick, :double, :hair, :medium_dashed, "\
+      ":dash_dot, :medium_dash_dot, :dash_dot_dot, :medium_dash_dot_dot, :slant_dash_dot]")
+  end
+
+  it "should get value with long name" do
+    @format.bottom = "thin"
+    assert_equal(:border_thin, @format.border_bottom)
+  end
+
+  it "should define aliases" do
+    @format.font_size = 20
+    assert_equal(@format.font_size, 20)
+
+    @format.font_name = "XXX"
+    assert_equal(@format.font_name, "XXX")
+  end
+
+end