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.

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

Error when using csv_row() - not designed for number inputs, updated to support now #841

Closed
ronisc opened this issue Jul 9, 2020 · 3 comments
Labels

Comments

@ronisc
Copy link

ronisc commented Jul 9, 2020

When trying to run the following:

name = "dog"
price = 10 
details = "small dog"
write `csv_row([name, price, details])` to product_list.csv

Getting an error of:
ERROR - undefined is not a constructor (evaluating 'cell_data.replace(/"/g, '""')')

@ronisc
Copy link
Author

ronisc commented Jul 9, 2020

In the meantime, I have managed to make it work by avoiding the use of csv_row().

name = "dog"
price = 10 
details = "small dog"
tempArray = [name, price, details]

write `tempArray` to product_list.csv

Given the above, I am not sure:

  1. What does the csv_row() do, and how it is helping?
  2. Why it is not working on my original questions. I am using it exactly the way it is present in the documentation.

@kensoh
Copy link
Member

kensoh commented Nov 28, 2020

Sorry @ronisc I was away the past few months - tebelorg/RPA-Python#144

I see, csv_row() helps to handle special situations (for eg single quotes in inputs, double quotes in inputs), and formats an array of variables into a string, to be ready to write to a text file. It is not working in your examples above because it is not designed to take in numbers as inputs. The documentation examples are reading text inputs from a webpage. Any data read from a webpage using read step is a string, even though it may be a string representing a number.

I'm considering this original design to not handle number inputs as a bug and have updated code to fix this. You can download the latest copy of the code from here and unzip to overwrite your existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip

In the next release, this will become part of the packaged zip files. Thanks for raising this!

Closing issue for now but please lemme know if you have any input or question.

@kensoh kensoh closed this as completed Nov 28, 2020
@kensoh kensoh changed the title Getting an error when trying to use csv_row() with local variables Getting an error when trying to use csv_row() - not designed for number inputs, updated to support now Nov 28, 2020
@kensoh kensoh changed the title Getting an error when trying to use csv_row() - not designed for number inputs, updated to support now Error when using csv_row() - not designed for number inputs, updated to support now Nov 28, 2020
@kensoh kensoh reopened this Nov 28, 2020
@kensoh
Copy link
Member

kensoh commented Dec 19, 2020

Closing issue since the latest packaged release TagUI v6.14 is out.

Release notes - https://github.com/kelaberetiv/TagUI/releases/tag/v6.14.0
To download v6.14 - https://tagui.readthedocs.io/en/latest/setup.html
Documentation - https://tagui.readthedocs.io/en/latest/index.html

@kensoh kensoh closed this as completed Dec 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants