Problem/Motivation
Let's assume we have a rest route on /node
similar to what #2293697: EntityResource POST routes all use the confusing default: use entity types' https://www.drupal.org/link-relations/create link template if available does.
Currently views overrides this route on /node
with its own internal controller no matter what
Proposed resolution
Just add it, if needed.
Remaining tasks
Figure out whether we want to filter by more than request verb. What about _format
etc.
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#2 | interdiff.txt | 911 bytes | dawehner |
#2 | 2730497-2.patch | 3.03 KB | dawehner |
Comments
Comment #2
dawehnerComment #3
dawehner.
Comment #4
dawehnerComment #5
Wim LeersI think that can happen in a follow-up. This solves a big frustration/WTF.
Comment #6
catchCommitted 9b650e9 and pushed to 8.2.x. Thanks!
Comment #9
Wim LeersThis actually was an incredibly incomplete fix. This was a small step in the right direction, but me saying in #5 that we could defer the
_format
stuff was actually a mistake now. (Although it did make the situation better in some cases.) We really need to fix that.Comment #10
Wim LeersAlso, this was not pushed to 8.1.
Comment #11
Wim LeersSo, that follow-up mentioned in #5 and #9 now is #2772537-11: REST Views override existing REST GET routes.
Comment #12
Wim LeersI'd reclose this, but this really should be
committedcherry-picked to 8.1 also.Comment #14
dawehnerI agree to be honest. Its quite a clusterfuck for actual sites.
Comment #15
Wim LeersComment #16
dawehnerLet's do a strict comparison here ...
I still like that we have a unit test here :)
Comment #17
Wim Leers#16: we just need to cherry-pick the commit from 8.2 to 8.1, we don't want to change it here I think?
Comment #18
dawehnerWell, I guess i'm a bit stricter with my own patches, but sure, we should just cherry pick, I haven't seen that.
Comment #20
catchCherry picked to 8.1.x.
Agreed on the strict comparison - I'd have asked for that if I'd spotted it. Let's do that in a follow-up though.
Comment #21
Wim LeersYes! And we now have that follow-up: #2772537: REST Views override existing REST GET routes.