Using WithMultipleSheets option will ignore the ability to import regular csv files

[timothymarois]

Just as it sounds, using WithMultipleSheets will prevent regular csv files from importing.

You would expect regular csv to still load on sheet 1 regardless because that's how it should work.

I posted on stack to see if anyone else could lead me in the right direction, no answers yet, so figured this is the next best shot.

https://stackoverflow.com/questions/71087727/laravel-csv-xls-maatwebsite-using-multiple-sheets-will-ignore-a-regular-csv

[patrickbrouwers]

As far as I know there shouldn't be a difference in handling. But given that CSV's don't have sheets, perhaps some PhpSpreadsheets are different. Best would be if you could provide a failing unit test for WithMultitpleSheets CSV imports (https://github.com/SpartnerNL/Laravel-Excel/blob/3.1/tests/Concerns/WithMultipleSheetsTest.php#L416)

[timothymarois]

@patrickbrouwers Right, but the real question is, how do you allow both file formats? because if you don't allow multiple sheets, then its going to try and import all sheets into the system, so it kind of puts you in hard place of not having the right way to import multiple file types. I'd like users to import either csv or xls and I only want the sheet one information, any other sheet would be ignored, but because of these options it doesn't seem like you can do that, so using multiple sheets allows you to just do the first sheet, but now you cant do csv.

I guess what i'd have to do is, have 2 separate import processes based on the file type, not ideal, but this package cant handle it, looks like that's the best route.

[patrickbrouwers]

The idea would be that it works for csv and xlsx, that's why I'm aksing for a failing unit test, it makes it easier to look into