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
#409 Aded UOM conversion check when adding a new price to a product. #6008
Conversation
{ | ||
return; | ||
} | ||
throw new AdempiereException("UOM Conversion to the selected UOM doesn't exist").markAsUserValidationError(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Create an AD_Message
record with translations (we'll need en_US, de_DE and de_CH) and use IMsg (Services.get(IMsg.class)
) to get a translatable string instance.
Passing hardcoded strings generally doesn't make sense for user-errors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fully agree
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pls check my comment. should be a quick fix
de.metas.business/src/main/java/de/metas/pricing/service/ProductPrices.java
Show resolved
Hide resolved
Here is an overview of what got changed by this pull request: Complexity increasing per file
==============================
- de.metas.business/src/main/java/de/metas/pricing/service/ProductPrices.java 2
See the complete overview on Codacy |
#6007