« first day (4934 days earlier)   
07:00 - 19:0019:00 - 20:00

7:09 PM
@TimWolla What does this mean in the class naming rfc "Diverging from this policy is allowed to keep internal consistency within a single extension"?
For example why is Dom\XMLDocument::saveXmlFile() not Dom\XmlDocument::saveXmlFile()?
Also morngins room
Should we not strive to just make it all consistent instead?
Having a standard, but diverging from it right of the bat feels a bit weird to me
 
@PeeHaa If existing class names within an extension diverge from the policy, then new classes may be added that also diverge from the policy, but are consistent with the existing class names. The various ext/curl classes would be an example (but they are already inconsistent).
 
Why not make everything consistent though?
 
@PeeHaa For XMLDocument-the-classname the DOM standard specifies the casing. For the methods I've opted to follow the policy and Niels as the effective current ext/dom maintainer agreed there.
For ext/dom it's not particularly great either way, I agree.
@PeeHaa externals.io/message/122975#123127 does this answer that question?
 
Couldn't that argument be made for everything
The IETF says it should be HTTP for example
 
For ext/dom it follows the "if the name follows an established, language-agnostic standard" bit. The DOM standard actually specifies the API exposed to the user.
 
7:17 PM
Would we not get in a situation that when writing any new classes that we have to bikeshed whether some authority / standard specifies the casing already or not?
 
It would be on the RFC author to point out the standard in question, preferably with references to other programming languages implementing it.
 
This just feels so inconsistent :)
For an RFC that tries to introduce consistency :P
On one hand we say this is bad, but on the other we say but it is fine
I understand the reasoning now though, even if I strongly disagree
Thanks
 
FWIW: There has already been an RFC that tried to introduce consistency (with all-upper acronyms), but everyone ignored it :-p
Because I disagree with all-upper, I want it as consistent as possible the other direction, even if it's not going to be perfect.
 
> with all-upper acronyms
Good riddance :P
That was very biased ^^
As long as it is consistent even though I dislike all upper
 
Quoting from my RFC:

> As an example ext/json has JsonException which should've been JSONException according to the previous RFC’s results. In fact JsonException’s RFC was created just 3 months (!) after the class naming RFC in September 2017.
 
7:26 PM
Anarchy!
 
The previous iteration didn't even survive 3 months in practice.
 
That's why I think letting extension maintainers decide is a bad idea honestly
Instead of enforcing the project standard when it is "including" in the project
 
Diverging from the policy would need to have a reason in an RFC and then it would be part of the vote.
 
Ah so the class naming one is going to have multiple votes?
 
No, why? I mean, when I want to add a new class to ext/curl, then in the RFC I'd say "The class is called CURLUrl for consistency with the existing curl-related classes".
(ignoring the fact that ext/curl is already inconsistent)
((and also ignoring the fact that curl is not an acronym in the first place))
 
7:33 PM
Maybe I am reading it wrong, but dom changes will be made as part of this rfc? And you also say diverging from policy would need a vote
 
Yes, the DOM changes will be made as part of the RFC and it will be part of the vote of accepting the new policy. The only DOM classes affected are the ones that were added in PHP 8.4 and the ones the introduced the DOM\ namespace (which will be renamed to Dom\).
So there is a clear distinction between “old ext/dom” and “new ext/dom”.
 
07:00 - 19:0019:00 - 20:00

« first day (4934 days earlier)