From 3d359446137dccd47001c2acef4e17f310a509a8 Mon Sep 17 00:00:00 2001 From: Marc Ducobu Date: Thu, 5 Mar 2015 18:00:07 +0100 Subject: [PATCH] Refactoring : resolving netbeans warnings --- Entity/Person.php | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Entity/Person.php b/Entity/Person.php index 33d68ad84..2212ff311 100644 --- a/Entity/Person.php +++ b/Entity/Person.php @@ -126,7 +126,7 @@ class Person { /** * - * @param \Chill\PersonBundle\Entity\AccompanyingPeriod $period + * @param \Chill\PersonBundle\Entity\AccompanyingPeriod $accompanyingPeriod * @uses AccompanyingPeriod::setPerson */ public function addAccompanyingPeriod(AccompanyingPeriod $accompanyingPeriod) { @@ -164,7 +164,7 @@ class Person { * * To check if the Person and its accompanying period are consistent, use validation. * - * @param AccompanyingPeriod + * @param accompanyingPeriod * @throws \Exception if two lines of the accompanying period are open. */ public function close(AccompanyingPeriod $accompanyingPeriod) @@ -596,9 +596,7 @@ class Person { public function checkAccompanyingPeriodIsNotCovering() { $periods = $this->getAccompanyingPeriodsOrdered(); - - $i = 0; - + foreach ($periods as $key => $period) { //accompanying period is open : we must check the arent any period after if ($period->isOpen()) {