[5.x] Add failing test for issue caused by PR #9585 (#10061)

Co-authored-by: John Koster <john@stillat.com>

src/Fields/Blueprint.php

@@ -514,7 +514,7 @@ public function ensureFieldInTab($handle, $config, $tab, $prepend = false)
 
         $this->ensuredFields[$handle] = compact('handle', 'tab', 'prepend', 'config');
 
-        $this->resetFieldsCache();
+        $this->resetBlueprintCache()->resetFieldsCache();
 
         return $this;
     }
@@ -573,7 +573,7 @@ public function removeTab($handle)
 
         Arr::pull($this->contents['tabs'], $handle);
 
-        return $this->resetFieldsCache();
+        return $this->resetBlueprintCache()->resetFieldsCache();
     }
 
     public function removeFieldFromTab($handle, $tab)
@@ -591,7 +591,7 @@ public function removeFieldFromTab($handle, $tab)
         // Pull it out.
         Arr::pull($this->contents['tabs'][$tab]['sections'][$sectionIndex]['fields'], $fieldKey);
 
-        return $this->resetFieldsCache();
+        return $this->resetBlueprintCache()->resetFieldsCache();
     }
 
     private function getTabFields($tab)
@@ -621,7 +621,7 @@ protected function ensureFieldInTabHasConfig($handle, $tab, $config)
         // Merge in new field config.
         $this->contents['tabs'][$tab]['sections'][$sectionKey]['fields'][$fieldKey]['field'] = array_merge($existingConfig, $config);
 
-        return $this->resetFieldsCache();
+        return $this->resetBlueprintCache()->resetFieldsCache();
     }
 
     public function validateUniqueHandles()
@@ -635,6 +635,13 @@ public function validateUniqueHandles()
         }
     }
 
+    protected function resetBlueprintCache()
+    {
+        $this->lastBlueprintHandle = null;
+
+        return $this;
+    }
+
     protected function resetFieldsCache()
     {
         if ($this->parent) {

tests/Fields/BlueprintTest.php

@@ -11,6 +11,7 @@
 use Statamic\Contracts\Query\QueryableValue;
 use Statamic\CP\Column;
 use Statamic\CP\Columns;
+use Statamic\Entries\Entry;
 use Statamic\Events\BlueprintCreated;
 use Statamic\Events\BlueprintCreating;
 use Statamic\Events\BlueprintDeleted;
@@ -783,6 +784,29 @@ public function it_ensures_a_field_exists_in_a_specific_tab()
         $this->assertEquals(['type' => 'textarea'], $blueprint->fields()->get('new')->config());
     }
 
+    /** @test */
+    public function it_can_add_fields_multiple_times()
+    {
+        $blueprint = (new Blueprint)
+            ->setNamespace('collections/collection_one')
+            ->setHandle('blueprint_one');
+
+        $entry = (new Entry)
+            ->collection('collection_one')
+            ->blueprint($blueprint);
+
+        $blueprint->setParent($entry);
+
+        $blueprint->ensureFieldsInTab(['field_one' => ['type' => 'text']], 'tab_one');
+
+        $this->assertTrue($blueprint->hasField('field_one'));
+
+        $blueprint->ensureField('field_two', ['type' => 'textarea']);
+
+        $this->assertTrue($blueprint->hasField('field_two'));
+
+    }
+
     /** @test */
     public function it_ensures_a_field_has_config()
     {