Switch REST to default to HAL

+++ b/core/modules/rest/lib/Drupal/rest/Plugin/rest/resource/EntityResource.php
@@ -155,6 +155,11 @@ public function patch($id, EntityInterface $entity) {
+      // Requests cannot overwrite id or uuid, so skip them.
+      if (in_array($name, array('id', 'uuid'))) {
+        continue;
+      }

why is the UUID protected from writing? I can see that most use cases will never do that, but why artificially restricting it?

Otherwise this looks good!

If we can't rely on uuid not changing, we can't build any functionality based off of uuid being, er, universally unique. If you want an object to have a different uuid... make a new object. (If we allow it, someone somewhere is going to change it and then we'll never be able to rely on the uuid again.)

The for-review patch is dead simple and seems fine. If anything, I'd ask if we should be centralizing our definition of default so that we don't have to restate it in 50 places.

The dependent issue is still CNW, though, so we can't RTBC this one.

#5: 1935538-05-rest-test-update.patch queued for re-testing.

Probably just need to upload the .txt file in #5 with a .patch extension.

xpost.

Just a silly typo fix.

This just removes hunks unrelated to this issue. I moved them into #1931976-28: Support deserialization for hal+json (needs more language handling tests). I'll follow up with one more reroll for #6.

If anything, I'd ask if we should be centralizing our definition of default so that we don't have to restate it in 50 places.

Agreed. Done here. This changes nearly every line so no point in an interdiff.

Thanks for fixing tests

Oh, JSON-LD. We blow a trumpet for thee. Do we need another patch to remove the module as a whole?

In the meantime, this looks like it cleans up an awful lot of hard-coding as well as switching to a more recognized(?) format, so woohoo.

Committed and pushed to 8.x. Thanks!

Actually. I imagine this will require a change to a change notice somewhere, else a new one to be made if it doesn't already exist.

Moved the change notice requirement to #1816354-43: Add a REST module, starting with DELETE.