In IRC, killes asked me if we could hide any project nodes from the browsing pages if they only had a 4.6.x release (e.g. http://drupal.org/project/citizenspeak not showing up at http://drupal.org/project/Modules/category/66). I thought the Active Drupal Core compatibility terms checkboxes at http://drupal.org/admin/project/project-release-settings would solve this, since removing a term from the "active" list means you can't filter by that version anymore, etc. Unfortunately, there's a bug in project_page_overview in project.module where it doesn't use this setting when filtering release nodes when browsing by all versions.
Furthermore, there's a bug that when you filter by all versions, we don't discard unpublished release nodes. :( So, we end up violating what I thought was the semantics, that if a project has no published releases, it doesn't show up in the browsing pages. Currently, it does. To test this (and since killes doesn't maintain it anymore), I unpublished both of the releases for http://drupal.org/node/68898 (which are 4.6.x only). And lo, Mailcommand still shows up at http://drupal.org/project/Modules/category/66.
Totally untested, but I'm pretty sure the attached patch will solve both problems.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | project_browse_active_tids.patch_1.txt | 3.69 KB | dww |
| project_browse_active_tids.patch.txt | 2 KB | dww |
Comments
Comment #1
dwwthis works fine once you expand a given category (e.g. http://drupal.org/project/Modules/category/66) but the category summary at http://drupal.org/project/Modules would still have the wrong totals. :( working on it...
Comment #2
dwwsomething like this. also untested.
Comment #3
dwwyay, tested on scratch.d.o and this solves all the above problems, and fixes some related bugs, too. ;)
Comment #4
dwwthis works, and killes won't review it, so i committed to HEAD and DRUPAL-4-7--2, and installed on d.o. ;)
Comment #5
(not verified) commented