Remove overloaded State constructors as they add more complexity and confusion

[gvagenas]

The overloaded constructors of State :

• https://github.com/Mobicents/RestComm/blob/119d58376c31cc63e73b0dbe1a9143ae990d5174/restcomm/restcomm.commons/src/main/java/org/mobicents/servlet/restcomm/fsm/State.java#L45-L45
• https://github.com/Mobicents/RestComm/blob/119d58376c31cc63e73b0dbe1a9143ae990d5174/restcomm/restcomm.commons/src/main/java/org/mobicents/servlet/restcomm/fsm/State.java#L49

doesn't make things clear about when we want an ActionOnEnter or ActionOnExit or ActionOnState.

We should remove the overloaded constructors and keep just the first one.

[steve07728]

I converted all the code using the overloaded constructors to use the base constructor and removed the overloaded constructors.

[gvagenas]

@steve07728 Thanks for the update. Will check your work as soon as possible