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

json.toVars adds " around variable content #1120

Closed
Merudo opened this issue Jan 15, 2020 · 1 comment
Closed

json.toVars adds " around variable content #1120

Merudo opened this issue Jan 15, 2020 · 1 comment
Assignees
@cwisniew
Labels
bug macro changes This issue adds or changes macro functions. Extra work is required (testing, wiki, code editor) tested This issue has been QA tested by someone other than the developer.

Comments

@Merudo
Copy link
Member

Merudo commented Jan 15, 2020

Describe the bug
The function json.toVars adds " and " around the content of the variable.

To Reproduce
Steps to reproduce the behavior:

  1. Type:
[h: array = json.append("[]", "a", "b", "c")]
[h: json.toVars(array, "variable")]
[strformat("The variable: %{variable1}.")]
  1. Notice the output is The variable: "b"., with " around the b.

Expected behavior
No " added around the content of the variable.

MapTool Info

  • Version: 1.5.11 Alpha 4, Develop (1/15/202)
  • Install: New

Desktop (please complete the following information):

  • OS: Windows
  • Version: 10
@cwisniew cwisniew mentioned this issue Jan 15, 2020
Merged
@Phergus Phergus added bug macro changes This issue adds or changes macro functions. Extra work is required (testing, wiki, code editor) labels Jan 15, 2020
@Merudo
Copy link
Member Author

Merudo commented Jan 16, 2020

Tested. Now that #1122 has been merged, json.toVars no longer adds " around the content of variables.

@Phergus Phergus added the tested This issue has been QA tested by someone other than the developer. label Jan 16, 2020
@Phergus Phergus closed this as completed Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cwisniew cwisniew

Labels
bug macro changes This issue adds or changes macro functions. Extra work is required (testing, wiki, code editor) tested This issue has been QA tested by someone other than the developer.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cwisniew @Merudo @Phergus