added case for german languaged Worksheets #32

Closed
OliverZ wants to merge 0 commits from gh-pages into gh-pages
OliverZ commented 2013-08-28 19:39:49 +00:00 (Migrated from github.com)

added case for german languaged Worksheets to not log "Unrecognized key in Heading Pairs" error to console.

added case for german languaged Worksheets to not log "Unrecognized key in Heading Pairs" error to console.
Niggler commented 2013-08-28 20:25:19 +00:00 (Migrated from github.com)

Can you share a file from a german version of Excel? A blank file is fine, I just want one that shows the German case.

Also, can you also modify the master branch as well? Add the same case in xlsx.js and bits/70_xlsx.js

Can you share a file from a german version of Excel? A blank file is fine, I just want one that shows the German case. Also, can you also modify the master branch as well? Add the same case in `xlsx.js` and `bits/70_xlsx.js`
OliverZ commented 2013-08-31 10:17:25 +00:00 (Migrated from github.com)

i added a testfile here

https://github.com/OliverZ/js-xlsx/blob/master/tests/test.xlsx

if you run

var XLSX = require('xlsx')
var xlsx = XLSX.readFile('test.xlsx');
var sheet_name_list = xlsx.SheetNames;
xlsx.SheetNames.forEach(function(y) {
  for (z in xlsx.Sheets[y]) {
    if(z[0] === '!') continue;
    console.log(y + "!" + z + "=" + JSON.stringify(xlsx.Sheets[y][z].v));
  }
});

it will print: Unrecognized key in Heading Pairs

i added a testfile here https://github.com/OliverZ/js-xlsx/blob/master/tests/test.xlsx if you run ``` var XLSX = require('xlsx') var xlsx = XLSX.readFile('test.xlsx'); var sheet_name_list = xlsx.SheetNames; xlsx.SheetNames.forEach(function(y) { for (z in xlsx.Sheets[y]) { if(z[0] === '!') continue; console.log(y + "!" + z + "=" + JSON.stringify(xlsx.Sheets[y][z].v)); } }); ``` it will print: Unrecognized key in Heading Pairs
Niggler commented 2013-10-29 21:01:38 +00:00 (Migrated from github.com)

I turned off that error message, as it appears to work properly. I'm trying to find a comprehensive list of the possible words, but that is proving hard to find.

I turned off that error message, as it appears to work properly. I'm trying to find a comprehensive list of the possible words, but that is proving hard to find.

Pull request closed

Sign in to join this conversation.
No description provided.