The client functionality itself works fine (mostly). However, unlike other Aegir features, the 'clients' checkbox on the features page does not mirror the state of the client.module on the modules page. In fact, disabling client.module removes the client feature entirely and enabling client.module doesn't activate the clients feature.

Comments

ergonlogic’s picture

Version: 6.x-1.6 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new904 bytes
new1.88 KB

Here are a couple patches. The first moves the feature hook to it's own .inc. The other adds the required 'module' entry so that the feature and module statuses are kept in sync.

thimothoeye’s picture

StatusFileSize
new1.87 KB

Perfect, thanks a lot!!

Works exactly as expected, except for a tiny whitespace error on line 33 of the first patch. Revised patch attached...

ergonlogic’s picture

Status: Needs review » Reviewed & tested by the community

I'm marking this RTBC, even though it's bad form, since these are my patches. However, I'm interpreting #2 as vouching for them.

anarcat’s picture

Status: Reviewed & tested by the community » Fixed

fix committed, thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.