When validating the form instead of the usual confirmation the merged PDFs are displayed

[guilamu]

Hello,

Thanks a lot for your work.

Your plugin is indeed merging GravityPDFs and the PDFs submitted through the form which is awesome! Thank you very much.

I have an issue though: when pressing "SEND" the form does not validate and I juste have the merged PDFs displayed instead.

Please advise.

Bests,

[pimteam]

Hello,

We just updated to version 1.4, have you tried it?

[guilamu]

Awesome, thanks for the update, it works now!

[pimteam]

Do you use 1.4 or 1.4.1?

[guilamu]

I just updated to 1.4.1 and there was an issue with 2 GravityWiz plugins. I updated those to yesterday's last versions and now everythings working fine.

Is there any way to have automatic updates on your plugin (like when a plugin is on the official wordpress store)?

Thanks again for your work, time and support.

[pimteam]

We'll look into it, for the moment I'm afraid there are no automatic updates.

[guilamu]

I've got the Wordpress critical error again.

Error logs: gravityforms_51de114de1606c842d21321b80f5017256b45c68.txt gravity-pdf_53600e8988517ca7485ea96893564e06b425b61a.txt php.txt

[pimteam]

Thanks, but I don't see any fatal errors in these logs. You'll need to look in your WP debug log.

[guilamu]

Even in the php.txt?

[Fri Jan 06 14:17:36.602250 2023] [php:error] [pid 30394] [client xxxxxxxxxxxxx:52359] PHP Fatal error: Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php:119\nStack trace:\n#0 /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php(207): gf_merge_pdfs_output()\n#1

Seems like an error to me, right?

[pimteam]

I did not notice there was a 3rd file. Let's try version 1.4.2

[guilamu]

Thank you again, I'll test it and report back ASAP.

[guilamu]

1.4.2 is not working, no validation, and sent to a blank PDF now.

[Fri Jan 06 18:45:00.234835 2023] [php:warn] [pid 5719] [client 77.207.134.10:53951] PHP Warning: Undefined variable $files in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207, referer: https://fo-snudi.fr/fiche-de-frais/ [Fri Jan 06 18:45:00.234858 2023] [php:warn] [pid 5719] [client 77.207.134.10:53951] PHP Warning: Undefined variable $entry_id in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 116, referer: https://fo-snudi.fr/fiche-de-frais/ [Fri Jan 06 18:45:00.234875 2023] [php:warn] [pid 5719] [client 77.207.134.10:53951] PHP Warning: foreach() argument must be of type array|object, null given in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 144, referer: https://fo-snudi.fr/fiche-de-frais/ [Fri Jan 06 18:45:44.095885 2023] [php:warn] [pid 31591] [client 77.207.134.10:53960] PHP Warning: Undefined variable $files in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207, referer: https://fo-snudi.fr/fiche-de-frais/ [Fri Jan 06 18:45:44.095907 2023] [php:warn] [pid 31591] [client 77.207.134.10:53960] PHP Warning: Undefined variable $entry_id in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 116, referer: https://fo-snudi.fr/fiche-de-frais/ [Fri Jan 06 18:45:44.095912 2023] [php:warn] [pid 31591] [client 77.207.134.10:53960] PHP Warning: foreach() argument must be of type array|object, null given in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 144, referer: https://fo-snudi.fr/fiche-de-frais/ gravityforms_5fbaf107ad3b55d5ba6809c16b9b4e5d4db069c2.txt gravity-pdf_8228f06e43c434143730c23cd573a1f77063ea43.txt

[guilamu]

Thanks for the update, still not working though. 1.4.3 results in a Wordpress Critical Failure.

Here are the logs:

