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

Gradient support can break Graphs containing special characters #5196

Closed
TheWitness opened this issue Jan 26, 2023 · 0 comments
Closed

Gradient support can break Graphs containing special characters #5196

TheWitness opened this issue Jan 26, 2023 · 0 comments
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone
v1.2.24

Comments

@TheWitness
Copy link
Member

Describe the bug

Gradient support double escapes certain characters that must be escaped causing Graphs not to render.

To Reproduce

Steps to reproduce the behavior:

  1. Create an Array Graph Item

  2. Put a special character like a colon in the Title Format

  3. Create a Graph using this template

  4. See error

Expected behavior

Bug Free Cacti
@TheWitness TheWitness added bug Undesired behaviour confirmed Bug is confirm by dev team labels Jan 26, 2023
@TheWitness TheWitness added this to the v1.2.24 milestone Jan 26, 2023
TheWitness added a commit that referenced this issue Jan 26, 2023
@TheWitness
Fixing #5196: Gradient issues
fe727fa 
Gradient support breaks certain Graphs with special characters in them
@TheWitness TheWitness added resolved A fixed issue porting required Requires porting to develop labels Jan 26, 2023
TheWitness added a commit that referenced this issue Jan 27, 2023
@TheWitness
QA: One remaining issue related to #5196
40bef39 
The gradient was not using the correct variable for displaying the Graph Item text format.
@netniV netniV changed the title Gradient support breaks certain Graphs with special characters in them Gradient support can break Graphs containing special characters Feb 15, 2023
@TheWitness TheWitness removed the porting required Requires porting to develop label Feb 20, 2023
@TheWitness TheWitness mentioned this issue Feb 20, 2023
Merged
@github-actions github-actions bot locked and limited conversation to collaborators May 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Milestone
v1.2.24
Development

No branches or pull requests
1 participant
@TheWitness