Case update texts. Some characters are removed #4318

Open
opened 2026-02-20 16:16:10 -05:00 by deekerman · 3 comments
Owner

Originally created by @QuickCRM on GitHub (Jun 9, 2020).

Issue

In a case, enter the following in Update text:

<Chemin>\<N°Ticket>\.*

What is saved is

\\*

Expected Behavior

The complete text should be saved

Actual Behavior

Possible Fix

Steps to Reproduce

  1. Try that on https://demo.suiteondemand.com

Context

Your Environment

  • SuiteCRM Version used: 7.8, 7.10, 7.11
  • Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)):
  • Environment name and version (e.g. MySQL, PHP 7):
  • Operating System and version (e.g Ubuntu 16.04):
Originally created by @QuickCRM on GitHub (Jun 9, 2020). <!--- Provide a general summary of the issue in the **Title** above --> <!--- Before you open an issue, please check if a similar issue already exists or has been closed before. ---> <!--- If you have discovered a security risk please report it by emailing security@suitecrm.com. This will be delivered to the product team who handle security issues. Please don't disclose security bugs publicly until they have been handled by the security team. ---> #### Issue <!--- Provide a more detailed introduction to the issue itself, and why you consider it to be a bug --> In a case, enter the following in Update text: ``` <Chemin>\<N°Ticket>\.* ``` What is saved is ``` \\* ``` <!--- Ensure that all code ``` is surrounded ``` by triple back quotes. This can also be done over multiple lines --> #### Expected Behavior <!--- Tell us what should happen --> The complete text should be saved #### Actual Behavior <!--- Tell us what happens instead --> <!--- Also please check relevant logs (suitecrm.log, php error.log etc.) --> #### Possible Fix <!--- Not obligatory, but suggest a fix or reason for the bug --> #### Steps to Reproduce <!--- Provide a link to a live example, or an unambiguous set of steps to --> <!--- reproduce this bug include code to reproduce, if relevant --> 1. Try that on https://demo.suiteondemand.com #### Context <!--- How has this bug affected you? What were you trying to accomplish? --> <!--- If you feel this should be a low/medium/high priority then please state so --> #### Your Environment <!--- Include as many relevant details about the environment you experienced the bug in --> * SuiteCRM Version used: 7.8, 7.10, 7.11 * Browser name and version (e.g. Chrome Version 51.0.2704.63 (64-bit)): * Environment name and version (e.g. MySQL, PHP 7): * Operating System and version (e.g Ubuntu 16.04):
Author
Owner

@Mac-Rae commented on GitHub (Jun 10, 2020):

Hi @QuickCRM,

This issue has been confirmed to exist and is replicable in the latest 7.10.x version.

Not sure how this would be fixed as I assume its the HTML stripper removing the < > symbols assuming there tags but good to note ether way.

Thanks 👍

@Mac-Rae commented on GitHub (Jun 10, 2020): Hi @QuickCRM, This issue has been confirmed to exist and is replicable in the latest 7.10.x version. Not sure how this would be fixed as I assume its the HTML stripper removing the < > symbols assuming there tags but good to note ether way. Thanks :+1:
Author
Owner

@QuickCRM commented on GitHub (Jun 10, 2020):

Hi @Mac-Rae,

Yes I guess it's due to HTML cleanup but I see absolutely no reason why that would be removed.
That's a real problem and there is no valid reason I can give to our SuiteCRM users

@QuickCRM commented on GitHub (Jun 10, 2020): Hi @Mac-Rae, Yes I guess it's due to HTML cleanup but I see absolutely no reason why that would be removed. That's a real problem and there is no valid reason I can give to our SuiteCRM users
Author
Owner

@pgorod commented on GitHub (Jun 10, 2020):

I've been investigating this (the generic problem with clean-ups) and I will post something with a proposal soon. We're simply not doing clean-up right, all over, and it's been bugging us a lot (pun intended).

Meanwhile the way forward for these smaller instances of the problem is to make the code use the data from the RAW_REQUEST instead of the _REQUEST. Then make sure you do some HTML clean-up before outputting it onto the screen, and that's it.

@pgorod commented on GitHub (Jun 10, 2020): I've been investigating this (the generic problem with clean-ups) and I will post something with a proposal soon. We're simply not doing clean-up right, all over, and it's been _bugging_ us a lot (pun intended). Meanwhile the way forward for these smaller instances of the problem is to make the code use the data from the RAW_REQUEST instead of the _REQUEST. Then make sure you do some HTML clean-up before outputting it onto the screen, and that's it.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/SuiteCRM-SuiteCRM#4318
No description provided.