Never Hardcode If Not Necessary
Hardcoded names or properties are hard to maintain and make code difficult to
extend. Workflow state names for example should always use an accessor like
shown below:
Good Examples:
if self.getSimulationState() in self.getPortalDraftOrderStateList():
Bad Examples:
if self.getSimulationState() == "draft":