Comments

joachim created an issue. See original summary.

Saviktor’s picture

Status: Active » Needs review
FileSize
670 bytes

I hope I didn't break the logical meaning of this doc comment

claudiu.cristea’s picture

Status: Needs review » Reviewed & tested by the community
joachim’s picture

Status: Reviewed & tested by the community » Needs work

> + * Returns the extension the derivative after applying this image effect.

Doesn't quite read right.

duozersk’s picture

Status: Needs work » Needs review
FileSize
673 bytes
566 bytes
somepal’s picture

Status: Needs review » Reviewed & tested by the community

Last patch good to go.. reviewed

xjm’s picture

Title: first line of docs for ImageEffectInterface::getDerivativeExtension() overruns » first line of docs for ImageEffectInterface::getDerivativeExtension() is too long
Version: 8.5.x-dev » 8.4.x-dev
xjm’s picture

Title: first line of docs for ImageEffectInterface::getDerivativeExtension() is too long » The one-line summary of ImageEffectInterface::getDerivativeExtension() is too long

  • xjm committed e59a03d on 8.5.x
    Issue #2923670 by duozersk, Saviktor, joachim: The one-line summary of...

  • xjm committed fab8ea6 on 8.4.x
    Issue #2923670 by duozersk, Saviktor, joachim: The one-line summary of...
xjm’s picture

Status: Reviewed & tested by the community » Fixed

Normally I'd close this as a duplicate of #2572709: Fix 'Drupal.Files.LineLength' coding standard, but getting a one-line summary under 80 characters requires rewriting it rather than rewrapping it, so in this case I think it's okay. (Note that normally though we should fix one coding standard rule across all of core and then enable the rule in the phpcs.xml.dist, rather than fixing a single instance.)

Committed and pushed to 8.5.x. Thanks! I also backported this to 8.4.x as a documentation fix.

Status: Fixed » Closed (fixed)

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