10000 [String] wrong plural form of "conspectus" · Issue #41124 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[String] wrong plural form of "conspectus" #41124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
connorhu opened this issue May 6, 2021 · 1 comment
Closed

[String] wrong plural form of "conspectus" #41124

connorhu opened this issue May 6, 2021 · 1 comment
Labels
Bug Status: Reviewed String

Comments

@connorhu
Copy link
Contributor
connorhu commented May 6, 2021

Symfony version(s) affected: 5.2.6

Description
Pluralize method of EnglishInflector said plural of conspectus is conspecti.
As you can see the correct plural form is conspectuses:
https://en.wiktionary.org/wiki/conspectus

How to reproduce

<?php

// composer require symfony/string

require 'vendor/autoload.php';

use Symfony\Component\String\Inflector\EnglishInflector;

$inf = new EnglishInflector();

var_dump($inf->pluralize('conspectus')[0] === 'conspectuses'); // false
var_dump(count($inf->pluralize('conspectus'))); // 1
@connorhu connorhu added the Bug label May 6, 2021
@derrabus
Copy link
Member
derrabus commented May 6, 2021

PR welcome, against the Inflector component on 4.4.

@makraz makraz mentioned this issue May 9, 2021
Merged
nicolas-grekas added a commit that referenced this issue May 10, 2021
@nicolas-grekas
bug #41147 [Inflector][String] wrong plural form of words ending by "…
9c9e645 
…pectus" (makraz)

This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[Inflector][String] wrong plural form of words ending by "pectus"

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #41124
| License       | MIT
| Doc PR        | N/A

Commits
-------

0dc8893 [Inflector][String] wrong plural form of words ending by "pectus"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Reviewed String
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nicolas-grekas @derrabus @connorhu @carsonbot
0