Model Validator registered multiple times

Description

Hi,

i am playing little bit with Replication feature. I notices that there is one major bug related to model validator - model validator on Client is registered to validation engine multiple times instead of once. I think this is not bug in Implementation of ExportModelValidator, but ModelValidatorEngine, because it looks like model validators are not removed from validation engine when user changes role or log out, but when user is logged again. validator is registers again too.

Steps to reproduce:
1. In Client window set org.adempiere.ExportModelValidator as model validation class
2. Rester your server to apply this change
3. Login into system
4. Change role and log in with same credentials

These steps cause that one Model Validator is registered multiple times in Model Validation Engine and modelChange method is executed multiple times = BUG.

Bug can be also seen when you try to debug addModelChange method of ModelValidationEngine class.

Environment

None

Attachments

1
  • 10 Dec 2014, 12:00 AM

Activity

Show:

Daniel Tamm December 27, 2014 at 11:24 AM

Thanks for catching and fixing this slightly smiling face Had the same problem in my replication setup... after a while the ExportModelValidator was registered so many times I had to restart everything not to choke the JMS topic with lots of duplicate messages. I'm very happy for the fix slightly smiling face Great work!

Tomáš Švikruha December 25, 2014 at 5:05 PM

I've not found any other issues, so I think yes slightly smiling face

Carlos Ruiz December 23, 2014 at 8:53 PM

, how is this going? ready to be integrated?

Tomáš Švikruha December 10, 2014 at 7:19 AM
Edited

I will continue with testing and if there will be some new issues/bugs I will let you know.

Thanks for your time slightly smiling face

Carlos Ruiz December 10, 2014 at 12:00 AM

ok , third version slightly smiling face
hope third time's a charm

Fixed

Details

Assignee

Reporter

Labels

Priority

Created December 5, 2014 at 9:00 AM
Updated May 4, 2015 at 5:43 PM
Resolved December 26, 2014 at 4:03 PM