api-platform / schema-generator

PHP Model Scaffolding from Schema.org and other RDF vocabularies
https://api-platform.com/docs/schema-generator/
MIT License
453 stars 108 forks source link

ambiguous EnglishInflector result may gives not expected adder/ remover method names #392

Open PawelSuwinski opened 1 year ago

PawelSuwinski commented 1 year ago

Description
Singularized form of adder/remover method name for collections is hardcoded to the first item from the list returned by EnglishInflector:
https://github.com/api-platform/schema-generator/blob/ed297c7e7c7670c23102176b22667efcaab8dcac/src/Model/Class_.php#L335

If transformation is ambiguous inflector return list of proposition, and in some cases first one it not we want to, for example:

$ php -r 'require "vendor/autoload.php"; echo (new Symfony\Component\String\Inflector\EnglishInflector())->singularize("languages")[0].PHP_EOL;'
languag

How to reproduce

types:
  Book:
    properties:
      languages:
        cardinality: "(1..*)"
        range: Language

gives:

class Book
(...)
   function addLanguag(Language $languag)
   (...)

Possible Solution

alanpoulain commented 1 year ago

The generator is using an interface: https://github.com/api-platform/schema-generator/blob/ed297c7e7c7670c23102176b22667efcaab8dcac/src/TypesGenerator.php#L87 You can already use an alternate inflector.

However having an additional config parameter could be a good idea.