Item9988: delRev fails because the logger code change was missed

pencil
Priority: Normal
Current State: Closed
Released In: 1.1.3
Target Release: patch
Applies To: Engine
Component:
Branches:
Reported By: KennethLavrsen
Waiting For:
Last Change By: KennethLavrsen
1.1.2 is less than 8 hours old when I find the next stupid bug

This time I get spammed and I want to delRev

And it fails because a CODE REFACTORING of logger was not done properly.

I have done nothing the past weeks than fixing code refactoring bugs.

We do not test code refactorings enough. We write a few corner cases in unit tests but completely fails running through all the basic features in a browser.

On top of it, this bug could have been detected by the person that changed the code by doing a proper search in all .pm files for the old sub name.

We need to get the code quality increased. We cannot continue like this.

Fix is simple and easy and I am checking it in now.

-- KennethLavrsen - 10 Nov 2010

 

ItemTemplate edit

Summary delRev fails because the logger code change was missed
ReportedBy KennethLavrsen
Codebase 1.1.2
SVN Range
AppliesTo Engine
Component
Priority Normal
CurrentState Closed
WaitingFor
Checkins distro:bdc1cc7b0622 distro:34e1b99fb0eb
TargetRelease patch
ReleasedIn 1.1.3
Topic revision: r5 - 16 Apr 2011, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy