[UW-2934] chore(composer): switched back to "wiosna-dev/common-library", to branch "feature/UW-2934_Add_support_for_PHP_8.x"

[UW-2538] fixes method

VERSION

@@ -1 +1 @@
-0.1.0
+0.1.1

composer.json

@@ -12,7 +12,7 @@
     "require": {
         "php": ">=5.6",
         "fguillot/json-rpc": "^1.2",
-        "meritoo/common-library": "^0.1.0"
+        "wiosna-dev/common-library": "dev-feature/UW-2934_Add_support_for_PHP_8.x"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^2.6",

phing/properties.dist

@@ -69,6 +69,7 @@ dir.tests = ${project.basedir}/tests
 dir.build = ${project.basedir}/build
 dir.reports = ${dir.build}/logs
 dir.reports.pdepend = ${dir.reports}/pdepend
+dir.reports.coverage = ${dir.reports}/phpunit_coverage
 
 # Data directories
 #

src/Base/Result/BaseParticipant.php

@@ -39,6 +39,13 @@ abstract class BaseParticipant extends BaseItem
      */
     protected $email;
 
+    /**
+     * Token of the participant
+     *
+     * @var string
+     */
+    protected $token;
+
     /**
      * Returns ID of the participant
      *
@@ -78,4 +85,14 @@ abstract class BaseParticipant extends BaseItem
     {
         return $this->email;
     }
+
+    /**
+     * Returns token of the participant
+     *
+     * @return string
+     */
+    public function getToken()
+    {
+        return $this->token;
+    }
 }

src/Result/Item/Participant.php

@@ -42,13 +42,6 @@ class Participant extends BaseParticipant
      */
     private $emailStatus;
 
-    /**
-     * Token of the participant
-     *
-     * @var string
-     */
-    private $token;
-
     /**
      * Language of the participant
      *
@@ -232,16 +225,6 @@ class Participant extends BaseParticipant
         return $this->emailStatus;
     }
 
-    /**
-     * Returns token of the participant
-     *
-     * @return string
-     */
-    public function getToken()
-    {
-        return $this->token;
-    }
-
     /**
      * Returns language of the participant
      *

src/Result/Item/ParticipantShort.php

@@ -28,6 +28,10 @@ class ParticipantShort extends BaseParticipant
                 $this->id = (int)$value;
                 break;
 
+            case 'token':
+                $this->token = trim($value);
+                break;
+
             case 'participant_info':
                 $this->firstName = trim($value['firstname']);
                 $this->lastName = trim($value['lastname']);
@@ -52,6 +56,7 @@ class ParticipantShort extends BaseParticipant
 
         $data = [
             'tid'              => $participant->getId(),
+            'token'            => $participant->getToken(),
             'participant_info' => $info,
         ];
 

src/Service/SurveyService.php

@@ -12,6 +12,8 @@ use Meritoo\Common\Collection\Collection;
 use Meritoo\LimeSurvey\ApiClient\Client\Client;
 use Meritoo\LimeSurvey\ApiClient\Exception\CannotProcessDataException;
 use Meritoo\LimeSurvey\ApiClient\Exception\MissingSurveySummaryException;
+use Meritoo\LimeSurvey\ApiClient\Exception\UnknownInstanceOfResultItem;
+use Meritoo\LimeSurvey\ApiClient\Exception\UnknownMethodException;
 use Meritoo\LimeSurvey\ApiClient\Result\Collection\Participants;
 use Meritoo\LimeSurvey\ApiClient\Result\Collection\Surveys;
 use Meritoo\LimeSurvey\ApiClient\Result\Collection\SurveysSummaries;
@@ -126,6 +128,8 @@ class SurveyService
      * @return Surveys
      *
      * @throws CannotProcessDataException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function getAllSurveys($onlyActive = false)
     {
@@ -160,10 +164,13 @@ class SurveyService
     /**
      * Returns information if survey with given ID exists
      *
-     * @param int  $surveyId       ID of survey to verify
+     * @param int $surveyId ID of survey to verify
      * @param bool $shouldBeActive (optional) If is set to true, survey should be active. If it's not, it shouldn't
      *                             be returned, even if exists. Otherwise - it doesn't matter (default behaviour).
      * @return bool
+     * @throws CannotProcessDataException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function isExistingSurvey($surveyId, $shouldBeActive = false)
     {
@@ -221,12 +228,15 @@ class SurveyService
     /**
      * Returns participants of given survey
      *
-     * @param int  $surveyId      ID of survey
+     * @param int $surveyId ID of survey
      * @param bool $onlyCompleted (optional) If is set to true, participants who completed survey are returned only.
      *                            Otherwise - all (default behaviour).
      * @return Collection
      *
      * @throws CannotProcessDataException
+     * @throws MissingSurveySummaryException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function getSurveyParticipants($surveyId, $onlyCompleted = false)
     {
@@ -275,11 +285,14 @@ class SurveyService
     /**
      * Adds participant with given data to survey with given ID
      *
-     * @param int    $surveyId  ID of survey
+     * @param int $surveyId ID of survey
      * @param string $firstName First name of the participant to add
-     * @param string $lastName  Last ame of the participant to add
-     * @param string $email     E-mail address of the participant to add
+     * @param string $lastName Last ame of the participant to add
+     * @param string $email E-mail address of the participant to add
      * @return Participant
+     * @throws CannotProcessDataException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function addParticipant($surveyId, $firstName, $lastName, $email)
     {
@@ -318,9 +331,13 @@ class SurveyService
     /**
      * Returns short data of one participant with given e-mail (participant of given survey)
      *
-     * @param int    $surveyId ID of survey
-     * @param string $email    E-mail address of the participant
+     * @param int $surveyId ID of survey
+     * @param string $email E-mail address of the participant
      * @return ParticipantShort|null
+     * @throws CannotProcessDataException
+     * @throws MissingSurveySummaryException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function getParticipant($surveyId, $email)
     {
@@ -343,7 +360,10 @@ class SurveyService
      * @param int $surveyId ID of survey
      * @return int
      *
+     * @throws CannotProcessDataException
      * @throws MissingSurveySummaryException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     public function getSurveyTokenCount($surveyId)
     {
@@ -374,6 +394,9 @@ class SurveyService
      *
      * @param int $surveyId ID of survey
      * @return SurveySummary|null
+     * @throws CannotProcessDataException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
      */
     private function getSurveySummary($surveyId)
     {
@@ -395,4 +418,44 @@ class SurveyService
 
         return $surveySummary;
     }
+
+    /**
+     * @param $surveyId
+     * @param $email
+     * @return bool
+     * @throws CannotProcessDataException
+     * @throws UnknownInstanceOfResultItem
+     * @throws UnknownMethodException
+     */
+    public function hasSurveyBeenCompletedByEmail($surveyId, $email)
+    {
+        $arguments = [
+            $surveyId,
+            $offset = 0,
+            $limit = 10,
+            $includeUnused = false,
+            ['completed'],
+            ['email' => $email]
+        ];
+
+        try {
+            /** @var Collection $participants */
+            $participants = $this
+                ->client
+                ->run(MethodType::LIST_PARTICIPANTS, $arguments)
+                ->getData(true);
+
+            foreach ($participants as $participant) {
+                if ('N' !== $participant['completed']) {
+                    return true;
+                }
+            }
+        } catch (CannotProcessDataException $exception) {
+            if (ReasonType::NO_PARTICIPANTS_FOUND !== $exception->getReason()) {
+                throw $exception;
+            }
+        }
+
+        return false;
+    }
 }