sialcasa / mvvmFX

an Application Framework for implementing the MVVM Pattern with JavaFX
Apache License 2.0
489 stars 104 forks source link

#539 Potential bug in ValidationVisualizerBase #541

Closed ntherrien closed 6 years ago

ntherrien commented 6 years ago

In my original analysis, I thought that if we were looping on changes, we should be doing something with each change object "c".

After further analysis, I have determined that the while loop isn't necessary. The purpose of the change listener is to update the validation message on the UI. Therefore, it only needs to be updated once with the highest priority message.

Conclusion: Removed the while loop.