Covivo/mobicoop

AdManager has 33 functions (exceeds 20 allowed). Consider refactoring.

Open

class AdManager
{
    public const AUTHORIZED_MATCHING_ALGORITHMS = [Ad::MATCHING_ALGORITHM_V2, Ad::MATCHING_ALGORITHM_V3];

    private $entityManager;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if (Criteria::FREQUENCY_REGULAR == $ad->getFrequency()) {
            if (
                is_null($ad->getSchedule())
                || (is_array($ad->getSchedule()) && 0 == count($ad->getSchedule()))
            ) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if (Criteria::FREQUENCY_REGULAR == $ad->getFrequency()) {
                if (
                    is_null($ad->getSchedule())
                    || (is_array($ad->getSchedule()) && 0 == count($ad->getSchedule()))
                ) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if (Criteria::FREQUENCY_REGULAR == $ad->getFrequency()) {
                $returnCriteria->setFrequency(Criteria::FREQUENCY_REGULAR);
                $returnCriteria->setToDate($ad->getReturnLimitDate() ? $ad->getReturnLimitDate() : null);
                $returnCriteria = $this->createTimesFromSchedule($ad->getSchedule(), $returnCriteria, 'returnTime', $marginDuration);
                $hasSchedule = $returnCriteria->isMonCheck() || $returnCriteria->isTueCheck()

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if (Criteria::FREQUENCY_REGULAR == $ad->getFrequency()) {
            $outwardCriteria->setFrequency(Criteria::FREQUENCY_REGULAR);
            $outwardCriteria->setToDate($ad->getOutwardLimitDate() ? $ad->getOutwardLimitDate() : null);
            $outwardCriteria = $this->createTimesFromSchedule($ad->getSchedule(), $outwardCriteria, 'outwardTime', $marginDuration);
            $hasSchedule = $outwardCriteria->isMonCheck() || $outwardCriteria->isTueCheck()

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        foreach ($ad->getOutwardWaypoints() as $position => $point) {
            $waypoint = new Waypoint();

            $address = ($point instanceof Address) ? $point : $this->createAddressFromPoint($point);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($ad->getReturnWaypoints() as $position => $point) {
                $waypoint = new Waypoint();

                $address = ($point instanceof Address) ? $point : $this->createAddressFromPoint($point);

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['thu']) && $schedule['thu']) {
                $adSchedule['thu'] = true;
                $adSchedule['thuOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['thuReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['thu'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['wed']) && $schedule['wed']) {
                $adSchedule['wed'] = true;
                $adSchedule['wedOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['wedReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['wed'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['sat']) && $schedule['sat']) {
                $adSchedule['sat'] = true;
                $adSchedule['satOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['satReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['sat'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['tue']) && $schedule['tue']) {
                $adSchedule['tue'] = true;
                $adSchedule['tueOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['tueReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['tue'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['fri']) && $schedule['fri']) {
                $adSchedule['fri'] = true;
                $adSchedule['friOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['friReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['fri'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['sun']) && $schedule['sun']) {
                $adSchedule['sun'] = true;
                $adSchedule['sunOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['sunReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['sun'])) {

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

            if (isset($schedule['mon']) && $schedule['mon']) {
                $adSchedule['mon'] = true;
                $adSchedule['monOutwardTime'] = !empty($schedule['outwardTime']) ? \DateTime::createFromFormat('H:i', $schedule['outwardTime']) : null;
                $adSchedule['monReturnTime'] = !empty($schedule['returnTime']) ? \DateTime::createFromFormat('H:i', $schedule['returnTime']) : null;
            } elseif (!isset($adSchedule['mon'])) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($proposal->getMatchingOffers() as $matching) {
                // We check if the matching have an ask
                foreach ($matching->getAsks() as $ask) {
                    // We check if the ask is accepted if yes we put the ask in the tab
                    if (Ask::STATUS_ACCEPTED_AS_DRIVER === $ask->getStatus() || Ask::STATUS_ACCEPTED_AS_PASSENGER === $ask->getStatus()) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($proposal->getMatchingRequests() as $matching) {
                // We check if the matching have an ask
                /** @var Ask $ask */
                foreach ($matching->getAsks() as $ask) {
                    // We check if the ask is accepted if yes we put the ask in the tab

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        foreach ($proposal->getMatchingOffers() as $matching) {
            // We check if the matching have an ask
            foreach ($matching->getAsks() as $ask) {
                // We check if the ask is accepted if yes we put the ask in the tab
                if (Ask::STATUS_ACCEPTED_AS_DRIVER === $ask->getStatus() || Ask::STATUS_ACCEPTED_AS_PASSENGER === $ask->getStatus()) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        foreach ($proposal->getMatchingRequests() as $matching) {
            // We check if the matching have an ask
            /** @var Ask $ask */
            foreach ($matching->getAsks() as $ask) {
                // We check if the ask is accepted if yes we put the ask in the tab

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        if ($ad->isOneWay()) {
            // the ad has explicitly been set to one way
            $outwardProposal->setType(Proposal::TYPE_ONE_WAY);
        } elseif (is_null($ad->isOneWay())) {
            // the ad type has not been set, we assume it's a round trip for a regular trip and a one way for a punctual trip

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        if ($ad->isOneWay()) {
            // the ad has explicitly been set to one way
            $outwardProposal->setType(Proposal::TYPE_ONE_WAY);
        } elseif (is_null($ad->isOneWay())) {
            // the ad type has not been set, we assume it's a round trip for a regular trip and a one way for a punctual trip

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            if ('punctual' == $frequency) {
                // Punctual journey
                $ad->setFrequency(Criteria::FREQUENCY_PUNCTUAL);
                $ad->setOutwardDate(\DateTime::createFromFormat('Y-m-d', $outward['mindate']));
                (isset($outward['maxdate'])) ? $ad->setOutwardLimitDate(\DateTime::createFromFormat('Y-m-d', $outward['maxdate'])) : '';

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                } else {
                    $ad->setFrequency(Criteria::FREQUENCY_PUNCTUAL);
                    $ad->setOutwardDate(\DateTime::createFromFormat('Y-m-d', $outward['mindate']));
                    (isset($outward['maxdate'])) ? $ad->setOutwardLimitDate(\DateTime::createFromFormat('Y-m-d', $outward['maxdate'])) : '';

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if (!$ad->isOneWay() && !$returnProposal->isPrivate() && !$returnProposal->isPaused() && !$returnProposal->isDynamic()) {
            $matchings = array_merge($returnProposal->getMatchingOffers(), $returnProposal->getMatchingRequests());
            foreach ($matchings as $matching) {
                $this->entityManager->refresh($matching);
                if (is_null($matching->getMatchingOpposite())) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if (!$outwardProposal->isPrivate() && !$outwardProposal->isPaused() && !$outwardProposal->isDynamic()) {
            $matchings = array_merge($outwardProposal->getMatchingOffers(), $outwardProposal->getMatchingRequests());
            foreach ($matchings as $matching) {
                $this->entityManager->refresh($matching);
                if (is_null($matching->getMatchingOpposite())) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        foreach ($ad->getOutwardWaypoints() as $position => $point) {
            $waypoint = new Waypoint();
            $waypoint->setAddress(($point instanceof Address) ? $point : $this->createAddressFromPoint($point));
            $waypoint->setPosition($position);
            $waypoint->setDestination($position == count($ad->getOutwardWaypoints()) - 1);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($ad->getReturnWaypoints() as $position => $point) {
                $waypoint = new Waypoint();
                $waypoint->setAddress(($point instanceof Address) ? $point : $this->createAddressFromPoint($point));
                $waypoint->setPosition($position);
                $waypoint->setDestination($position == count($ad->getReturnWaypoints()) - 1);

• Read up
  Read up