Refactoring : resolving netbeans warnings

This commit is contained in:
Marc Ducobu 2015-03-05 18:00:07 +01:00
parent 2dc2479304
commit 3d35944613

View File

@ -126,7 +126,7 @@ class Person {
/** /**
* *
* @param \Chill\PersonBundle\Entity\AccompanyingPeriod $period * @param \Chill\PersonBundle\Entity\AccompanyingPeriod $accompanyingPeriod
* @uses AccompanyingPeriod::setPerson * @uses AccompanyingPeriod::setPerson
*/ */
public function addAccompanyingPeriod(AccompanyingPeriod $accompanyingPeriod) { public function addAccompanyingPeriod(AccompanyingPeriod $accompanyingPeriod) {
@ -164,7 +164,7 @@ class Person {
* *
* To check if the Person and its accompanying period are consistent, use validation. * 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. * @throws \Exception if two lines of the accompanying period are open.
*/ */
public function close(AccompanyingPeriod $accompanyingPeriod) public function close(AccompanyingPeriod $accompanyingPeriod)
@ -597,8 +597,6 @@ class Person {
{ {
$periods = $this->getAccompanyingPeriodsOrdered(); $periods = $this->getAccompanyingPeriodsOrdered();
$i = 0;
foreach ($periods as $key => $period) { foreach ($periods as $key => $period) {
//accompanying period is open : we must check the arent any period after //accompanying period is open : we must check the arent any period after
if ($period->isOpen()) { if ($period->isOpen()) {