Hi guys,
I've noticed that, even though the client feature is disabled, a normal user cannot create a site if it's not associated to any client. I tried with a new user and some basic permissions and what I get is a form_error saying:
Your user is not associated with any clients so you are not allowed to create new sites
I looked into the code and I tried to see if there's a fast solution to fix that, but then I noticed that the whole aegir platform is based on this client id (for instance the task queues are always empty for the user that is not associated to any client).
I think that this feature is great, but I'm afraid that right now if someone doesn't need it that is forced to use it. And the weird thing is when Aegir is installed this feature is disabled.
I'd like to help to patch it, but I think there's a lot of code involved.
Comments
Comment #1
ergonlogichosting_client is a dependency of hosting_site, and enabled on install. See #2004188: Feature reverse dependencies are not enforced, for follow-up on ensuring that this dependency relationship is enforced.
Comment #2
ergonlogicActually, currently in our 1.x branch, hosting_client is not a declared dependency of hosting_site, when in fact it clearly is. Oddly, the 'Clients' hosting feature is listed under 'experimental' for some reason.
The easiest fix I can think of is to make hosting_client is a dependency of hosting_site. While we're at it, I think it'd be good to flag it HOSTING_FEATURE_REQUIRED (since it is), and move it our of 'experimental' (since it isn't).
Comment #3
ergonlogicThe fix itself is trivial, so I'm marking this as 'needs review' to get some other eyes on it.
Comment #4
anarcat commentedi thought we had another issue about this and even that it was already fixed, so go ahead.
Comment #5
helmo commentedThe related issue is probably: #1585820: Disabling the client feature yields an error
Comment #6
ergonlogicYeah, it looks like it was fixed in 23a1f17b1