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

Don't replace fields containing parentheses #48

Open
NithoPamc opened this issue Aug 16, 2021 · 0 comments
Open

Don't replace fields containing parentheses #48

NithoPamc opened this issue Aug 16, 2021 · 0 comments

Comments

@NithoPamc
Copy link

Hi,

I am new to FPDM and have a problem the first time I use it.

When I want to replace a field whose value is:
/V (\(Saisir l'adresse sur 3 lignes minimum\)\rDESTINATAIRE\rADRESSE \rCP VILLE ))
He replaces with:
/V <20727565206465732063616E61726473>\rDESTINATAIRE\rADRESSE \rCP VILLE )
instead of
/V <20727565206465732063616E61726473>

So I corrected line 836 on /fpdm/fpdm.php :
if(preg_match('#/V\s?[<(]([^>)]*)[>)]#', $CurLine, $a, PREG_OFFSET_CAPTURE))
in
if(preg_match('#/V\s?[<(]([^>]*)[>)]#', $CurLine, $a, PREG_OFFSET_CAPTURE))

But is this the correct correction?

@NithoPamc NithoPamc changed the title FPDM Do not replace fields containing parentheses FPDM Don't replace fields containing parentheses Aug 16, 2021
@NithoPamc NithoPamc changed the title FPDM Don't replace fields containing parentheses Don't replace fields containing parentheses Aug 16, 2021
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

No branches or pull requests

1 participant