[Wed Jan 11 08:59:09.608676 2023] [php:error] [pid 11637] [client 90.85.102.82:54927] PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function class@anonymous::__construct(), 0 passed in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 202 and exactly 2 expected in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php:203\nStack trace:\n#0 /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php(202): class@anonymous->__construct()\n#1 /var/www/snudi/wp-includes/class-wp-hook.php(308): {closure}()\n#2 /var/www/snudi/wp-includes/plugin.php(205): WP_Hook->apply_filters()\n#3 /var/www/snudi/wp-content/plugins/gravity-forms-pdf-extended/src/Helper/Helper_PDF.php(301): apply_filters()\n#4 /var/www/snudi/wp-content/plugins/gravity-forms-pdf-extended/src/Model/Model_PDF.php(1141): GFPDF\\Helper\\Helper_PDF->generate()\n#5 /var/www/snudi/wp-content/plugins/gravity-forms-pdf-extended/src/Model/Model_PDF.php(1056): GFPDF\\Model\\Model_PDF->process_and_save_pdf()\n#6 /var/www/snudi/wp-content/plugins/gravity-forms-pdf-extended/src/Model/Model_PDF.php(989): GFPDF\\Model\\Model_PDF->generate_and_save_pdf()\n#7 /var/www/snudi/wp-includes/class-wp-hook.php(310): GFPDF\\Model\\Model_PDF->notifications()\n#8 /var/www/snudi/wp-includes/plugin.php(205): WP_Hook->apply_filters()\n#9 /var/www/snudi/wp-content/plugins/gravityforms/gravityforms.php(7015): apply_filters()\n#10 /var/www/snudi/wp-content/plugins/gravityforms/common.php(1888): gf_apply_filters()\n#11 /var/www/snudi/wp-content/plugins/gravityforms/common.php(2067): GFCommon::send_notification()\n#12 /var/www/snudi/wp-content/plugins/gravityforms/includes/api.php(2225): GFCommon::send_notifications()\n#13 /var/www/snudi/wp-content/plugins/gravityforms/common.php(2073): GFAPI::send_notifications()\n#14 /var/www/snudi/wp-content/plugins/gravityforms/form_display.php(1667): GFCommon::send_form_submission_notifications()\n#15 /var/www/snudi/wp-content/plugins/gravityforms/form_display.php(171): GFFormDisplay::handle_submission()\n#16 /var/www/snudi/wp-content/plugins/gravityforms/gravityforms.php(844): GFFormDisplay::process_form()\n#17 /var/www/snudi/wp-includes/class-wp-hook.php(308): GFForms::maybe_process_form()\n#18 /var/www/snudi/wp-includes/class-wp-hook.php(332): WP_Hook->apply_filters()\n#19 /var/www/snudi/wp-includes/plugin.php(565): WP_Hook->do_action()\n#20 /var/www/snudi/wp-includes/class-wp.php(797): do_action_ref_array()\n#21 /var/www/snudi/wp-includes/functions.php(1332): WP->main()\n#22 /var/www/snudi/wp-blog-header.php(16): wp()\n#23 /var/www/snudi/index.php(17): require('...')\n#24 {main}\n thrown in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 203, referer: https://fo-snudi.fr/fiche-de-frais/

[pimteam]

Updated again, it should be good now.

[guilamu]

Thanks again, but that's not it. Now I'm being redirected to the PDF instead of the confirmation page.

[Wed Jan 11 09:08:56.884256 2023] [access_compat:error] [pid 12160] [client 93.23.248.8:62410] AH01797: client denied by server configuration: /var/www/snudi/wp-content/uploads/wpsc/, referer: https://fo-snudi.fr/wp-content/uploads/2022/ [Wed Jan 11 09:08:59.573719 2023] [access_compat:error] [pid 12160] [client 93.23.248.8:62410] AH01797: client denied by server configuration: /var/www/snudi/wp-content/uploads/wpsc/, referer: https://fo-snudi.fr/wp-content/uploads/2022/ [Wed Jan 11 10:09:00.459069 2023] [php:warn] [pid 13702] [client 90.85.102.82:57665] PHP Warning: Undefined variable $entry_id in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 116, referer: https://fo-snudi.fr/fiche-de-frais/

[guilamu]

1.4.5 totally kills my wordpress site. [Wed Jan 11 15:05:14.479309 2023] [php:error] [pid 21714] [client 212.235.15.142:55614] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:15.254739 2023] [php:error] [pid 22144] [client 212.235.15.140:60376] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:15.718288 2023] [php:error] [pid 22284] [client 212.235.15.142:55662] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:17.806860 2023] [php:error] [pid 22126] [client 77.207.134.10:57399] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:18.225795 2023] [php:error] [pid 22247] [client 212.235.15.140:59152] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:20.190548 2023] [php:error] [pid 22243] [client 212.235.15.140:59158] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:20.513329 2023] [php:error] [pid 22283] [client 77.207.134.10:57400] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:21.397584 2023] [php:error] [pid 22245] [client 77.207.134.10:57402] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:25.421918 2023] [php:error] [pid 22224] [client 77.207.134.10:57405] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207, referer: https://fo-snudi.fr/wp-admin/update.php?action=upload-plugin&package=9879&overwrite=update-plugin&_wpnonce=3095938119 [Wed Jan 11 15:05:32.366788 2023] [php:error] [pid 22246] [client 205.185.121.252:54698] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:54.508083 2023] [php:error] [pid 21714] [client 62.90.131.202:41024] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:05:55.362155 2023] [php:error] [pid 22144] [client 62.90.131.202:41096] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207 [Wed Jan 11 15:06:20.275657 2023] [php:error] [pid 22284] [client 92.184.121.151:58714] PHP Parse error: syntax error, unexpected token "}" in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 207, referer: https://www.google.com/

