Needs work
Project:
Balance Tracker
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Sep 2010 at 08:28 UTC
Updated:
16 Nov 2016 at 17:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
brianV commentedWow - that's an awesome patch. Thanks for submitting this back!
I am probably going to review and add these features one at a time in different issues rather than accepting a single, monolithic patch. Given that I have very limited availability at this point, it would be incredibly helpful if someone were able to break the different components of this patch into different issues, so each can be pulled individually.
Comment #2
brianV commentedI am reviewing portions of this patch one by one, and committing as necessary.
Token integration has been committed to dev.
http://drupal.org/cvs?commit=447242
http://drupal.org/cvs?commit=447244
Comment #3
Ciprian Oancia commentedPatch fails on several lines
Comment #4
brianV commentedParts of this patch are already integrated into core, so yes, the patch doesn't apply cleanly...
Comment #5
Ciprian Oancia commentedI saw, i had to do it by hand :)
Comment #6
brianV commentedCiprian:
Please let me know how the other portions of the patch test out. I don't have time to really test them right now, but I would like to roll some of these features in if I can.
If you really want to help, it would really be helpful for me to have the separate portions of this patch into their own issue, so each can be tested and applied individually instead of a single large ball of patch.
Comment #7
Ciprian Oancia commentedHi :)
I wish i knew more than basic php & drupal 7 behaviours as for example with rules integration i still have bugs eq:
Unable to add this event as it does not provide all variables utilized by the reaction rule unlabeled.
I do attach the rules integration .inc file as its outputed by the applied patch.
And i do hope in some day someone would implement #1064648: Payements , right now im one of the users in need of such thing :)
Comment #8
kiphaas7 commentedI took the effort to break up the initial patch in 5 somewhat smaller patches. They are cascading; in other words, one needs to apply first #1, then #2, etc...
Comment #9
ramy_asha commentedWill such a patch ever exist for Drupal 7.x? I've installed Balance Tracker on one of our test sites and it's only 80% of what I'm looking for, I need "Invite Commissions" to work but it relies on "CA" which is missing, and from what I've seen is obsolete and replaced by the "rules" module. How can I integrate Invite Commissions with rules? Or better yet is there any way to get Balance Tracker & Invite Commissions fully functional on Drupal 7.x? Thanks in advance,
Comment #10
sergeypavlenko commentedThank you very much for the patches, I will try soon to apply all patches for your version of the module 7.x-1.x-dev.
After fixing all the errors of the module, the module and complete the transition to Drupal 7, we'll port "Invite Commissions" on D7.
Commits:
2_tab.patch - faebe53
4_admin_form.patch - 599f387 ( Was not added to the patch - the parameter "balance_table_sort", because it does need to write custom functionality. Many lines of patch previously been in the module. )
5_post_cleanup.patch - ecf69e6
Comment #11
kevinquillen commentedIs this a patch for D7 or D6?
Comment #12
brianV commentedpretty sure this patch was submitted against the D6 version. However, parts of it are already rolled into core.