-
Notifications
You must be signed in to change notification settings - Fork 757
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
Move some PDO file entities at sibling positions, wkhtmltox fragment dir #4330
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohhhh that's how I was meant to add the new PDO subclasses for 8.4
Yes, please. The classes should be listed within their respective driver pages. I don't know what this PR changes. I was hoping that it would fix the build failure I keep getting locally that I cannot fix, but it didn't. But if this PR fixes some error then let's merge it. |
You mean, meant do be side by side to old PDO drivers "chapter" (this PR), or to inside, as kamil-tekiela requested? |
I can make the changes, but it may make your local build failures worse. Let's first try to fix this mysterious build failure. You are building on Windows? The build failure is in if you have NO local changes, you can try these commands. Beware that they will erase any local change that you have:
|
Thank you @alfsb. That worked. I can compare your PR and the live version now and I can't spot any difference, so I think it's safe to merge. |
Glad that it worked. And... also saddened to know that this two decade old heisenbug still haunts the PHP manual.
Merging this will enable the broken file testing being (way) more strict that it is now. But there is no hurry for that. I will wait for the decision to rename or not, rearrange within or not, to avoid sending piece-wise chantes to translations. |
* php/doc-en#4288 を反映(PDOのディレクトリ構成変更) * php/doc-en#4298 を反映(不要セクションの削除) * Update EN-Revision * php/doc-en#4330 を反映(PDOの階層構造の変更)
While creating a new tool for detecting problematic XML files (from libxml perspective), I got reports of two invalid XML files in wkhtmltox, that are really XML fragment files, and various files in PDO hierarchy.
The "error" was about "Extra content at the end of the document", caused by entities used outside elements. To avoid losing that test, this PR changes the position of these entities of
s at sibling positions of other <reference>
s, maintaining the structure of XML, and to better reflect how the manual is rendered at the end.But... are these files intended to be positioned in sibling positions? It is rendered nowadays as:
It's is possible to have
<reference>
s inside<reference>
s, so an alternative is to move these class documentation inside their defining PDO drivers files.Also, I suggest to reaming the titleabrev to something that is more easy to search and render more naturally in image above.
MySQL PDO Driver" instead of ``MySQL (PDO)
, and so on.