Closed (fixed)
Project:
Drupal core
Version:
8.3.x-dev
Component:
big_pipe.module
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
14 Dec 2016 at 13:52 UTC
Updated:
4 Jan 2017 at 10:35 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
wim leersComment #3
effulgentsia commentedIn core, we also have
RenderCacheInterface, which is marked @internal, andLinkGeneratorInterface, where each of its methods is marked @internal. Are we saying, we should in principle remove those interfaces too? Not this issue's scope; just curious if the same logic behind this patch applies broadly to core.Comment #4
wim leersI think you're right, those had interfaces created for the sake of creating interfaces. There was a time when we were pretty much blindly extracting interfaces out of classes. Doing it blindly was probably wrong. But now we can't change that.
We can still extract an interface later — that's equivalent with adding an API. But we can never remove an API. Hence this approach makes sense.
A different angle:
Comment #5
fabianx commentedRTBC - I agree, interfaces just for the sake of interfaces makes no sense.
Comment #6
wim leersPreferably, this gets committed after #2657684: Refactor BigPipe internals to allow a contrib module to extend BigPipe with the ability to stream anonymous responses and prime Page Cache for subsequent visits, because that one is much harder to reroll.
Comment #7
wim leers#2657684: Refactor BigPipe internals to allow a contrib module to extend BigPipe with the ability to stream anonymous responses and prime Page Cache for subsequent visits is in! Rebasing #2…
Comment #8
wim leersComment #9
wim leersOops, this is the right patch.
Comment #11
wim leersComment #12
alexpottCommitted eea7ce5 and pushed to 8.3.x. Thanks!
Comment #14
wim leersYay, this unblocked #2797169: Mark BigPipe as stable/non-experimental!