- fix response update condition

- fix refresh response import bug

src/Commands/SurveyheroResponseImportCommand.php

@@ -31,9 +31,9 @@ public function __construct(SurveyResponseImportService $surveyResponseImportSer
 
     public function handle(): int
     {
-        $truncateResponses = $this->option('fresh');
+        $refreshResponses = $this->option('fresh');
 
-        if ($truncateResponses) {
+        if ($refreshResponses) {
             $this->deleteResponses();
         }
 
@@ -50,6 +50,10 @@ public function handle(): int
         foreach ($surveys as $survey) {
             /* @var SurveyContract $survey */
             try {
+                if($refreshResponses){
+                    $survey->survey_last_imported = null;
+                    $survey->save();
+                }
                 $importInfo->addInfo($this->importService->importSurveyResponses($survey));
             } catch (SurveyNotMappedException $exception) {
                 $this->error("{$exception->getMessage()} Survey '$survey->name' with Surveyhero ID $survey->surveyhero_id");

src/Contracts/SurveyContract.php

@@ -24,4 +24,11 @@ public function surveyQuestions(): HasMany;
     public function completedResponses(): Collection;
 
     public function hasResponses(): bool;
+
+    /**
+     * Checks if the response timestamp is more recent than the last updated survey timestamp.
+     * @param string $responseLastUpdatedIsoTimestamp
+     * @return bool
+     */
+    public function doesResponseNeedsToBeUpdated(string $responseLastUpdatedIsoTimestamp): bool;
 }

src/Models/Survey.php

@@ -2,6 +2,7 @@
 
 namespace Statikbe\Surveyhero\Models;
 
+use Carbon\Carbon;
 use Illuminate\Database\Eloquent\Collection;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
@@ -45,4 +46,14 @@ public function hasResponses(): bool
     {
         return $this->surveyResponses()->count() > 0;
     }
+
+    /**
+     * @inheritDoc
+     */
+    public function doesResponseNeedsToBeUpdated(string $responseLastUpdatedIsoTimestamp): bool {
+        if($this->survey_last_imported) {
+            return Carbon::parse($responseLastUpdatedIsoTimestamp)->gt($this->survey_last_imported);
+        }
+        else return true;
+    }
 }

src/Services/SurveyResponseImportService.php

@@ -92,7 +92,7 @@ public function importSurveyResponse($responseId, SurveyContract $survey, array
         }
 
         $responseAnswers = $this->client->getSurveyResponseAnswers($survey->surveyhero_id, $responseId);
-        if ($responseAnswers && Carbon::parse($responseAnswers->last_updated_on)->gt($survey->survey_last_imported)) {
+        if ($responseAnswers && $survey->doesResponseNeedsToBeUpdated($responseAnswers->last_updated_on)) {
             //do not import already imported data that is not updated.
             /* @var SurveyResponseContract $existingResponseRecord */
             $existingResponseRecord = app(SurveyheroRegistrar::class)->getSurveyResponseClass()::where('surveyhero_id', $responseId)->first();