# [RFC] enum SortDirection - Thread: https://externals.io/message/130226 - Original archive: https://news-web.php.net/php.internals/130226 - Started by tim@bastelstu.be on 2026-03-02T20:45:39+00:00 - Messages: 11 - RFC: enum SortDirection (Implemented (PHP 8.6)) — https://externals.io/rfc/sort_direction_enum.md - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · RFCs at https://externals.io/rfc.md · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #130226 — tim@bastelstu.be — 2026-03-02T20:45:39+00:00 URL: https://externals.io/message/130226#130226 Hi please consider this email as a formal request for comments on my modest proposal of adding a new `enum SortDirection` to PHP’s standard library. I have made the full proposal available on the World Wide Web at: https://wiki.php.net/rfc/sort_direction_enum Yours sincerely Tim Düsterhus --- ## #130227 — Bob Weinand — 2026-03-02T21:43:19+00:00 URL: https://externals.io/message/130226#130227 · In reply to: #130226 Hey Tim, > Hi > > please consider this email as a formal request for comments on my > modest proposal of adding a new `enum SortDirection` to PHP’s standard > library. > > I have made the full proposal available on the World Wide Web at: > > https://wiki.php.net/rfc/sort_direction_enum > > Yours sincerely > Tim Düsterhus That's a good idea, but I wouldn't introduce such class without some usages at least. I'd recommend considering immediately upgrading the proposed functions to use SortDirection and redefine the constants in terms of the enum. I.e. set const `SORT_ASC` = SortDirection::Ascending; - similar for others like the `SCANDIR_SORT_DESCENDING` etc.. This is a reasonable BC break, which is also trivially fixed by simply accepting int|SortDirection in existing functions. Thanks, Bob --- ## #130228 — tim@bastelstu.be — 2026-03-02T22:12:56+00:00 URL: https://externals.io/message/130226#130228 · In reply to: #130227 Hi > That's a good idea, but I wouldn't introduce such class without some > usages at least. > > I'd recommend considering immediately upgrading the proposed functions > to use SortDirection and redefine the constants in terms of the enum. > > I.e. set const `SORT_ASC` = SortDirection::Ascending; - similar for others > like the `SCANDIR_SORT_DESCENDING` etc.. > > This is a reasonable BC break, which is also trivially fixed by simply > accepting int|SortDirection in existing functions. A GitHub search for "language:PHP SORT_ASC" reveals that the existing constants are used in userland code for custom purposes (e.g. within a `switch()` statement). Redefining them to the enum would bring little value, but unnecessarily break them. See also https://wiki.php.net/rfc/correctly_name_the_rounding_mode_and_make_it_an_enum for a precedent (specifically the Future Scope section there). As suggested in the “Future Scope” of the featured RFC widening the core functions to accept the enum and update the default value to the enum case can just happen without an RFC, since this should not be observable except by Reflection. I want to avoid a situation of explicitly listing *some* of the functions in the RFC, missing one and then later having a discussion of whether or not it requires a follow-up RFC to *also* adjust that missed one. Best regards Tim Düsterhus --- ## #130229 — Derick Rethans — 2026-03-02T22:14:58+00:00 URL: https://externals.io/message/130226#130229 · In reply to: #130227 >Hey Tim, > > > > […] > > >That's a good idea, but I wouldn't introduce such class without some usages at least. > >I'd recommend considering immediately upgrading the proposed functions to use SortDirection and redefine the constants in terms of the enum. > >I.e. set const `SORT_ASC` = SortDirection::Ascending; - similar for others like the `SCANDIR_SORT_DESCENDING` etc.. > >This is a reasonable BC break, which is also trivially fixed by simply accepting int|SortDirection in existing functions. If its a BC break, it can't go into a 8.× for something that isn't *necessary*. But I also don't see how accepting both int and this new enum for existing built-in functions is a BC break. Changing the existing constants to something else is probably not a good idea, as other user functions or libraries might have reused them. cheers Derick --- ## #130230 — Larry Garfield — 2026-03-03T00:36:00+00:00 URL: https://externals.io/message/130226#130230 · In reply to: #130229 > > > […] > > > > If its a BC break, it can't go into a 8.× for something that isn't *necessary*. > > But I also don't see how accepting both int and this new enum for > existing built-in functions is a BC break. > > Changing the existing constants to something else is probably not a > good idea, as other user functions or libraries might have reused them. > > cheers > Derick I concur. I'd like to see the functions updated now in the RFC, so we get a sense of how many functions will be impacted. Not because I'm concerned about a widened type breaking anything, but more so readers can see the value this brings. The Yii DB layer (at least in v2, I haven't used v3 yet) used SORT_ASC/SORT_DESC for itself, so I agree that redefining those is unwise. Nit: Should we use Asc and Desc, for more compact code? Enums are great, but they can get verbose at times. :-) (SortDirection::Descending is 25 characters long, for what is ultimately a named boolean.) On the whole, though, I'm in favor. --Larry Garfield --- ## #130247 — tim@bastelstu.be — 2026-03-03T20:50:38+00:00 URL: https://externals.io/message/130226#130247 · In reply to: #130230 Hi > I concur. I'd like to see the functions updated now in the RFC, so we get a sense of how many functions will be impacted. Not because I'm concerned about a widened type breaking anything, but more so readers can see the value this brings. I see the main value in helping userland not to reinvent the wheel every time and to set some further precedent in including types for this kind of ubiquitous concept in the standard library. I don't want to commit to supporting the enum for every possible function in the standard library for the reasons I mentioned in my reply to Bob and also because some of the functions already have a very messy API signature where backfitting the enum is not exactly trivial. > Nit: Should we use Asc and Desc, for more compact code? I considered using the abbreviations, but then intentionally decided for the full names. Our naming policy (https://github.com/php/policies/blob/main/coding-standards-and-naming.rst#acronyms) states: > Abbreviations and acronyms as well as initialisms SHOULD be avoided wherever possible. I don't believe that this enum is a case where it is warranted to diverge from the SHOULD. My rule of thumb here is that for an abbreviation to be acceptable, it must be the “more common form” of the term in question. As an example: I would say “I am sorting the list in descending order by name” or something like that. However when I am talking about the protocol that is commonly using TCP port 80 I would say “I am sending a HTTP request”, not “I am sending a request using the hypertext transfer protocol”. HTTP is the much much more commonly used term there and thus using the acronym is preferable. Also note how I used TCP instead of Transmission Control Protocol there. Doctrine, as listed in the references section of the RFC, has also opted to use the full names in their `Order` enum. Based on this Stack Overflow answer, using the full name also seems to be more common in the C# ecosystem: https://stackoverflow.com/a/22963304 Given it's also a strongly typed enum case rather than a “magic string” such as `'ASC'`, I expect IDE autocompletion to reduce much of the typing effort. Best regards Tim Düsterhus --- ## #130309 — Larry Garfield — 2026-03-13T18:11:31+00:00 URL: https://externals.io/message/130226#130309 · In reply to: #130247 > Hi > > > > […] > > I see the main value in helping userland not to reinvent the wheel every > time and to set some further precedent in including types for this kind > of ubiquitous concept in the standard library. > > I don't want to commit to supporting the enum for every possible > function in the standard library for the reasons I mentioned in my reply > to Bob and also because some of the functions already have a very messy > API signature where backfitting the enum is not exactly trivial. I'm not suggesting we update every single function in one RFC shot. But the RFC should include updating at least one or two so that the Enum is actually used. Otherwise, there's no real advantage to putting it in core rather than FIG. The reason to put it in core is so it can be used in core. So let's use it in core, even if not the full scope of eventual usage. --Larry Garfield --- ## #130364 — tim@bastelstu.be — 2026-03-16T15:48:36+00:00 URL: https://externals.io/message/130226#130364 · In reply to: #130309 Hi Am 2026-03-13 19:11, schrieb Larry Garfield: > Otherwise, there's no real advantage to putting it in core rather than > FIG. The reason to put it in core is so it can be used in core. So > let's use it in core, even if not the full scope of eventual usage. I believe the RFC is sufficiently clear in how the enum can be useful for the PHP stdlib, listing some examples as future scope. And even if there would not (yet) be anything in the stdlib that could make use of it, I believe that asking users to install third party libraries with vendor-namespaces for a *data type* that is so broadly useful that it will likely be (transitively) required by every major framework is providing a good developer experience. Best regards Tim Düsterhus --- ## #130365 — tim@bastelstu.be — 2026-03-16T15:53:26+00:00 URL: https://externals.io/message/130226#130365 · In reply to: #130226 Hi Am 2026-03-02 21:45, schrieb Tim Düsterhus: > please consider this email as a formal request for comments on my > modest proposal of adding a new `enum SortDirection` to PHP’s standard > library. > > I have made the full proposal available on the World Wide Web at: > > https://wiki.php.net/rfc/sort_direction_enum Two weeks have passed since the initial proposal of the RFC and no changes have been made. I plan to open voting later this week. Best regards Tim Düsterhus --- ## #130367 — Larry Garfield — 2026-03-16T16:55:42+00:00 URL: https://externals.io/message/130226#130367 · In reply to: #130364 > Hi > > Am 2026-03-13 19:11, schrieb Larry Garfield: > > […] > > I believe the RFC is sufficiently clear in how the enum can be useful > for the PHP stdlib, listing some examples as future scope. And even if > there would not (yet) be anything in the stdlib that could make use of > it, I believe that asking users to install third party libraries with > vendor-namespaces for a *data type* that is so broadly useful that it > will likely be (transitively) required by every major framework is > providing a good developer experience. > > Best regards > Tim Düsterhus To be clear, I'm not suggesting that FIG would be a better place for it. I am only saying that "let's add this... and then maybe later we'll get around to quietly using it without another RFC" is a poor message and poor expectation setting. When will I be able to use it for core functions? Which functions? How will I even know? What's the plan here other than "we can do that later quietly without telling anyone" (because vanishingly few PHP devs look at Internals PRs that have no RFC)? That's what is missing. --Larry Garfield --- ## #130370 — tim@bastelstu.be — 2026-03-16T18:00:48+00:00 URL: https://externals.io/message/130226#130370 · In reply to: #130367 Hi Am 2026-03-16 17:55, schrieb Larry Garfield: > […] Which functions? How will I even know? What's the plan here other > than "we can do that later quietly without telling anyone" (because > vanishingly few PHP devs look at Internals PRs that have no RFC)? > That's what is missing. As with any other user-facing change, the changes to the function signatures would be listed in the NEWS file and the UPGRADING guide and would then be incorporated into the documentation, both in the function’s signature and the changelog that is available on every function page (e.g. “8.6.0 The $sorting_order parameter was widened to int|SortDirection”). I suppose that tools like Rector will also be able to suggest migrating from the old constants to the enum when that happens. Best regards Tim Düsterhus