[pimteam]

Sorry, fixed now.

[guilamu]

Thanks. Still not working: redirected to the PDF instead of the confirmation page.

[Wed Jan 11 19:01:47.574300 2023] [php:warn] [pid 28374] [client 77.207.134.10:50184] PHP Warning: Undefined variable $entry_id in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 209, referer: https://fo-snudi.fr/fiche-de-frais/ [Wed Jan 11 19:01:47.574325 2023] [php:warn] [pid 28374] [client 77.207.134.10:50184] PHP Warning: Undefined property: class@anonymous::$ in /var/www/snudi/wp-content/plugins/gravityforms-merge-pdfs-main/loader.php on line 209, referer: https://fo-snudi.fr/fiche-de-frais/

[pimteam]

Next fix added.

[guilamu]

Still the same issue, but nothing in apache logs this time. gravity-pdf_aeae2b87de7570b2918bef65f3e4dc5af9a16c0f.txt gravityforms_11a3ff6e345e98b80783efa4737d2492624f8ed2.txt

[guilamu]

Hello, would you mind to look into it please? Thank you very much!

[verysiberian]

Hi, Guilamu. Bob is away until later this week, but I'm happy to try to help. Would you please describe the current issue in detail and include one or more screenshots?

Best regards, Rob

On Mon, Jan 16, 2023 at 3:05 AM Guilamu @.***> wrote:

Hello, would you mind to look into it please? Thank you very much!

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383700217, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GVABVRGFDO2QB7IW3WSUFPJANCNFSM6AAAAAATQYISZ4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Hello Rob,

Thank you very much for your support.

Well a screenshots would not be of any help in this case: when you clic on the "SEND" button at the en of the form, instead of being redirect to the validation message/page you are redirected to the merged PDF created with your plugin.

All the relevants logs are provided in my previous messsage.

I can provide you with login credentials if you want to try it yourself.

Here's a video of the bug.

[verysiberian]

Sure, please email them directly to me @.***).

On Mon, Jan 16, 2023 at 6:10 AM Guilamu @.***> wrote:

Hello Rob,

Thank you very much for your support.

Well a screenshots would not be of any help in this case: when you clic on the "SEND" button at the en of the form, instead of being redirect to the validation message/page you are redirected to the merged PDF created with your plugin.

All the relevants logs are provided in my previous messsage.

I can provide you with login credentials if you want to try it yourself.

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383960529, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6EATA3N6P3KW76A4ZLWSU3CNANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Sent from my iPhone

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Your email has been removed form your message.

I've added a video of the bug happening: https://www.awesomescreenshot.com/video/14022238?key=340ce540cdcd72654429fcecc083052a

[verysiberian]

Thanks for the video, but I will need login credentials if I am going to assist you. There are a number of things I would need to check. Otherwise, feel free to wait for Bob to return, and we will see if he is able to help.

On Mon, Jan 16, 2023 at 6:18 AM Guilamu @.***> wrote:

Your email has been removed form your message.

I've added a video of the bug happening: https://www.awesomescreenshot.com/video/14022238?key=340ce540cdcd72654429fcecc083052a

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383971601, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GEZVLRCOI64JGRHNTWSU4BRANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Sent from my iPhone

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

I'm all for giving you the credentials, but I still need some way to send them to you ^^.

[guilamu]

I just created a chat space on my own Nextcould instance here: https://bluf.fr/call/xdfr6gn5

[verysiberian]

Sorry, my email address was stripped automatically from my reply. You may send the login credentials securely through this contact form:

https://cerebralconsulting.net/contact-us/

On Mon, Jan 16, 2023 at 6:25 AM Guilamu @.***> wrote:

I'm all for giving you the credentials, but I still need some way to send them to you ^^.

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383980405, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Sorry, my email address was stripped automatically from my reply. You may send the login credentials securely through this contact form: https://cerebralconsulting.net/contact-us/ … On Mon, Jan 16, 2023 at 6:25 AM Guilamu @.> wrote: I'm all for giving you the credentials, but I still need some way to send them to you ^^. — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Email sent!

