Skip to content
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.

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

src/locales/*/address/city.ts is inplausible #1471

Closed
ST-DDT opened this issue Oct 18, 2022 · 13 comments
Closed

src/locales/*/address/city.ts is inplausible #1471

ST-DDT opened this issue Oct 18, 2022 · 13 comments
Assignees
Labels
c: locale Permutes locale definitions m: location Something is referring to the location module p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug
Milestone

Comments

@ST-DDT
Copy link
Member

ST-DDT commented Oct 18, 2022

export default ['{{address.city_prefix}}'];

It should most likely point to city_name or be deleted.

Note: After #1469 is merged these files will then be located under src/locales/*/location/city.ts.

@ST-DDT ST-DDT added p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug c: locale Permutes locale definitions m: location Something is referring to the location module labels Oct 18, 2022
@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

On hold till #1469 is fixed.

@ST-DDT ST-DDT added the s: on hold Blocked by something or frozen to avoid conflicts label Oct 18, 2022
@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

Also

export default ['{{address.city_prefix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

Also

export default ['{{address.city_prefix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

Also

export default ['{{address.city_prefix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default ['{{address.city_prefix}}'];

@ST-DDT ST-DDT changed the title src/locales/en_AU_ocker/address/city.ts is inplausible src/locales/*/address/city.ts is inplausible Oct 18, 2022
@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default [
'{{address.city_prefix}}',
'{{address.city_prefix}}{{address.city_suffix}}',
];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default [
'{{address.city_prefix}}',
'{{address.city_prefix}}{{address.city_suffix}}',
];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default ['{{address.city_prefix}}{{address.city_suffix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default ['{{address.city_prefix}}{{address.city_suffix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

export default ['{{address.city_prefix}}{{address.city_suffix}}'];

@ST-DDT
Copy link
Member Author

ST-DDT commented Oct 18, 2022

I might have missed some files, so the other city.ts files should be checked again.

@Shinigami92 Shinigami92 removed the s: on hold Blocked by something or frozen to avoid conflicts label Oct 25, 2022
@matthewmayer
Copy link
Contributor

matthewmayer commented Dec 26, 2022

Here are all the different patterns in use

{{company.adjective}}{{company.noun}}{{location.city_suffix}} : dv
{{location.city_name}} : ar,az,cz,de_AT,de_CH,dv,en,en_GH,en_IN,fa,fr,fr_CH,he,hr,id_ID,lv,mk,ne,pl,pt_PT,ro,ru,sk,tr,uk,vi
{{location.city_name}}{{location.city_suffix}} : ko,nb_NO
{{location.city_prefix}} : en_AU_ocker,en_NG,en_ZA,es,es_MX,fr_BE,nl,nl_BE
{{location.city_prefix}} {{company.noun}}{{location.city_suffix}} : dv
{{location.city_prefix}} {{person.first_name}} : de,dv,en,ge,it,ur
{{location.city_prefix}} {{person.first_name}} {{location.city_suffix}} : it
{{location.city_prefix}} {{person.first_name}}{{location.city_suffix}} : de,en,ge,ur
{{location.city_prefix}} {{person.firstName}} : af_ZA,el,en_AU,en_CA,en_IE,en_US,fr_CA,ne,pt_BR,ro,tr,zu_ZA
{{location.city_prefix}} {{person.firstName}}{{location.city_suffix}} : af_ZA,el,en_AU,en_CA,en_IE,en_US,fr_CA,ne,pt_BR,ro,tr,zu_ZA
{{location.city_prefix}} {{person.last_name}} : hy
{{location.city_prefix}} {{person.lastName}} : en_GB
{{location.city_prefix}} {{person.lastName}}{{location.city_suffix}} : en_GB
{{location.city_prefix}} {{person.male_first_name}} : uk
{{location.city_prefix}}{{location.city_suffix}} : fr_BE,nl,nl_BE,sv,zh_CN,zh_TW
{{location.city_prefix}}{{person.first_name}}{{location.city_suffix}} : ja
{{location.city_prefix}}{{person.last_name}}{{location.city_suffix}} : ja
{{person.first_name}} {{location.city_suffix}} : it
{{person.first_name}}{{location.city_suffix}} : de,dv,en,ge,ja,ur
{{person.firstName}}{{location.city_suffix}} : af_ZA,el,en_AU,en_CA,en_IE,en_US,fr_CA,ne,pt_BR,ro,tr,zu_ZA
{{person.last_name}} {{location.city_suffix}} : it
{{person.last_name}}{{location.city_suffix}} : de,dv,en,ge,hy,ja,ur
{{person.lastName}}{{location.city_infix}}{{person.lastName}} : en_GB
{{person.lastName}}{{location.city_suffix}} : af_ZA,el,en_AU,en_CA,en_GB,en_IE,en_US,fr_CA,ne,pt_BR,ro,tr,zu_ZA

These are the only ones which don't include city_name or last or first names as a basis.

{{company.adjective}}{{company.noun}}{{location.city_suffix}} : dv
{{location.city_prefix}} {{company.noun}}{{location.city_suffix}} : dv
{{location.city_prefix}}{{location.city_suffix}} : fr_BE,nl,nl_BE,sv,zh_CN,zh_TW
{{location.city_prefix}} : en_AU_ocker,en_NG,en_ZA,es,es_MX,fr_BE,nl,nl_BE

These probably need manual checking, for example for en_NG the city_prefix file should just be renamed to city_name, but for example for sv i think its correct, its generating a fake city name by combining various prefixes and suffixes.

@xDivisionByZerox
Copy link
Member

Resolved by #2094.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: locale Permutes locale definitions m: location Something is referring to the location module p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug
Projects
No open projects
Status: Done
Development

No branches or pull requests

4 participants