Laravel Forge & Laravel Homestead (potentially other hosted panels for VPS, for instance Ploi on some servers) started installing & configuring Swoole as a reactive PHP module. Swoole uses a function called go()
as one of their main called functions, and this makes Kirby fail on these servers.
It would be nice to:
redirect()
makes sense, and wrapping both in a function_exists
check, and let the developers to make the choice.i created a issue at github similar to this idea-ticket once but it seemed to got “lost” or moved. afaik the go helper is used internally by kirby only once.
i also think it would be a plus if kirby would play nice with these setups instead of insisting it owns “go()” and just crash and burn if not.
Well any of the https://getkirby.com/docs/reference/templates/helpers helpers could potentionaly clash. I think they should be only used in userspace right? And thus should be possible to not load them if user doesn’t want to.
@bnomei We have moved your issue to our internal backlog repo. We also have this issue with some ideas for a plan forward: https://github.com/getkirby/kirby/issues/3062
Feedback is well appreciated. :)
@Lukas Bestle that issue seems to have moved to a private repo, but if I remember correctly, there was an issue with two possible pain points:
Imo: core should not use the global helpers at all, but rather the class functions those helpers wrap, always. As for user space collisions: I think the onus of knowing what global functions belong to where is on users, specifically those “advanced” enough to pull other stuff in.
I have no view into support tickets, so maybe that generates a problem for you at the intersection of “knows how to composer require” and “doesn’t fully understand what that means in regards to global functions”, but I imagine it’s quite a small intersection 🤔
That’s the plan. :)