State, transition names

[oyanyev]

State, transition names

[oyanyev]

PlantUml accepts titles for entities:
state InvalidBarcode as "Title"

• unique state.name is converted to entity itself and
• non-unique state.displayName to title

transition.displayName is converted to arrow description without any problem

[ayanyev]

Added PR from my personal Github account. Should not break any existing code.
#90

Published to local maven, added to my project - works.

[nsk90]

I dont think it is safe to relax "unique name" restriction.
APIs like findState(name: String, ...) if based on it.

But possibly we can upgrade exportToPlantUml() function to accept converters/beautifiers for names.
Converters/beautifiers are functions that take original state/transition/machine and return printable name.

So if your state names follow some schema like Block1.Group1.State1 you can simply drop the path and leave only State1 as printable name.

What do you think about it?

[oyanyev]

I have another idea:
displayName or umlName which can be defaulted to name, but can be overriden and will be used solely in umls

[nsk90]

yes, this is also possible, but this requires to change many APIs that are not related to the export feature.

What about plantUml itself, how it handles non unique names? I suppose it can have a problems with transitions directions?

[oyanyev]

I can create PR, if you do not see that it might have side effects

[nsk90]

looks this issue is related to #82

Some kind of meta data objects should cover both cases.
I do not see any side effects, yes you can open a PR, thank you.

[oyanyev]

Started with PR and already have one question.

internal class CheckUniqueNamesVisitor : RecursiveVisitor {
   ...
    private val transitionNames = mutableSetOf<String>()

   ...

    override fun <E : Event> visit(transition: Transition<E>) {
        transition.name?.let { check(transitionNames.add(it)) { "Transition name is not unique: $it" } }
    }
}

transition.name visitor has no sense so far. Name is used only there and in export to uml. Can this check be omitted?
If check not applicable, then name can be not unique and can be safely used for uml labels

[nsk90]

it is also used here, so the name behaviour should not change.

/**
 * Find transition by name. This might be used to start listening to transition after state machine setup.
 */
fun TransitionStateApi.findTransition(name: String) = transitions.find { it.name == name }

[nsk90]

Done in https://github.com/nsk90/kstatemachine/releases/tag/v0.27.0
Thanks, for the contribution!