Given I'm on a dev/local site and don't have Xero integration, I don't want to blow up the site when xero_php_load() fails to catch XeroException provided by xero.php.

CommentFileSizeAuthor
#1 xero-2357349-catch-exception.patch1.19 KBmradcliffe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
FileSize
xero-2357349-catch-exception.patch1.19 KB
mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Status: Active » Needs review
mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe at Kosada commented
Status: Needs review » Fixed

Fixed some issues and committed after testing.

  • mradcliffe committed c096e1a on 7.x-1.x 
    Issue #2357349 by mradcliffe: Add try/catch around xero_php_load() to...

Status: Fixed » Closed (fixed)

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