Refactor project module to use Views
nedjo - August 2, 2006 - 15:04
| Project: | Project |
| Version: | 5.x-1.x-dev |
| Component: | Projects |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed |
Description
In project browsing, I unwittingly put a lot of code in there that basically emulates what Views does much better. (I hadn't looked at Views yet.) Ideally I think we'd refactor it to use the Views module. See also this issue on project issue tracking: http://drupal.org/node/76725.
#1
see my comments in http://drupal.org/node/76725 (which apply here, too).
#2
see http://drupal.org/node/76725#comment-142606, in particular.
#3
Here's a very rough initial stab at views enabling project.module and project_release.module.
Right now the patch just adds the field definitions and a style plugin for displaying project overviews in a similar manner to how we display them now.
If it weren't for releases, this might be pretty close to finished. But I haven't figured out a good way to filter project nodes by whether there are any release nodes of a certain version associated with a given project. I'll take a crack at this when I get a chance, if someone doesn't beat me to it.
#4
There are some translatable strings inside this patch like
t('The project\'s homepage.'). This is not correct and should be fixed. It should look liket("The project's homepage.")if you have a single quote inside a translatable string. thx,#5
@hass: You are more than welcome to reroll the patch.
#6
I've re-rolled the patch with the backslash-single quotes removed from the strings that hass mentioned above.
#7
Reroll to remove offset and fuzz. I think this also still needs work, so setting status back to CNW.
#8
My understanding is that this is postponed for now and will be done as part of the port of the project module for Drupal 6. See http://groups.drupal.org/node/9500 for more information.
#9
Changing title so that when this issue is linked in other issues it's obvious which module this is for.