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

White-Space on Parameter Values #42

Open
jkeczan opened this issue Sep 5, 2019 · 1 comment
Open

White-Space on Parameter Values #42

jkeczan opened this issue Sep 5, 2019 · 1 comment

Comments

@jkeczan
Copy link

jkeczan commented Sep 5, 2019

We have hit an issue in the report runner where we have parameter values that require a leading space and an optional trailing space.

The current implementation does not support this functionality since it is calling trim on the ParameterValue in the ParameterCore.cs file.

We can easily remove the .trim() function for our purposes but a parameter flag of -T On|Off would be beneficial in this case.

@mhertzfeld
Copy link

The Trim function has been removed by mikecrf121 in a fork he made of my repo.
https://github.com/mhertzfeld/CrystalReportsNinja/commits?author=mikecrf121

I have merged mikecrf121's change into my repo. You can test the change in the following branch of my repo.
https://github.com/mhertzfeld/CrystalReportsNinja/tree/1.4.0.0

If you still would like a parameter added for this functionality you can create an issue under my repo and I will look into addressing it.

https://github.com/mhertzfeld/CrystalReportsNinja/issues

danielleevandenbosch pushed a commit to danielleevandenbosch/CrystalReportsNinja that referenced this issue Apr 28, 2021
Removed extra instance of CSV in the readme file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants