Problem/Motivation

Looks like this needs, at least, a summary line that is helpful.

/**
 * Defines an
 *
 * @see \Drupal\migrate\Plugin\RequirementsInterface
 */
class RequirementsException extends \RuntimeException {

Proposed resolution

Decide on a suitable summary line for this class.

Remaining tasks

Make a patch
Review

CommentFileSizeAuthor
#2 2937964-2.patch499 bytesjofitz

Comments

quietone created an issue. See original summary.

jofitz’s picture

jofitz
he/him
commented
Status: Active » Needs review
StatusFileSize
new 2937964-2.patch499 bytes

Made a patch.

maxocub’s picture

maxocub commented
Status: Needs review » Reviewed & tested by the community

This looks good, thank you!

  • catch committed c0505dd on 8.6.x 
    Issue #2937964 by Jo Fitzgerald, quietone: RequirementException summary...

  • catch committed da5cca8 on 8.5.x 
    Issue #2937964 by Jo Fitzgerald, quietone: RequirementException summary...
catch’s picture

catch
he/him
Primary language English
commented
Version: 8.6.x-dev » 8.5.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!

Status: Fixed » Closed (fixed)

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