From 8d1d4ed2d7cce6ad1b0160b7737f2e48569b5aba Mon Sep 17 00:00:00 2001 From: "Dr. Raef" Date: Thu, 12 Jan 2017 18:44:22 -0500 Subject: [PATCH] Aged Brie quality increases by 2 When SellIn <=0 "Aged Brie" Quality increases by 1 as SellIn decreases. Line 78 to 84 will make "Aged Brie" Quality increases by 1 . A unit test for "Aged Brie" that has a SellIn value of 0 or less will result in a quality increase of 2. That is incorrect since the requirement states that "Aged Brie" Quality increases by 1 as SellIn approcahes. Lines 78 to 84 have been commented out to rectify the problem. --- csharp/GildedRose.cs | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/csharp/GildedRose.cs b/csharp/GildedRose.cs index a0017b2e..3712b882 100644 --- a/csharp/GildedRose.cs +++ b/csharp/GildedRose.cs @@ -75,15 +75,15 @@ namespace GildedRose Items[i].Quality = Items[i].Quality - Items[i].Quality; } } - else - { - if (Items[i].Quality < 50) - { - Items[i].Quality = Items[i].Quality + 1; - } - } - } - } + //else + //{ + // if (Items[i].Quality < 50) + // { + // Items[i].Quality = Items[i].Quality + 1; + // } + //} + } + } } }