Hello. Thank you for the latest update. While testing the new update,and specificlly the select-other option, I stumbled on the warning which I only receive after clicking on the Webform tab. I also think there might be some issues with the modifiers 'nl','-all','nl-all' as they do not seem to be included in the PDF correctly. I will experiment a little bit more to see whether I am doing something wrong on my side.
Thanks for your help and time.

Comments

alice-kahn’s picture

alice-kahn commented

I'm received the same warning message after submission the webform;
Warning: Wrong parameter count for nl2br()
webform2pdf/webform2pdf.module on line 418

Please help,

Thanks and regards

mr.york’s picture

mr.york commented

Patch: http://drupalcode.org/project/webform2pdf.git/blobdiff/390bdcfa83ac016aa...
New release: 6.x-2.1

alice-kahn’s picture

alice-kahn commented

Mr. York you rock!!!

It works for me now, thanks alot you are a life saver!

pierog1971’s picture

pierog1971 commented

Thank you for the fix. I believe there is still an issue with the modifiers in connection to having the select_other option. Allowing multiple selections from a list including one entered under other do not show up in the PDF while they do under Results/view. I tried using no modifier as well as -nl and the -all . In all cases, the other selection is missing in the PDF while it is there in Results/view.
Allowing only one selection seems to work better in regard to displaying the other option. The -all modifier works showing all choices. The nl-all does not work at all as far as I can tell. The only other thing I noticed about the PDF display using the -nl modifier is that there is a comma after the last item in the list generating the following PDF display: "apples, oranges," . If you select other and enter, "bananas", the PDF will display:
"apples, oranges, ,bananas" with the two commas separating oranges from the bold and underlined bananas.
Hope this makes some sense.

shafter’s picture

shafter commented
Status: Active » Closed (fixed)

Please open a new issue for these questions, and try to describe your problems more specifically.
Please include the following when describing the test cases:
-Extra modules used in the case (i.e. select_or_other)
-Step-by-step description of the current behavior with a simple example including the exact PDF output
-Desired PDF output or behavior.

Thanks for your time to do that. Setting this to fixed to close this issue.

mr.york’s picture

mr.york commented
Status: Closed (fixed) » Fixed
pierog1971’s picture

pierog1971 commented

Thank you. I will do that.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.