Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It should be good to have the possibility of creating a REST export when you create a new view in admin/structure/views/add
Actually we can create a Page and a Block but no a REST export.
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff.txt | 2.52 KB | pfrenssen |
#8 | 2317085-8-views-quick_rest_export-ignore_whitespace-do-not-test.patch | 11.87 KB | pfrenssen |
#8 | 2317085-8-views-quick_rest_export.patch | 15.4 KB | pfrenssen |
Comments
Comment #1
pfrenssenComment #2
pfrenssenHere's an initial implementation. No tests yet.
There are two versions of the patch, one of which is ignoring whitespace changes. I had to indent a block of code, so this makes it a bit easier to review.
Comment #4
pfrenssenAdded a test. Added a second patch that ignores whitespace again to make it easier to review.
Comment #5
dawehnerwow!
Comment #6
pfrenssenComment #7
alexpottREST export vs REST API - i think we should be consistent and always call this "REST export" since that chimes with what is already in views.
Comment #8
pfrenssenGood point, thanks!
Comment #9
dawehnerYou could argue that it would be nice to use $this->t() instead of t() directly if we partly change this file already.
Comment #10
dawehnerManual tests worked fine, I don't consider #9 as a blocker given in which state that file is.
Thank for pieter!
Comment #11
alexpottCommitted 887eea4 and pushed to 8.0.x. Thanks!