Add the ability to set a profile as default

PR https://github.com/bojanz/profile/pull/24

Posting patch of PR.

+++ b/src/Entity/Profile.php
@@ -260,4 +265,35 @@ public function getCacheTagsToInvalidate() {
+    $profiles = \Drupal\node\Entity\Node::loadMultiple();
+
+    // Ensure that all other profiles are set to not default.
+    foreach ($profiles as $profile) {
+      if ($profile->profile_id != $this->profile_id) {
+        $profile->setDefault(0);
+      }
+    }

You're loading nodes here, and not saving the expected profile.

Here is patch with fixes. First check to make sure entity being saved is default, load properly.

Checking the failing tests.

This is throwing some plugin not found exception, not sure about this error - will have to look into it.

+++ b/config/install/profile.type.main.yml
@@ -4,4 +4,5 @@ registration: false
+status: false

+++ b/config/schema/profile.schema.yml
@@ -26,6 +26,9 @@ profile.type.*:
+    default_profile:
+      type: boolean
+      label: 'Default'

We do not need to define "default" here, because its schema is defined within base field definitions.

Can we try it without the postSave and see if that's the issue? Help narrow it down.

Yes I tried without postSave, still getting the same error. I am trying to compare the new added with existing status profile field, I am checking what we are missing.

Here's an updated patch. Moved "default" to be "primary" since default is reserved. Tried to set default value.

Randomly on UI side clearing caches fixed it.

Giving my patch a shot, seemed to pass OK on Travis.. https://travis-ci.org/mglaman/profile/builds/98537590

Ok, so since my patch did pass, here is interdiff.

Fix some comment mishaps.

Thanks for all the help, subhojit777!

+++ b/src/Entity/Profile.php
@@ -259,4 +279,26 @@ class Profile extends ContentEntityBase implements ProfileInterface {
+          $profile->save();

This produces a call to itself :p

Are you sure?

+    // Check if this profile is, or became the default.
+    if ($this->isDefault()) {
+      /** @var \Drupal\profile\Entity\ProfileInterface[] $profiles */
+      $profiles = $storage->loadMultipleByUser($this->getOwner(), $this->getType());
+
+      // Ensure that all other profiles are set to not default.
+      foreach ($profiles as $profile) {
+        if ($profile->id() != $this->id()) {
+          $profile->setDefault(FALSE);
+          $profile->save();
+        }
+      }
+    }

We're looping other other profile entities who have been marked as not default, which would miss this logic.

This is working as expected. There was mistake in my code logic itself :p