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

#23396 escape double quotes #24637

Merged
merged 2 commits into from
Apr 17, 2023
Merged

#23396 escape double quotes #24637

merged 2 commits into from
Apr 17, 2023

Conversation

erickgonzalez
Copy link
Contributor

When a title field has double quotes it breaks the relationship field, so we need to escape them.

Since changes are in JSP no test was done.

@github-actions
Copy link

github-actions bot commented Apr 14, 2023

Unit Tests Report

1 423 tests  ±0   1 413 ✔️ ±0   3m 57s ⏱️ +5s
   140 suites ±0        10 💤 ±0 
   140 files   ±0          0 ±0 

Results for commit 15676c9. ± Comparison against base commit cffb861.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Apr 14, 2023

Postman Tests Report

     66 files  ±0  1 412 suites  ±0   3h 34m 32s ⏱️ - 7m 9s
   632 tests ±0     628 ✔️ ±0  0 💤 ±0  4 ±0 
2 348 runs  ±0  2 341 ✔️ ±0  0 💤 ±0  7 ±0 

For more details on these failures, see this check.

Results for commit 15676c9. ± Comparison against base commit cffb861.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Apr 14, 2023

Integration Tests [postgres] Report

   414 files  +1     414 suites  +1   1h 51m 7s ⏱️ + 11m 27s
3 973 tests +1  3 944 ✔️  - 4  23 💤 ±0  6 +5 
3 992 runs  +1  3 963 ✔️  - 4  23 💤 ±0  6 +5 

For more details on these failures, see this check.

Results for commit 15676c9. ± Comparison against base commit cffb861.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Apr 14, 2023

Integration Tests [mssql] Report

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 15676c9. ± Comparison against base commit cffb861.

♻️ This comment has been updated with latest results.

@dotcms-sonarqube
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@erickgonzalez erickgonzalez merged commit 294f3d2 into master Apr 17, 2023
18 of 24 checks passed
@erickgonzalez erickgonzalez deleted the issue-23396 branch April 17, 2023 18:15
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.

Double quotes in content title - error trying to show data in relate table.
4 participants