Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor messaging errors #1556

Closed
dazey3 opened this issue Jun 27, 2022 · 0 comments · Fixed by #1558, #1559, #1560 or #1557
Closed

Minor messaging errors #1556

dazey3 opened this issue Jun 27, 2022 · 0 comments · Fixed by #1558, #1559, #1560 or #1557

Comments

@dazey3
Copy link
Contributor

dazey3 commented Jun 27, 2022

Looking through org.eclipse.persistence.internal.localization and org.eclipse.persistence.internal.localization.i18n, there are a couple errors/mistakes I would like to rectify. These were reported by our internal translation team

{ "staticweave_commandline_help_message_17of19", " To weave all entites contained in C:\\\\foo-source.jar with its persistence.xml contained within the C:\\\\foo-containing-persistence-xml.jar," },
{ "staticweave_commandline_help_message_18of19", " and output to C:\\\\foo-target.jar:" },
{ "staticweave_commandline_help_message_19of19", " StaticWeave -persistenceinfo C:\\\\foo-containing-persistence-xml.jar -classpath C:\\\\classpath1;C:\\\\classpath2 C:\\\\foo-source.jar C:\\\\foo-target.jar"},

Too many escape characters. The correct string should read:

"    StaticWeave -persistenceinfo C:\foo-containing-persistence-xml.jar -classpath C:\classpath1;C:\classpath2 C:\foo-source.jar C:\foo-target.jar"

{ "7343", "Multiple VPD identifiers (tenant discriminator context property) have been specified. Entity [{1}] uses [{0}] and Entity [{3]} uses [{2}]. When using a Multitenant VPD strategy, there can only be one tenant discriminator column per entity and its context property must be consistent across all the Multitenant VPD entities."},

typo. [{3]} -> [{3}]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment