Drupal 10, the latest version of the open-source digital experience platform with even more features, is here.I would like to surface a link to the Hostmaster site. In devshop we don't link to the "Sites" page so it's really a pain to find the hostmaster site.
I figured since this is where you do things like Enable HTTPS, we should make the hostmaster site more accessible.
The upcoming patch adds a MENU_NORMAL_ITEM for /hostmaster, which simply redirects to hosting/c/hostmaster.
I know it seems weird but it's drupal. This is the simplest cleanest way to do it.
I thought through the alternative, and it would be messier: Adding a URL alias for /hostmaster and a primary link to that node programmatically, on install and update, isn't good. As a friend at the DrupalNorth sprints said: "No State".
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 2992202-hostmaster-link.patch | 1.02 KB | Jon Pugh |
Comments
Comment #3
Jon PughComment #4
Jon PughComment #5
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedThis would at least need some access check to not show it to people that are lacking 'administer aegir' permissions.
And maybe add a
'weight' => 100,line to get to the end of the menu list.Comment #6
Jon PughActually, i recommend using "drupal_valid_path" to check against "node/$nid".
This way all permissions are delegated to the node view page.
++On making it 100.
Comment #7
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedComment #9
Jon PughComment #10
helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commentedSorry, missing an access check
Comment #12
Jon PughComment #13
Jon PughSorry about that, I was thinking only of the urlpath, forgot visibility of the link.
Added drupal_valid_path() as it's the easiest way to check for access rights.
Comment #17
Jon Pugh