In sf_user and sf_node, there are a few usages of print_r() without using the HTML < pre > tags.

Patch coming.

CommentFileSizeAuthor
#3 1118712-bettermessages-3.patch3.38 KBkostajh
#1 1118712-BetterMessages.patch3.25 KBdpearcefl

Comments

dpearcefl’s picture

dpearcefl commented
StatusFileSize
new 1118712-BetterMessages.patch3.25 KB

Patch attached.

dpearcefl’s picture

dpearcefl commented
Status: Active » Needs review
kostajh’s picture

kostajh commented
StatusFileSize
new 1118712-bettermessages-3.patch3.38 KB

Thanks for working on this. I made a few changes to wrap strings in drupal_set_message with t(), and also to use placeholders for displaying the response.

kostajh’s picture

kostajh commented
Status: Needs review » Fixed

This is in 6.x-2.x-dev. Thanks!

Commit: http://drupalcode.org/project/salesforce.git/commit/5af9f14

Status: Fixed » Closed (fixed)

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