Add $set_defaults parameter to DatabaseExportableController create() method

+++ b/lib/Drupal/ctools/DatabaseExportableController.phpundefined
@@ -310,12 +310,17 @@ class DatabaseExportableController extends ExportableControllerBase {
+        if ($set_defaults) {
+          $data[$field] = !empty($info['default']) ? $info['default'] : NULL;
+        }
+        else {
+          $data[$field] = NULL;

I think the ternary is still fine:
$data[$field] = ($set_defaults && !empty($info['default'])) ? $info['default'] : NULL;

This looks fine now

     foreach ($this->schema['fields'] as $field => $info) {
       // Get a default if nothing exists.
       if (!isset($data[$field])) {
-        $data[$field] = !empty($info['default']) ? $info['default'] : NULL;
+        $data[$field] = ($set_defaults && !empty($info['default'])) ? $info['default'] : NULL;
       }
     }

This sets a key to NULL if it has a default but we chose not to set defaults, but does not set the key if we neglected to give it a default. We should set all schema keys regardless of whether it has an explicit default.

  $default = NULL;
  if ($set_defaults && !empty($info['default'])) {
    $default = $info['default'];
  }

Also we seem to have lost the 'object default' I had. This existed because longtext and blob cannot have defaults in SQL but sometimes need defaults, so I created a non sql key as a workaround. We should probably retain that as well. Perhaps with a comment so that other people understand why I did that. ;)

Marking this active for this adjustment.

The 8.x-1.x branch has been abandoned, 8.x-2.x is being rewritten from scratch, so this is no longer relevant.

Feel free to reopen if there's a project it's still appropriate for.