search

GravityKit / GravityExport-Lite

GravityExport Lite. Export all Gravity Forms entries to Excel (.xlsx) via a secret (shareable) url.
https://gfexcel.com
GNU General Public License v2.0
12 stars 8 forks source link

Prevents notification error from bubbling up to end user. #166

Closed doekenorg closed 1 year ago

doekenorg commented 1 year ago

This PR resolves an issue where the notification could not be stored, and this exception bubbles up to the end user. https://wordpress.org/support/topic/php-fatal-error-notifications-could-not-be-stored/#post-16788286

I'm catching the exception, and re-throwing it as a MigrationException; because these are simply logged.

However, since this exception isn't breaking the migration itself (just the storing of the notification about it) I added a NonBreakingMigrationException which will be caught earlier and logged; but will not break the migration process.