Form switching

rasa/core/actions/forms.py

@@ -264,7 +264,7 @@ def extract_other_slots(
         if they are set by corresponding entities from the user input
         else return `None`.
         """
-        slot_to_fill = tracker.get_slot(REQUESTED_SLOT)
+        slot_to_fill = self.get_slot_to_fill(tracker)
 
         entity_type_of_slot_to_fill = self._get_entity_type_of_slot_to_fill(
             slot_to_fill, domain
@@ -317,13 +317,26 @@ def extract_other_slots(
 
         return slot_values
 
+    def get_slot_to_fill(self, tracker: "DialogueStateTracker") -> Optional[str]:
+        """Get the slot to be filled.
+
+        When switching to another form, the requested slot setting is still from the
+        previous form and must be ignored.
+
+        Returns:
+        The slot name or None
+        """
+        return (
+            tracker.get_slot(REQUESTED_SLOT)
+            if tracker.active_loop_name == self.name()
+            else None
+        )
+
     def extract_requested_slot(
         self, tracker: "DialogueStateTracker", domain: Domain
     ) -> Dict[Text, Any]:
-        """Extract the value of requested slot from a user input
-        else return `None`.
-        """
-        slot_to_fill = tracker.get_slot(REQUESTED_SLOT)
+        """Extract the value of requested slot from a user input else return `None`."""
+        slot_to_fill = self.get_slot_to_fill(tracker)
         logger.debug(f"Trying to extract requested slot '{slot_to_fill}' ...")
 
         # get mapping for requested slot
@@ -446,7 +459,7 @@ async def validate(
         slot_values = self.extract_other_slots(tracker, domain)
 
         # extract requested slot
-        slot_to_fill = tracker.get_slot(REQUESTED_SLOT)
+        slot_to_fill = self.get_slot_to_fill(tracker)
         if slot_to_fill:
             slot_values.update(self.extract_requested_slot(tracker, domain))
 
@@ -461,14 +474,7 @@ async def validate(
             # to be filled by the user.
             if isinstance(event, SlotSet) and not event.key == REQUESTED_SLOT
         )
-        user_rejected_manually = any(
-            isinstance(event, ActionExecutionRejected) for event in validation_events
-        )
-        if (
-            slot_to_fill
-            and not some_slots_were_validated
-            and not user_rejected_manually
-        ):
+        if slot_to_fill and not some_slots_were_validated:
             # reject to execute the form action
             # if some slot was requested but nothing was extracted
             # it will allow other policies to predict another action