[verysiberian]

Thanks, I am logged in. Would you be able to let me access the plugins section? I would like to use the Health Check and Troubleshooting plugin to do some tests without disrupting your live site.

On Mon, Jan 16, 2023 at 9:13 AM Guilamu @.***> wrote:

Sorry, my email address was stripped automatically from my reply. You may send the login credentials securely through this contact form: https://cerebralconsulting.net/contact-us/ … <#m6019447554565274132> On Mon, Jan 16, 2023 at 6:25 AM Guilamu @.> wrote: I'm all for giving you the credentials, but I still need some way to send them to you ^^. — Reply to this email directly, view it on GitHub <#1 (comment) https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383980405>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Email sent!

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1384197569, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6B63XPAO6F6YHVAPPTWSVQSDANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[verysiberian]

I submitted a test entry using GP Live Preview, and the confirmation worked as expected. Please see the attached screenshot.

On Mon, Jan 16, 2023 at 9:49 AM Rob Davis @.***> wrote:

Thanks, I am logged in. Would you be able to let me access the plugins section? I would like to use the Health Check and Troubleshooting plugin to do some tests without disrupting your live site.

On Mon, Jan 16, 2023 at 9:13 AM Guilamu @.***> wrote:

Sorry, my email address was stripped automatically from my reply. You may send the login credentials securely through this contact form: https://cerebralconsulting.net/contact-us/ … <#m_-3145343664338318374_m6019447554565274132> On Mon, Jan 16, 2023 at 6:25 AM Guilamu @.> wrote: I'm all for giving you the credentials, but I still need some way to send them to you ^^. — Reply to this email directly, view it on GitHub <#1 (comment) https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1383980405>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 https://github.com/notifications/unsubscribe-auth/AE32N6E4FY25K3MMHW5IBDDWSU45LANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Email sent!

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1384197569, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6B63XPAO6F6YHVAPPTWSVQSDANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Indeed!

Very strange...

It's getting late in this neck of the woods, so I'll be investigating this tomorrow and get back to you.

Thanks for your help!

[verysiberian]

Understood! Have a good night. I'm sure that we can figure this out.

On Mon, Jan 16, 2023 at 10:00 AM Guilamu @.***> wrote:

Indeed!

Very strange...

It's getting late in this neck of the woods, so I'll be investigating this tomorrow and get back to you.

Thanks for your help!

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1384250323, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6HQ6F4IO2Z2TILPOBLWSVWA5ANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Okay so thanks to you I was able to pinpoint the bug.

The only difference when you tried with the "live preview" was that notifications weren't sent.

So I tried again the live form with notifications disabled and it did work.

Next I had to figure out which notification was breaking the validation and it was the one with the "{all_fields}" merge tag. If I remove the {all_fields} merge tag in that notification it was working again.

You can try it yourself, it seems that your "Merge PDF" field is not compatible with the {all_fields} merge tag.

Thanks for your help, have a good one!

[guilamu]

Another bug I just discovered: if I attach any GravityPDF generated PDF to a notification, the notification is not sent and the confirmation is not working (instead the user is redirected to the PDF when validating the form).

[verysiberian]

Hi, Guilamu. Thanks for the updates. Including the attached merged PDF in a notification has not, at least yet, been a part of the workflow of any site on which I'm using the merge PDFs plugin. As such, it is not supported at this time, but we can think about making it so. In the meantime, have you used the Gravity Wiz All Fields plugin?

https://gravitywiz.com/gravity-forms-all-fields-template/

It is free and lets you exclude specific fields from the {all_fields} merge tag. Hence, as a workaround, you could insert the field ID for the Merge PDFs field in the notification and/or confirmation as needed, e.g.,

https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier

Best regards, Rob

On Tue, Jan 17, 2023 at 4:47 AM Guilamu @.***> wrote:

Another bug I just discovered: if I attach any GravityPDF generated PDF to a notification, the notification is not sent and the confirmation is not working (instead the user is redirected to the PDF when validating the form).

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1385214355, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GEMJ5QZ36PNFGISPDWSZ2EZANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Thanks again for your support.

