Most exception classes of Feeds extend RuntimeException. It would be more logical to have a generic FeedsRuntimeException and let the exception classes extend that class.

CommentFileSizeAuthor
#2 feeds-runtime-exception-2868065-2.patch3.58 KBMegaChriz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MegaChriz created an issue. See original summary.

MegaChriz’s picture

MegaChriz CreditAttribution: MegaChriz as a volunteer commented
Status: Active » Needs review
FileSize
feeds-runtime-exception-2868065-2.patch3.58 KB

Also removed the class InterfaceNotImplementedException, which isn't thrown anywhere.

MegaChriz’s picture

MegaChriz CreditAttribution: MegaChriz as a volunteer commented
Status: Needs review » Fixed

Committed #2.

  • MegaChriz committed 1325ad4 on 8.x-3.x 
    Issue #2868065 by MegaChriz: Create generic FeedsRuntimeException

Status: Fixed » Closed (fixed)

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