The watchdog wrapper in the DrupalSalesforce object isn't properly passing error messages to the watchdog function. A very quick-fix style patch to this is attached. However, it looks like this call to watchdog is leftover from a 5.x upgrade as the strings that are passed to it are passed through t().

CommentFileSizeAuthor
sf_fix_log.patch659 bytesjmiccolis

Comments

Bevan’s picture

Bevan commented
Status: Active » Needs review

Makes sense, though I hadn't noticed this bug. I (or someone other than the author) needs to test for it, then check the fix. Once RTBC I'll commit.

Jonah Ellison’s picture

Jonah Ellison commented
Priority: Normal » Critical
Status: Needs review » Reviewed & tested by the community

Verified and tested. Please implement this. It would have saved me a lot of time and headache if I was able to see what was wrong with my server without having to hack/debug this module.

Bevan’s picture

Bevan commented
Status: Reviewed & tested by the community » Fixed

Fixed in DRUPAL-6--2 by using watchdog according to Drupal 6 standards. See http://drupal.org/cvs?commit=245212

Bevan’s picture

Bevan commented

Removed

Bevan’s picture

Bevan commented
Status: Fixed » Active

Wrong node

Bevan’s picture

Bevan commented
Status: Active » Fixed

Still wrong node

Status: Fixed » Closed (fixed)

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