Skip to content

Bugfix - Multi Values Properties Set#583

Merged
talarian1 merged 4 commits intojfrog:masterfrom
talarian1:gradle-props-bug
Dec 19, 2021
Merged

Bugfix - Multi Values Properties Set#583
talarian1 merged 4 commits intojfrog:masterfrom
talarian1:gradle-props-bug

Conversation

@talarian1
Copy link
Copy Markdown
Contributor

@talarian1 talarian1 commented Dec 7, 2021

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

Fixes #567

@talarian1 talarian1 changed the title Bugfix - Multi Values Propeties Bugfix - Multi Values Properties Set Dec 7, 2021
@talarian1 talarian1 requested a review from eyalbe4 December 7, 2021 22:58
matrix.append(";").append(encodeProperties ? encode(propertyKey) : propertyKey)
.append("=").append(encodeProperties ? encode(propertyValue) : propertyValue);
for (String multiPropertyValue : matrixParams.get(propertyKey)) {
// Due to known bug in Artifactory, in order to support multi values properties
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worth to add a reference to this bug in Artifactory.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
I couldn't find a public open issue for that. I will contact our suitable team to make sure it will be fixed soon.

@talarian1 talarian1 merged commit 3e1e674 into jfrog:master Dec 19, 2021
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

Successfully merging this pull request may close these issues.

Multi-value properties are not possible to set using Gradle plugin

3 participants