Hi, Guilamu. Thanks for the updates. Including the attached merged PDF in a notification has not, at least yet, been a part of the workflow of any site on which I'm using the merge PDFs plugin. As such, it is not supported at this time, but we can think about making it so. In the meantime, have you used the Gravity Wiz All Fields plugin? https://gravitywiz.com/gravity-forms-all-fields-template/ It is free and lets you exclude specific fields from the {all_fields} merge tag. Hence, as a workaround, you could insert the field ID for the Merge PDFs field in the notification and/or confirmation as needed, e.g., https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier Best regards, Rob …

Hello, I just tried it while excluding the MergePDF field and it's not working. FYI, I used this merge tag : {all_fields:exclude[79,75.3]}

As soon as I remove the merge tag it works again.

[verysiberian]

Sorry to hear that. If my account on your site is still active, I'm happy to take another look. If I could access your plugin section, too, that would be great. I promise not to break anything.

On Tue, Jan 17, 2023 at 6:15 AM Guilamu @.***> wrote:

Thanks again for your support.

Hi, Guilamu. Thanks for the updates. Including the attached merged PDF in a notification has not, at least yet, been a part of the workflow of any site on which I'm using the merge PDFs plugin. As such, it is not supported at this time, but we can think about making it so. In the meantime, have you used the Gravity Wiz All Fields plugin? https://gravitywiz.com/gravity-forms-all-fields-template/ It is free and lets you exclude specific fields from the {all_fields} merge tag. Hence, as a workaround, you could insert the field ID for the Merge PDFs field in the notification and/or confirmation as needed, e.g., https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier Best regards, Rob … <#m807218770704929661>

Hello, I just tried it while excluding the MergePDF field and it's not working. FYI, I used this merge tag : {all_fields:exclude[79,75.3]}

As soon as I remove the merge tag it works again.

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1385336926, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GY3DCUOLS3FMOLLK3WS2EN7ANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Sorry to hear that. If my account on your site is still active, I'm happy to take another look. If I could access your plugin section, too, that would be great. I promise not to break anything. … On Tue, Jan 17, 2023 at 6:15 AM Guilamu @.> wrote: Thanks again for your support. Hi, Guilamu. Thanks for the updates. Including the attached merged PDF in a notification has not, at least yet, been a part of the workflow of any site on which I'm using the merge PDFs plugin. As such, it is not supported at this time, but we can think about making it so. In the meantime, have you used the Gravity Wiz All Fields plugin? https://gravitywiz.com/gravity-forms-all-fields-template/ It is free and lets you exclude specific fields from the {all_fields} merge tag. Hence, as a workaround, you could insert the field ID for the Merge PDFs field in the notification and/or confirmation as needed, e.g., https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier Best regards, Rob … <#m807218770704929661> Hello, I just tried it while excluding the MergePDF field and it's not working. FYI, I used this merge tag : {all_fields:exclude[79,75.3]} As soon as I remove the merge tag it works again. — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GY3DCUOLS3FMOLLK3WS2EN7ANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Yes, your account is still alive, please do as you see fit (worst case scenario I've got backups).

[verysiberian]

I just exported your form and imported it onto one of my sites that uses many of the same plugins. It worked well, and I received the notification. No improper redirects occurred. I continue to suspect that you have a conflict with your theme or another plugin. I cannot test this, though, without having access to your plugins section and being able to install/activate the Health Check and Troubleshooting plugin to avoid messing with your live site. If you want to handle that yourself, please feel free to do so, but that would be my recommendation.

Best regards, Rob

On Tue, Jan 17, 2023 at 9:32 AM Guilamu @.***> wrote:

Sorry to hear that. If my account on your site is still active, I'm happy to take another look. If I could access your plugin section, too, that would be great. I promise not to break anything. … <#m1789754440494462854> On Tue, Jan 17, 2023 at 6:15 AM Guilamu @.> wrote: Thanks again for your support. Hi, Guilamu. Thanks for the updates. Including the attached merged PDF in a notification has not, at least yet, been a part of the workflow of any site on which I'm using the merge PDFs plugin. As such, it is not supported at this time, but we can think about making it so. In the meantime, have you used the Gravity Wiz All Fields plugin? https://gravitywiz.com/gravity-forms-all-fields-template/ https://gravitywiz.com/gravity-forms-all-fields-template/ It is free and lets you exclude specific fields from the {all_fields} merge tag. Hence, as a workaround, you could insert the field ID for the Merge PDFs field in the notification and/or confirmation as needed, e.g., https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier https://gravitywiz.com/gravity-forms-all-fields-template/#field-filtering-modifier Best regards, Rob … <#m807218770704929661> Hello, I just tried it while excluding the MergePDF field and it's not working. FYI, I used this merge tag : {all_fields:exclude[79,75.3]} As soon as I remove the merge tag it works again. — Reply to this email directly, view it on GitHub <#1 (comment) https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1385336926>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6GY3DCUOLS3FMOLLK3WS2EN7ANCNFSM6AAAAAATQYISZ4 https://github.com/notifications/unsubscribe-auth/AE32N6GY3DCUOLS3FMOLLK3WS2EN7ANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Yes, your account is still alive, please do as you see fit (worst case scenario I've got backups).

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1385608425, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6HAKM3YC6MYNQ7SDTLWS23Q7ANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Hello Rob,

