This one might be complicated by the current table's ability to gain extra columns for the different services supported by views. By we could just change the format of this table?

Comments

Deciphered’s picture

Deciphered CreditAttribution: Deciphered commented
Status: Active » Postponed

So I started this one, but it became clear that due to the modular nature of the servers list (the ability for different types of services to be added via hook_hosting_service_type()) and the current display choice it probably wouldn't be best served as a View.

I will expose the fields from the hosting_services table to Views, but at this stage I have no intent on going any further than that.

anarcat’s picture

anarcat CreditAttribution: anarcat commented
Status: Postponed » Needs work

this is almost there, see #588728: Meta: Views integration redux.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Project: » Hostmaster (Aegir)
Version: » 6.x-2.x-dev
anarcat’s picture

anarcat CreditAttribution: anarcat commented
Project: Hostmaster (Aegir) » Hosting
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Component: Code » Views integration
cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented

Any movement on this? It's really weird to have everything be views except for one list.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented

@cweagans, not that I'm aware of. This is a bit complicated as described in #1, so we may need to find an alternate way to display this. Suggestions (or patches ;) are welcome though.

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented

I think a way forward on this would be to provide a field that renders all the services available as single list, and then replace the columns with that, and then also provide a way to filter the view easily, so if you wanted to view just the servers with service X, that would be easy.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented

Perhaps something like this would work?


 +-------------------------------------------------------------------------------+
 |Server name                                + Services                          |
 |-------------------------------------------|-----------------------------------|
 |server1.mysite.com                         |apache, apache_ssl, mysql          |
 |server2.mysite.com                         |apache, apache_ssl, mysql          |
 |db.mysite.com                              |mysql                              |
 |db2.mysite.com                             |mysql                              |
 |                                           |                                   |
 +-------------------------------------------------------------------------------+

Just treat the services like Taxonomy terms instead of separate columns, then maybe give a way to filter by what service a server has on it. I think that would be sufficient. What do you think?

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented

Ha, you posted before I did. Let's do that. I'll build it today and roll a patch.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented

Turns out that I probably won't be working on this, so if somebody else wants to run with it, feel free :)

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Version: 6.x-2.x-dev » 7.x-3.x-dev
Issue summary: View changes
Status: Needs work » Fixed

Taking a queue from the suggestions above, we now provide a concatenated list of services. In a template_preprocess hook, we then expand these back out to separate headers and fields. The new view is identical to the original.

Fixed in af856ca (hosting) and 2a0f5b6 (hostmaster).

Status: Fixed » Closed (fixed)

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

  • Commit af856ca on 7.x-3.x, dev-helmo-3.x by ergonlogic: 
    Issue #1876354: Convert list of servers to use views.

  • Commit af856ca on 7.x-3.x, dev-helmo-3.x by ergonlogic: 
    Issue #1876354: Convert list of servers to use views.