On a package page, show table listing sites and platforms using the package

A piece of a screencapture ...

It looks great! But please follow Aegir standards and rename it to hosting_package_use.

I think this is a great candidate for core itself, i don't see why it should be a separate module. Please work on patching the existing core if you have time, that would be great!

The pager doesn't seem like a big deal, but this would be a blocker however: #1515540: Show only sites and platforms accessible by the user.

Thanks for your contribution!

Great I would love to see this in Aegir.

This relates somewhat to: #1000368: Provide module version/install status across platforms

Here's a patch host hostmaster core.

* I've removed the ClientID column, as I don't really see a good use for it. Or it whould have to be turned into a nice link.
* Added a clear:both css style to prevent the list from overlapping with the task queue in Firefox.

Setting the correct version branch

I've added an extra WHERE clause to the query.
It now filters out information about deleted sites.

One other thought is that maybe we should check hosting_platform_client_access. This could be done with the code below, but I doubt that a regular client would access this page.

LEFT JOIN {hosting_platform_client_access} pca ON n2.nid = pca.pid
LEFT JOIN {hosting_client_user} c ON c.client = pca.cid

and in the WHERE clause:
AND (c.user=%d OR pca.pid IS NULL)

One other nice to have, but way out of scope here, would be to have info on site specific packages in the hostmaster database. Currently only packages from the platform are loaded in the hostmaster database.

Tested with 6.x-2.0-alpha1, works OK.

We are using it (the patch) in BOA for a long time already, however there is something to improve, I think, because it shows usage also for deleted sites - see the screenshot.

Oh, maybe we are using that older patch then? I will check that again.

Of course we have applied that older patch, sorry for the noise!

It is RTBC already.

patch doesn't apply anymore, sorry!

1 out of 1 hunk ignored -- saving rejects to file modules/hosting/hosting.css.rej
patching file modules/hosting/package/hosting_package.module
Hunk #1 FAILED at 325.
1 out of 1 hunk FAILED -- saving rejects to file modules/hosting/package/hosting_package.module.rej

@omega8 - can you pinpoint a commit or branch we can merge?

It seems to be just some lines that have shifted.
Patch -F1000 works wonders for these cases ;)

patch -p1 -F1000 < 1515416-10-package_usage_list.patch 
patching file modules/hosting/hosting.css
patching file modules/hosting/package/hosting_package.module
Hunk #1 succeeded at 325 with fuzz 3.

aah the fuzz, i always forget about the fuzz. -F3 was enough, even ;)

thanks!

After the Views integration, I believe we have lost this functionality. Either way, we should look at re-implementing it in Views. Since we're already doing this for install profile packages, hopefully it won't be too difficult.

OK, so this if in package/hosting_package.module:284 just kills it:

  if (!$node->package_type == 'profile') {

It was probably intended as:

  if ($node->package_type != 'profile') {

The view hosting_site_list is also shown for both modules and profiles. But only shows something for profiles.
What I'm missing there though is the package version info. Do we have views integration for the package instances?

ref: #588728: Meta: Views integration redux

I've re-opened #1876358: Convert list of packages to use views and posted a second issue #2019631: Views handler for package versions not numeric

wait, that means we can close this issue already. :)

No, sorry but #22 was extra. #20 and #21 are not resolved.

I think we need an extra view for this. One that takes a package as the basis, listing all sites that have the package enabled, with their respective version.

The current path: node/% (for packages)

Will take a look at this issue tomorrow.

Pretty simple view + Upgrade path:

Repo: http://drupalcode.org/sandbox/deciphered/2001932.git/shortlog/refs/heads...
Patch: http://drupalcode.org/sandbox/deciphered/2001932.git/patch/11bf084681546...

Thanks.
I'd like to filter Disabled modules by default. This patch adds an exposed filter...

One strange thing... in views_handler_field_hosting_package_status.inc we render t('Enabled') : t('Disabled'). Where do we need to declare that to also have it for the exposed filter?

Fixed in dd1995c (hostmaster) and 4f6297b (hosting).