Filament•7mo ago

Import action make field nullable does not work

I want to store fields in db even if they are not valid or empty as null, but I can't get this to work. I have tried to cast the field to null but it always says "must be valid date".

  return [
    ImportColumn::make('salutation')
        ->rules(['max:255']),
    ImportColumn::make('firstname')
        ->requiredMapping()
        ->rules(['required', 'max:255']),
    ImportColumn::make('lastname')
        ->requiredMapping()
        ->rules(['required', 'max:255']),
    ImportColumn::make('email')
        ->rules(['nullable', 'max:255']),
    ImportColumn::make('phone')
        ->rules(['max:255']),
    ImportColumn::make('whatsapp_number')
        ->rules(['max:255']),
    ImportColumn::make('follow_up')
        ->castStateUsing(fn($state) => $state ? Carbon::parse($state) : null) // here for example
        ->rules(['max:255']),
    ImportColumn::make('notes')
        ->rules(['max:10000']),
];

  return [
    ImportColumn::make('salutation')
        ->rules(['max:255']),
    ImportColumn::make('firstname')
        ->requiredMapping()
        ->rules(['required', 'max:255']),
    ImportColumn::make('lastname')
        ->requiredMapping()
        ->rules(['required', 'max:255']),
    ImportColumn::make('email')
        ->rules(['nullable', 'max:255']),
    ImportColumn::make('phone')
        ->rules(['max:255']),
    ImportColumn::make('whatsapp_number')
        ->rules(['max:255']),
    ImportColumn::make('follow_up')
        ->castStateUsing(fn($state) => $state ? Carbon::parse($state) : null) // here for example
        ->rules(['max:255']),
    ImportColumn::make('notes')
        ->rules(['max:10000']),
];

Help is much appreciated! 🙂

Solution:

Thanks solved it like so ``` protected function beforeValidate(): void {...

Jump to solution

24 Replies

toeknee•7mo ago

You need to set your column for date to be nullable in the db.

kekitekOP•7mo ago

it is already $table->timestamp('follow_up')->nullable();

toeknee•7mo ago

email has nothing to do with a valid date? which I suspect is follow_up

kekitekOP•7mo ago

Sorry I was copying the wrong line of code. The problem is still the same. Updated the post above

Dennis Koch•7mo ago

I have tried to cast the field to null but it always says "must be valid date".

Is that an exception or a validation message?

kekitekOP•7mo ago

This is a validation message I see in the document at the end of an row

kekitekOP•7mo ago

No matter what I always get validation errors, struggling for days now.

toeknee•7mo ago

Provide your test CSV

kekitekOP•7mo ago

Can't provide the whole file, but I want not valid data to be stored as null

toeknee•7mo ago

You need to remove not valid data then and just cast it.

kekitekOP•7mo ago

So you mean I should only provide valid data in my files?

toeknee•7mo ago

No, you can cast it https://filamentphp.com/docs/3.x/actions/prebuilt-actions/import#casting-state which adjusts that it will be, so casting to null if invalid will work if you allow nulls in that column.

Import action - Actions - Filament

kekitekOP•7mo ago

Tried that already, but no success

$table->timestamp('follow_up')->nullable();

$table->timestamp('follow_up')->nullable();

->castStateUsing(function ($value) {
    try {
        $date = new \DateTime($value);
        return $date->getTimestamp();
    } catch (\Exception $e) {
        return null;
    }
}),

->castStateUsing(function ($value) {
    try {
        $date = new \DateTime($value);
        return $date->getTimestamp();
    } catch (\Exception $e) {
        return null;
    }
}),

toeknee•7mo ago

And what is the state? is the state null? If it iss then your not allow to store nulls. Remove the rules too

kekitekOP•7mo ago

Did that already

 ImportColumn::make('follow_up')
  ->castStateUsing(function ($value) {
      try {
          $date = new \DateTime($value);
          return $date->getTimestamp();
      } catch (\Exception $e) {
          return null;
      }
  }),

 ImportColumn::make('follow_up')
  ->castStateUsing(function ($value) {
      try {
          $date = new \DateTime($value);
          return $date->getTimestamp();
      } catch (\Exception $e) {
          return null;
      }
  }),

And what is the state? is the state null? If it iss then your not allow to store nulls.

And what is the state? is the state null? If it iss then your not allow to store nulls.

How to change that?

toeknee•7mo ago

try { $date = new \DateTime($value); $newState = $date->getTimestamp(); } catch (\Exception $e) { $newState = null; } dd($newState);

kekitekOP•7mo ago

ok does nothing

toeknee•7mo ago

Interesting, try logging it instead. The importer could be catching the exception Logging on the try and the catch

kekitekOP•7mo ago

Ok I get nothing in my log

ImportColumn::make('follow_up')
      ->castStateUsing(function ($value) {
          try {
              $date = new \DateTime($value);
              $newState = $date->getTimestamp();
          } catch (\Exception $e) {
              $newState = null;
              Log::info('Follow up date: ' . $newState);
          }
      }),

ImportColumn::make('follow_up')
      ->castStateUsing(function ($value) {
          try {
              $date = new \DateTime($value);
              $newState = $date->getTimestamp();
          } catch (\Exception $e) {
              $newState = null;
              Log::info('Follow up date: ' . $newState);
          }
      }),

same here

 try {
      $date = new \DateTime($value);
      $newState = $date->getTimestamp();
      Log::info('Follow up date: ' . $newState);
  } catch (\Exception $e) {
      $newState = null;
      Log::info('Follow up date: ' . $newState);
  }

 try {
      $date = new \DateTime($value);
      $newState = $date->getTimestamp();
      Log::info('Follow up date: ' . $newState);
  } catch (\Exception $e) {
      $newState = null;
      Log::info('Follow up date: ' . $newState);
  }

Is there someone else who has an idea how to handle this? Please really need this

LeandroFerreira•7mo ago

maybe using a beforeValidate? $this->data['follow_up'] = $this->data['follow_up'] ? Carbon::parse($this->data['follow_up']) : null https://filamentphp.com/docs/3.x/actions/prebuilt-actions/import#lifecycle-hooks

Import action - Actions - Filament

kekitekOP•7mo ago

Thanks I will try this 🙂

LeandroFerreira•7mo ago

If it doesn't work, provide a mini repo on Github to reproduce this issue.. I can take a look

kekitekOP•7mo ago

Oh wow thanks for your effort Will check it later no time at the moment

Solution

kekitek•7mo ago

Thanks solved it like so

protected function beforeValidate(): void
{
    $this->data['follow_up'] = $this->parseFollowUp($this->data['follow_up']);
}

/**
 *  Parse the follow up date.
 */
protected function parseFollowUp($followUp)
{
    if (empty($followUp)) {
        return null;
    }

    try {
        return Carbon::parse($followUp)->getTimestamp();
    } catch (\Exception $e) {
        return null;
    }
}

protected function beforeValidate(): void
{
    $this->data['follow_up'] = $this->parseFollowUp($this->data['follow_up']);
}

/**
 *  Parse the follow up date.
 */
protected function parseFollowUp($followUp)
{
    if (empty($followUp)) {
        return null;
    }

    try {
        return Carbon::parse($followUp)->getTimestamp();
    } catch (\Exception $e) {
        return null;
    }
}

Gaming

Programming

Import action make field nullable does not work