[BUG] State is advanced for no apparent reason

[lakhoune]

In the case where we trigger a function call and return, the bot manager will advance the state if there is an empty intent following the last state. This is done in the following line:

las2peer-social-bot-manager-service/social-bot-manager/src/main/java/i5/las2peer/services/socialBotManagerService/model/Messenger.java

Lines 292 to 297 in abfaf35

	} else if (state.getFollowingMessages().get("") != null) {
	// check whether bot action needs to be triggered without user input
	state = state.getFollowingMessages().get("");
	stateMap.put(channel, state);
	this.previousStateInConversation.put(channel, state);

This breaks the current implementation of a default path that should be taken if no other intent matches. As an example for the mensa bot: If you want to add a review the bot asks you to either write a comment or say "no". In that case every intent other than rejection should go on that path. And to allow for any intent to be recognized the field is left empty. But the actual behavior is that the state is just skipped and users cannot make any comment