Closed (fixed)
Project:
Balance Tracker
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jul 2009 at 09:17 UTC
Updated:
3 Aug 2009 at 14:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
brianV commentedI don't have any plans personally - Views integration is not my strong point. However, if anybody wants to submit a patch for it, I would be happy to roll it in.
Comment #2
hitfactory commentedPatches are not my strong point :) but I need this so attached is the 6.x-1.4 release with views integration. Maybe you can create a patch and others can test. At present it doesn't do anything fancy. It just exposes the data to Views for filtering, sorting on etc and allows relationships to be made via the user ID. No default views are provided since the obvious cases like user/%/balance are already handled by the module. If this is something others are interested in, perhaps it could be busted out into a Balance Tracker Views submodule.
Comment #3
brianV commentedkidrobot,
Thanks for taking charge on this. The new code seems to work pretty well in my (rather light) testing. I've decided that this is a 'must-have' feature for the next version of BT, and so I've committed your code to CVS. Any bugs found can be given their own issues this way.
The only change I would suggest is making the comments in balance_tracker.views.inc comply with the Drupal Coding Standards:
http://drupal.org/coding-standards#comment
I would appreciate it if people can take some time and test the new Views integration code that's in CVS - I want to ensure it is bombproof before releasing it. The CVS code can be checked out by following the instructions here: Checking out from the contributions repository. In short, the CVS command is:
cvs -z6 -d:pserver:anonymous:anonymous@cvs.drupal.org:/cvs/drupal-contrib checkout -d balance_tracker-HEAD contributions/modules/balance_trackerComment #4
brianV commented