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

FIX: SUPPLIER_PROPOSAL_FREE_TEXT get HTML tags removed #32455

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

warnerbryce
Copy link
Contributor

@warnerbryce warnerbryce commented Dec 23, 2024

FIX

on the admin page of Supplier Proposal,
You have an Fckeditor frame where you can set the const SUPPLIER_PROPOSAL_FREE_TEXT
Compared to other FREE_TEXT on other modules, this one get preg_match before writing into database.
The problem is that the GETPOST function remove the HTML TAGS of the text entered.
You will get a bad looking text when printing onto PDF.

I corrected the GETPOST function to put the argument check on restrickhtml.

EDIT :
There is others CONSTANTS to write into database on this page, so i made the GETPOST modification just for SUPPLIER_PROPOSAL_FREE_TEXT

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.

2 participants