Bug #8151: Technique Editor - whitespace as separator in "File ensure keys -> values present" method - Rudder - Issue Tracker

Actions

Copy link

Bug #8151

closed

Technique Editor - whitespace as separator in "File ensure keys -> values present" method

Added by David Svejda over 8 years ago. Updated over 2 years ago.

Status:

Released

Priority:

N/A

Assignee:

Benoît PECCATTE

Category:

Generic methods

Target version:

ncf-0.x

Pull Request:

https://github.com/Normation/ncf/pull...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

Creating/modifying my own Technique.

When I use method "File ensure keys -> values present", I cannot put the whitespace in the "Separator between key and value" field as hinted, because when I just hit the Spacebar, it doesn't accept it. I've tried to use the "\s" there, but it doesn't seem to work as expected.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Vincent MEMBRÉ over 8 years ago

Project changed from Rudder to 41
Category deleted (~~Web - Technique editor~~)
Assignee set to Vincent MEMBRÉ
Target version set to 0.x

Actions

Copy link

Updated by Vincent MEMBRÉ over 8 years ago

Status changed from New to In progress

Actions

Copy link

Updated by Vincent MEMBRÉ over 8 years ago

Status changed from In progress to Pending technical review
Assignee changed from Vincent MEMBRÉ to François ARMAND
Pull Request set to https://github.com/Normation/ncf/pull/338

PR https://github.com/Normation/ncf/pull/338

Actions

Copy link

Updated by François ARMAND over 8 years ago

Some more context on the bug: for now, all input fields in the Technique Editor are trimed, and it's the reason why we are seeing the bug.

We can't simply stop triming all fields, because in lots of case, we DO want timming. We are on it.

Actions

Copy link

Updated by Nicolas CHARLES over 8 years ago

As a workaround, we could define somewhere a variable that contains only space, and use it as a separator.

Actions

Copy link

Updated by Nicolas CHARLES over 8 years ago

Status changed from Pending technical review to In progress
Assignee changed from François ARMAND to Vincent MEMBRÉ

Correctingicket status: it is not in technical review, as the PR has been closed

Actions

Copy link

Updated by Jonathan CLARKE over 8 years ago

Subject changed from Technique Editor - whitespace as separator to Technique Editor - whitespace as separator in "File ensure keys -> values present" method
Pull Request deleted (~~https://github.com/Normation/ncf/pull/338~~)

We are implementing a "constraints" system in ncf (generic methods, API and Technique editor). Once that's ready (later this week), we'll be able to add a constraint to this generic method and fix this.

Actions

Copy link