Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixed issue when CSV data wasn't properly escaped in HTML output #60

Merged
merged 8 commits into from
Jul 14, 2018
Merged

Fixed issue when CSV data wasn't properly escaped in HTML output #60

merged 8 commits into from
Jul 14, 2018

Commits on Apr 20, 2018

  1. Reformat code to jQuery style and add missing semicolons

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    539b927 View commit details
    Browse the repository at this point in the history

  2. Rename variables to camelCase as required by jQuery code style. Add m… 

    …issing var
    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    1f44758 View commit details
    Browse the repository at this point in the history

  3. Extract tableBody as variable with jQuery object

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    427606a View commit details
    Browse the repository at this point in the history

  4. Extract $containerElement as variable with jQuery object

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    5f6f4fd View commit details
    Browse the repository at this point in the history

  5. Use jQuery text() method to safely render table head with HTML escaping

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    34393bb View commit details
    Browse the repository at this point in the history

  6. Instead of changing CSV data value with its formatted value now we ma… 

    …king formatting just in time of rendering so original csvData variable is unchanged.

 To make it easier to determine template function corresponding to cell index we convert array of tuples into usual JS object (i.e. map/dictionary)
    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    6076437 View commit details
    Browse the repository at this point in the history

  7. Use jQuery text() method to safely render table rows with HTML escaping

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    77ec839 View commit details
    Browse the repository at this point in the history

  8. Added important notes to README.md about HTML escaping

    sponomarev committed Apr 20, 2018
    Configuration menu
    Copy the full SHA
    255a6e6 View commit details
    Browse the repository at this point in the history