| Description |
{color:#1f497d}Hi KR,{color}
{color:#1f497d}Upon investigating the issue, I found out that there’s a glitch in our code in how we handle Audit Log, I haven’t applied the fix yet to their live and just planning to include this along w/ other fixes like the .Tiff thing and etc..{color}
{color:#1f497d}What happened was, we have this validation in Audit Log that we added way back when we were doing Load Testing w/ some of the modules and one of the issue we found before was whenever we create transactions(Contract, Invoice and etc..) through Load Testing tool, the same Audit Log entry goes to the original transaction.{color}
{color:#1f497d}Example:{color}
{color:#1f497d}We try creating multiple Contracts out of Contract – 1 so during load testing, depending on the speed and the duration it could create a bunch of Contracts, however, the Audit Log entries of those newly created Contracts would go to the original one w/c is Contract – 1 and it’s causing inconsistency w/ the performance analytics.{color}
{color:#1f497d}To solve the issue, we added a validation where we prevent the “same audit log entry” from being inserted.{color}
{color:#1f497d}However the problem now is in the case of Ralph, refer to screen shot below.{color}
{color:#1f497d} !ralph.png|height=726,width=917! {color}
{color:#1f497d}He was able to already do a Disabling/Enabling on June 6 and June 13 and so the Disabling/Enabling on June 20 was prevented from being inserted w/c is the bug.{color}
{color:#1f497d}On the other hand, irelyadmin user (me) was able to log Disabling/Enabling because I’m a different user than Ralph but was also been able to log the entries once for the same kind of entry.{color}
!irelyadmin.png|height=724,width=913!
{color:#1f497d}{color}
{color:#1f497d}So in other words here, if Ralph or Irelyadmin, disables/enables the user and do other stuff than that then it would still log properly because it’s technically a different entry as per the validation.{color}
{color:#1f497d}So now, I believe that the validation is already obsolete and must be removed from our code since it’s intended purpose was only for the Load Testing to return accurate performance analytics.{color}
{color:#1f497d}Thanks,{color}
|