Skip to content

Commit

Permalink
Case insensitive table/column names (AlaSQL#909)
Browse files Browse the repository at this point in the history
* 1) yy.casesensitive option doesn't update after changing value of alasql.options.casesensitive. Therefore, used alasql.options.casesensitive in alasqlparser.js
2) Now, as alasqlparser file when casesensitive options set to false sets all the column names to lowercase therefore in 84form.js file we are changing all column names to lowercase. This way now whatever case whether upper, lower or mixed case column names are specified in excel then also alasql can read data for all those columns when alasql.options.casesensitive is set to false.

* 1) removed if?then:else, restructured the code and added comments
2) added test cases for Mixed Case, Upper Case and Lower Case. Both positive and negative tests are used
3) added files generated at build time in the commit
  • Loading branch information
amitauras authored and mathiasrw committed Jul 20, 2017
1 parent d16eaf4 commit af32605
Show file tree
Hide file tree
Showing 10 changed files with 22,596 additions and 22,507 deletions.
500 changes: 250 additions & 250 deletions dist/alasql-worker.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/alasql-worker.min.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit af32605

Please sign in to comment.