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

Sync filesystem functions files with EN #7

Merged
merged 6 commits into from
May 24, 2024

Conversation

grzegorz-bankowski
Copy link
Contributor

No description provided.

Copy link
Contributor

@Sobak Sobak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, mnóstwo plików i naprawdę super jakość. Rzuciły mi się w oczy dosłownie tylko trzy miejsce, a w jednym chcę podpytać Cię o zdanie

@@ -31,6 +31,8 @@
<function>basename</function> bierze pod uwagę obecnią lokalizację systemu, więc aby widzieć
prawidłowe basename ze ścieżek zawierających znaki spoza alfabetu łacińskiego, musi zostać
ustawiona odpowiednia lokalizacja przy pomocy funkcji <function>setlocale</function>.
Jeśli parametr <parameter>path</parameter> zawiera znaki, które są nieprawidłowe dla
bieżących ustawień regionalnych, zachowanie <function>basename</function> jest niezdefiniowane.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To jest coś co na 100% chcę opisać w readme (glossary, czy jakkolwiek to nazwiemy), wydaje mi się że trzeba tutaj coś dobrze pomyśleć. Zakładam że tłumaczyłeś frazę "locale settings". Nie wiem czy istnieje jakieś powszechnie używane tłumaczenie w j. polskim, więc ja tłumaczyłem na "ustawienia locale", co brzmi fatalnie ale przynajmniej pozwala wyszukać coś w Google na ten temat.

Co Ty na to, żeby połączyć Twoje lepsze tlumaczenie z moim i tłumaczyć to jako "ustawień regionalnych (locale)"? Wtedy mógłbym to z czystym sumieniem dać do tego glossary bo chyba łączy zalety obu.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super pasuje mi to "ustawień regionalnych (locale)" i oczywiście będzie to w README / Glossary.

reference/filesystem/functions/dirname.xml Outdated Show resolved Hide resolved
reference/filesystem/functions/dirname.xml Outdated Show resolved Hide resolved
reference/filesystem/functions/file-get-contents.xml Outdated Show resolved Hide resolved
@Sobak Sobak merged commit a3f93db into php:master May 24, 2024
1 check passed
@grzegorz-bankowski grzegorz-bankowski deleted the filesystem-functions branch May 29, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants