In core/lib/Drupal/Core/StreamWrapper/StreamWrapperInterface.php, there is a typo. The comments state the following:

 * Provides a Drupal interface and classes to implement PHP stream wrappers for
 * public, private, and temporary files. Extends the StreamWrapperInterface
 * with methods expected by Drupal stream wrapper classes.

However, the class extends PhpStreamWrapperInterface, not StreamWrapperInterface as is stated in the comment.

interface StreamWrapperInterface extends PhpStreamWrapperInterface {
CommentFileSizeAuthor
#2 core-typo_in_stream_wrapper_interface-2975451-2-d8.patch758 bytesJaypan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jaypan created an issue. See original summary.

Jaypan’s picture

Jaypan CreditAttribution: Jaypan commented
Issue summary: View changes
FileSize
core-typo_in_stream_wrapper_interface-2975451-2-d8.patch758 bytes

Patch attached.

Jaypan’s picture

Jaypan CreditAttribution: Jaypan commented
Status: Active » Needs review
timKruijsen’s picture

timKruijsen CreditAttribution: timKruijsen at Ordina Digital Services commented
Status: Needs review » Reviewed & tested by the community

Looks good to me

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Reviewed & tested by the community » Fixed

Good catch. Committed to 8.6.x and backported to 8.5.x as a documentation fix. Thanks for patching this!

  • xjm committed 3c5bf74 on 8.6.x 
    Issue #2975451 by Jaypan: Incorrect reference in StreamWrapperInterface....

  • xjm committed 41ac46c on 8.5.x 
    Issue #2975451 by Jaypan: Incorrect reference in StreamWrapperInterface....
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Version: 8.6.x-dev » 8.5.x-dev

Status: Fixed » Closed (fixed)

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