Custom FileNamer, method originalFileName not working.

[EgoRoman]

I'm trying to create a custom class for naming files when uploading.
Using this docs https://spatie.be/docs/laravel-medialibrary/v9/advanced-usage/naming-files

Created class:

namespace App\Http;

use Spatie\MediaLibrary\MediaCollections\Models\Media;
use Spatie\MediaLibrary\Support\FileNamer\FileNamer;
use Spatie\MediaLibrary\Conversions\Conversion;

class ProductGalleryFileNamer extends FileNamer
{
  public function originalFileName(string $fileName): string
  {
    $mediaItem = Media::where('name', $fileName)->get();

    if ($mediaItem->count()) {
      $fileName = $fileName . '_' . $mediaItem->count(); // if file with same name exists then add suffix
    }

    return pathinfo($fileName, PATHINFO_FILENAME);
  }

  public function conversionFileName(string $fileName, Conversion $conversion): string
  {
    $strippedFileName = pathinfo($fileName, PATHINFO_FILENAME);

    $mediaItem = Media::where('name', $fileName)->get();

    if ($mediaItem->count()) {
      return "{$mediaItem->count()}-{$strippedFileName}-{$conversion->getName()}";
    } else {
      return "{$strippedFileName}-{$conversion->getName()}";
    }
  }

  public function responsiveFileName(string $fileName): string
  {
    return pathinfo($fileName, PATHINFO_FILENAME);
  }
}

Method 'conversionFileName' works perfect.

I need a suffix to be added to the original name if a file with that name already exists,
but method 'originalFileName' the method does not produce any effect.

[PaolaRuby]

that code is terrible, Media::where('name', $fileName)->get(), better Media::where('name', $fileName)->count()
you misunderstood how it works originalFileName
also you must use markdown correctly, example:

public function responsiveFileName(string $fileName): string
{
    return pathinfo($fileName, PATHINFO_FILENAME);
}

Here is your mistake

laravel-medialibrary/src/Support/FileNamer/FileNamer.php

Lines 10 to 16 in 1351471

	public function originalFileName(string $fileName): string
	{
	$extLength = strlen(pathinfo($fileName, PATHINFO_EXTENSION));
	$baseName = substr($fileName, 0, strlen($fileName) - ($extLength ? $extLength + 1 : 0));
	return $baseName;
	}

[PaolaRuby]

And why is there only such content in my vendor\spatie\laravel-medialibrary\src\Support\FileNamer\FileNamer.php?

Ok that is weird, maybe you accidentally deleted that, just get a clean copy with composer composer update

[EgoRoman]

Nope, is the same after the update.
This is composer.lock:

"name": "spatie/laravel-medialibrary",
            "version": "9.0.0",
            "source": {
                "type": "git",
                "url": "https://github.com/spatie/laravel-medialibrary.git",
                "reference": "0ddacce06fbd33de321f01c1ad8c402cda9c6b10"
            },

[PaolaRuby]

we are on version 9.8.1, even your composer is wrongly configured

[EgoRoman]

Hallelujah. Installed the version 9,8,1 and now it works. Thanks.

But I used to install the command from the documentation (https://spatie.be/docs/laravel-medialibrary/v9/installation-setup):

composer require "spatie/laravel-medialibrary:^9.0.0"

And the version 9,0,0 was installed.

[PaolaRuby]

Must be composer require spatie/laravel-medialibrary:"^9.0.0", documentation error