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

Convert curl-getinfo() constant list to table #779

Merged
merged 3 commits into from
Jan 31, 2024
Merged

Convert curl-getinfo() constant list to table #779

merged 3 commits into from
Jan 31, 2024

Conversation

haszi
Copy link
Contributor

@haszi haszi commented Jan 29, 2024

This PR converts the constant list for the second (option) parameter of curl_getinfo() to a table (as it was done in doc-en here).

@Girgias
Copy link
Member

Girgias commented Jan 29, 2024

Note, there is a comment at the top of translations which has a revcheck which takes the git hash to which commit this is syncing the file.

Here it should be 8b3f89ed6171cb711b00afb0fcacbea3462537b9 :)

Copy link
Contributor

@victor-prdh victor-prdh left a comment

Choose a reason for hiding this comment

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

Hi, spotted some mistakes here (unrelated to the change).

Maybe we must handle it in another PR ?

Avec l'option
<constant>CURLOPT_FOLLOWLOCATION</constant> désactivée : URL de
redirection trouvée dans la dernière transaction, qui devra être
interrogé manuellement par la suite. Si l'option
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
interrogé manuellement par la suite. Si l'option
interrogée manuellement par la suite. Si l'option

<row>
<entry><constant>CURLINFO_HTTPAUTH_AVAIL</constant></entry>
<entry>
Masque binaire indiquant la(/les) méthode(s) d'authentification disponbile en accord avec la réponse précédente
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Masque binaire indiquant la(/les) méthode(s) d'authentification disponbile en accord avec la réponse précédente
Masque binaire indiquant la(/les) méthode(s) d'authentification disponible en accord avec la réponse précédente

<row>
<entry><constant>CURLINFO_PROXYAUTH_AVAIL</constant></entry>
<entry>
Masque binaire indiquant la(/les) méthode(s) d'authentification proxy disponbile en accord avec la réponse précédente
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Masque binaire indiquant la(/les) méthode(s) d'authentification proxy disponbile en accord avec la réponse précédente
Masque binaire indiquant la(/les) méthode(s) d'authentification proxy disponible en accord avec la réponse précédente

<row>
<entry><constant>CURLINFO_COOKIELIST</constant></entry>
<entry>
Tous les cookies connue
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Tous les cookies connue
Tous les cookies connus

<entry><constant>CURLINFO_CONTENT_LENGTH_DOWNLOAD_T</constant></entry>
<entry>
Le
content-length du téléchargement. Cette valeur est lu depuis le champ
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
content-length du téléchargement. Cette valeur est lu depuis le champ
content-length du téléchargement. Cette valeur est lue depuis le champ

<entry><constant>CURLINFO_CONTENT_LENGTH_UPLOAD_T</constant></entry>
<entry>
La taille
spécifié pour le téléversment. -1 si la taille est inconnue
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
spécifié pour le téléversment. -1 si la taille est inconnue
spécifiée pour le téléversement. -1 si la taille est inconnue

@Girgias Girgias merged commit 83a628b into php:master Jan 31, 2024
1 check passed
@Girgias
Copy link
Member

Girgias commented Jan 31, 2024

The grammar fixes should be done in a seperate PR

@haszi haszi deleted the Convert-curl-getinfo-constant-list-to-table branch January 31, 2024 09:52
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.

3 participants