Limit available digits in ADE prefix text field

3dcitydb / plugin-ade-manager

Importer/Exporter plugin to dynamically extend a 3D City Database instance to store and manage CityGML Application Domain Extensions (ADEs)

Apache License 2.0

2 stars 0 forks source link

Limit available digits in ADE prefix text field #3

Closed FxKu closed 6 years ago

FxKu commented 6 years ago

I was curious if I could also use a looong prefix for an ADE and if it appears in the DDL scripts ;) Turns out the tool only allows 4 characters. This is good. But, maybe the textfield in the GUI should have the same behaviour to not surprise users. Btw, I haven't tested with numbers or special characters, yet ;)

yaozhihang commented 6 years ago

Thanks for the tests. Yes, we need some more hints for users when they interact with the GUI . I'll improve this soon.

yaozhihang commented 6 years ago

OK, fixed. see https://github.com/3dcitydb/plugin-ade-manager/commit/d72f63b85fba87c97b0ff78287685c7eb387181b