Skip to content

Synchronize manual with stubs for PHP 8.4 - part 3 #4179

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

Merged
merged 5 commits into from
Dec 1, 2024

Conversation

kocsismate
Copy link
Member

No description provided.

Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

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

Thank you!

Comment on lines 89 to 94
<row>
<entry>8.4.0</entry>
<entry>
This class is now <literal>final</literal>.
</entry>
</row>
Copy link
Member

Choose a reason for hiding this comment

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

Incorrect this is part of https://wiki.php.net/rfc/directory-opaque-object which is PHP 8.5.

Please use a worktree and the stubs from PHP-8.4 to do these changes.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, I forgot that this was voted for PHP 8.5. Problem with using an older branch when generating the manual pages is that gen_stub.php usually receives improvements in master only, therefore I usually just disregard the stub changes that are not yet relevant.

Copy link
Member

Choose a reason for hiding this comment

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

But you could use the stub files from PHP-8.4 with master gen_stub, no?

<varlistentry xml:id="soapclient.props.--soap-fault">
<varlistentry xml:id="soapclient.props.soap-fault">
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if those soap changes are correct, I'm pretty sure you are breaking links with this rn.

Copy link
Member Author

Choose a reason for hiding this comment

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

AFAIR we had a discussion about this in an MR, and you preferred to omit the leading hyphens when generating these IDs. #3367 (comment) (I only found a small discussion with Haszi, but probably the main one happened when dealing with constants)

Copy link
Member

Choose a reason for hiding this comment

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

Can you check if those IDs are used in a linken attribute or not? But in the same vein, you should remove the single hyphen from the constants below.

Copy link
Member Author

Choose a reason for hiding this comment

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

I found no collision. In the same time, I removed the leading single hyphens.

@Girgias Girgias merged commit c0e4870 into php:master Dec 1, 2024
2 checks passed
@kocsismate kocsismate deleted the php84-sync-3 branch December 2, 2024 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants