[Bug]: Conflict on date timestamp and a numeric value

[ManoMahe]

Is the bug applicable and reproducable to the latest version of the package and hasn't it been reported before?

• Yes, it's still reproducable

What version of Laravel Excel are you using?

3.1

What version of Laravel are you using?

9

What version of PHP are you using?

8.0.28

Describe your issue

The Excel sheet contains numeric values like the Excel date timestamp format also leads to importing the data wrongly.

For example, an Excel column numeric date timestamp value of date 28/02/2024 (in Maatwesite, its captured value is 45350) in column A, and the same Excel sheet column B has a numeric value of 45444 (let us consider this column value as amount).
If the User by mistake wrongly formatted the values like column A has amount and column B has Excel date timestamp, then hit the import button allows the user to import the Excel data wrongly into DB.

I need to control it and tell the user about their mistake before importing large data.

Because the Date::excelToDateTimeObject($row['colum_A'])->format('Y-m-d') allows us to convert the amount column data like numeric values also into the date in this method.

I can't able to catch this wrong date column import in this package. So we need to figure out a way to capture the date column as it is so that we can able to check the date format and stop filling the DB with the wrong data import.

How can the issue be reproduced?

Below is my Import class function sample:

public function collection(Collection $rows)
{
    foreach ($rows as $key => $row)
    {
        $date = \PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row[0])->format('Y-m-d H:i:s');
        $amount = $row[1];

        // DB insert code goes here
    }
}

Attached file to reproduce the issue.

Maatwebsite_issue_reproduce.xlsx

What should be the expected behaviour?

Need to check and throw a bug report while initiating data import when column A is not a valid date column and if there is a valid date then the data should be imported into DB.

[patrickbrouwers]

This is something you should tackle in your own code, like validation rules etc.

[ManoMahe]

Yeah @patrickbrouwers, I have tried that way too... Added below my rules function for your reference,

public function rules(): array
{
    return [
        '0' => 'required|date_format:m/d/Y',
        '1' => 'required',
    ];
}

Anyhow the function of this rule is not going to help us right? Because Excel gives us the date timestamp which is in numeric value only then I don't get how this rule will work...

[ManoMahe]

@patrickbrouwers Do I need to do any version upgrade? Because still I can't figure out a solution to this problem...