#12 pulished tests

siwapp · Feb 10, 2012 · c004a5f · c004a5f
1 parent 0239974
commit c004a5f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/Siwapp/CoreBundle/Entity/AbstractInvoice.php b/src/Siwapp/CoreBundle/Entity/AbstractInvoice.php
@@ -529,11 +529,12 @@ public function removeItem($mixed)
     {
         if($mixed instanceof \Siwapp\CoreBundle\Entity\AbstractItem)
         {
-            foreach($this->getItems() as $ref => $item)
+            $items = $this->getItems();
+            foreach($items as $ref => $item)
             {
                 if($item === $mixed)
                 {
-                    unset($this->items[$ref]);
+                    unset($items[$ref]);
                     break;
                 }
             }

diff --git a/src/Siwapp/InvoiceBundle/Tests/Model/InvoiceTest.php b/src/Siwapp/InvoiceBundle/Tests/Model/InvoiceTest.php
@@ -34,7 +34,7 @@ public function testTrial()
         $new_item->setQuantity(1);
         $new_item->setDiscount(0);
         $this->em->persist($new_item);
-        $test_invoice->addItem($new_item);
+        $test_invoice->addNewItem($new_item);
         $this->em->flush();
         $this->assertEquals($test_invoice->getGrossAmount(),8172.36);
         // recalculate when updating item