Skip to content

Fixes Example #2 using fileperms - #793

Open
brcontainer wants to merge 2 commits into
php:masterfrom
brcontainer:patch-1
Open

Fixes Example #2 using fileperms#793
brcontainer wants to merge 2 commits into
php:masterfrom
brcontainer:patch-1

Conversation

@brcontainer

Copy link
Copy Markdown

Same as: php/doc-en#5842

Incorrect:

case 0x8000:
    $info = 'r';

Same as: php/doc-en#5842

Incorrect:

```php
case 0x8000:
    $info = 'r';
```
@alfsb

alfsb commented Sep 8, 2026

Copy link
Copy Markdown
Member

Olá, e obrigado pelo issue. Esse arquivo foi recentemente alterado na documentação em inglês, e já está na fila de atualização aqui na tradução. Após atualizado, daí vai ser possível verificar se o problema continuará.

@brcontainer

Copy link
Copy Markdown
Author

Olá, e obrigado pelo issue. Esse arquivo foi recentemente alterado na documentação em inglês, e já está na fila de atualização aqui na tradução. Após atualizado, daí vai ser possível verificar se o problema continuará.

Obrigado por responder rapidamente. Tenho uma dúvida, o sistema aqui (pt_BR) é diferente dos demais repositórios que usam a sincronia + <!-- EN-Revision: ... --> ?

@alfsb

alfsb commented Sep 8, 2026

Copy link
Copy Markdown
Member

Obrigado por responder rapidamente. Tenho uma dúvida, o sistema aqui (pt_BR) é diferente dos demais repositórios que usam a sincronia + <!-- EN-Revision: ... --> ?

Não, não é (ainda que, no detalhe, existam pequenas diferenças entre as traduções). Por isso mesmo que eu consegui ver que o arquivo foi recém alterado, e está na fila de alterações. Tradutores também revisam PRs, e é bem capaz que a sua atualização, que parece bem completa aliás, talvez seja aceita.

A minha mensagem deve ter parecida confusa, porque até então eu só tinha visto o comentário inicial com uma pequena alteração, em relação a mudança no arquivo, que é grande. Só depois que eu vi que vocẽ mandou a atualização do arquivo inteiro, e daí faria mais sentido revistar e aplicar, inteiro.

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