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
Fix setter methods in Log comment trait #93
Conversation
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.
To prevent potential data loss please revert the field rename.
The only problem I see with the code was the getter.
I suspect everything did actually work apart from the UI updating.
this.sudoUser = sudoUser; | ||
} |
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.
indents off
this.sudoUser = sudoUser; | |
} | |
this.sudoUser = sudoUser; | |
} |
@@ -15,7 +15,7 @@ | |||
@NonNull | |||
private String sudoUser = ""; | |||
|
|||
private boolean logSuccess = false; | |||
private boolean logSuccessOnly; |
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.
Revert the rename as you only needed to fix the getter actually.
@jetersen Thanks for noting that. I almost didn't consider data loss. 😓 |
Log comment trait's fields were not final so the object values and the serialised values were different.Update: The databoundsetter methods were not properly named.