Thanks for your help.

You have access to everything since you have and Administrator account.

You should be able to activate/install whatever you want, are you not?

[verysiberian]

You're right -- sorry about that. My French is terrible! Thanks to Google Translate, I now see where the plugins have been all along. 😛 We silly Americans are mostly monolinguals and not very good at this sort of thing.

On Wed, Jan 18, 2023 at 4:45 AM Guilamu @.***> wrote:

Hello Rob,

Thanks for your help.

You have access to everything since you have and Administrator account.

You should be able to activate/install whatever you want, are you not?

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1386848127, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6DTIONBKH4A2OV3AU3WS7CWBANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

You're right -- sorry about that. My French is terrible! Thanks to Google Translate, I now see where the plugins have been all along. 😛 We silly Americans are mostly monolinguals and not very good at this sort of thing. … On Wed, Jan 18, 2023 at 4:45 AM Guilamu @.> wrote: Hello Rob, Thanks for your help. You have access to everything since you have and Administrator account. You should be able to activate/install whatever you want, are you not? — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6DTIONBKH4A2OV3AU3WS7CWBANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Oh my bad, I though you could switch the site to English in your user settings.

[verysiberian]

Please send me the link for the page on the frontend with the form when you have a moment.

On Wed, Jan 18, 2023 at 4:50 AM Guilamu @.***> wrote:

You're right -- sorry about that. My French is terrible! Thanks to Google Translate, I now see where the plugins have been all along. 😛 We silly Americans are mostly monolinguals and not very good at this sort of thing. … <#m7195796556110675104> On Wed, Jan 18, 2023 at 4:45 AM Guilamu @.> wrote: Hello Rob, Thanks for your help. You have access to everything since you have and Administrator account. You should be able to activate/install whatever you want, are you not? — Reply to this email directly, view it on GitHub <#1 (comment) https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1386848127>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6DTIONBKH4A2OV3AU3WS7CWBANCNFSM6AAAAAATQYISZ4 https://github.com/notifications/unsubscribe-auth/AE32N6DTIONBKH4A2OV3AU3WS7CWBANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.> -- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/ Cerebral Consulting, LLC https://cerebralconsulting.net/ -- This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Oh my bad, I though you could switch the site to English in your user settings.

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1386853288, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6CARLOFASUNAYSGPITWS7DFPANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

https://fo-snudi.fr/fiche-de-frais/

[verysiberian]

It's password protected. Can you send the password and I'll change it to Private so that it stays off the live site?

On Wed, Jan 18, 2023 at 5:07 AM Guilamu @.***> wrote:

https://fo-snudi.fr/fiche-de-frais/

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1386875716, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6BLJWQD5THXNYFW7NTWS7FGVANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Oh my bad, the password is : FRAIS

[guilamu]

Hello, any update?

Thnak you very much!

[verysiberian]

You're up late! I am actually going to get it again here in a few minutes.

On Thu, Jan 19, 2023 at 3:01 PM Guilamu @.***> wrote:

Hello, any update?

Thnak you very much!

— Reply to this email directly, view it on GitHub https://github.com/pimteam/gravityforms-merge-pdfs/issues/1#issuecomment-1397605755, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE32N6AF4J64N5H725ETR7DWTGTRZANCNFSM6AAAAAATQYISZ4 . You are receiving this because you commented.Message ID: @.***>

-- Robert N. Davis, PhD, MS, ABPP Houston Neuropsychology Associates, PLLC https://houston-npa.com/

Cerebral Consulting, LLC https://cerebralconsulting.net/

--

This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information or otherwise be protected by law. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

[guilamu]

Yep! Still up for 1 hour or so and then I'll be back in 8/9 hours. Thanks.