Here are some new features that i will work on next week.

# Highest climbers (which affiliates are referring more than they did before )
# Additional UI for adding/editing/deleting categories
# Allow the possibility of circumventing writing to the affiliates tables for the case when affiliate referrals are being
collected by parsing apache logs ( a colleague has a working perl script that can do exactly this )
# Show affiliate banners on the list / edit pages

Best, Paul

CommentFileSizeAuthor
#14 affiliates.336656.2e.patch533 bytesAnonymous (not verified)
#11 affiliates.336656.2d.patch13.3 KBAnonymous (not verified)
#9 affiliates.336656.2c.patch13.22 KBAnonymous (not verified)
#8 affiliates.336656.2b.patch11.3 KBAnonymous (not verified)
#7 affiliates.336656.2.patch10.34 KBAnonymous (not verified)
#3 affiliates.336656.1.patch10.66 KBAnonymous (not verified)
#2 Picture 2.png22.09 KBAnonymous (not verified)
#2 Picture 3.png163.41 KBAnonymous (not verified)
#2 Picture 4.png71.45 KBAnonymous (not verified)

Comments

kbahey’s picture

kbahey commented

1 and 2 are great, but please be mindful of performance. Use caching where appropriate to avoid excessive queries on every page load.

3 is very interesting. If the script can be licensed under the GPL, then we can include it in a tools directory, with your colleague credited on it. I am sure it is helpful to others.

4 I don't understand which list/edit pages.

All in all, go for it!

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new Picture 4.png71.45 KB
new Picture 3.png163.41 KB
new Picture 2.png22.09 KB

A collection of screen shots of what i had in mind for #4

As soon i have something working ill release a patch for peer review. I'll see if i can get #3
done today.

Paul

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new affiliates.336656.1.patch10.66 KB

First patch for review.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
kbahey’s picture

kbahey commented

I gave it a visual inspection, and it looks OK. Some queries are scary, but you are caching them. The update query with a subselect is scary too, but if it works, then I am fine with it.

Perhaps at some point we should move all the points storage to the Userpoints module and let it be our "bank" for points. This way other actions can happen when points are gained, and lots of other modules can be used (e.g. role changes, badges integration with userpoints, ...etc.).

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Active

Hi Khalid

Userpoints module integration sounds v. interesting.

I have committed the work in the above patch into CVS @ DRUPAL-6--1.
As soon as the development branch is updated ill make a request to have these
changes pulled onto spreadthunderbird stage server @ mozilla for some final testing and then
you will be able to check out the changes on spreadthunderbird.com shortly after

I have asked my colleague if he would make his apache parsing script generic so that it
can be made available for download on drupal.org. Ill get back to you on that shortly.
Here is the script as it currenty stands...
http://viewvc.svn.mozilla.org/vc/projects/spreadfirefox.com/logs-parse/a...

Tomorrow i will work on ..

#Additional UI for adding/editing/deleting categories

Best, Paul

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new affiliates.336656.2.patch10.34 KB

Second patch . Almost ready for review .

I'll do some more testing on Monday . The clock has beaten me :-(

Paul

Anonymous’s picture

Anonymous (not verified) commented
Assigned: » Unassigned
Status: Active » Needs review
StatusFileSize
new affiliates.336656.2b.patch11.3 KB

Second patch for review.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new affiliates.336656.2c.patch13.22 KB

Adding the ability to host buttons on the hosting server.

Anonymous’s picture

Anonymous (not verified) commented

#Show affiliate banners on the list / edit pages

I'll see if i can show the affiliate banner on the list page using jquery tooltips

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new affiliates.336656.2d.patch13.3 KB

fixed a problem with editing a banner and uploading a new banner

Anonymous’s picture

Anonymous (not verified) commented

Committed the recent changes to the development branch.

Please test and provide feedback.

Best, Paul

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Active
Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »
StatusFileSize
new affiliates.336656.2e.patch533 bytes

Small change which allows you to automatically view your banners @ affiliates/admin provided you have lightbox2 installed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

jusyjim’s picture

jusyjim commented

Hi, Tuning in late but can I ask if these upgrades will ever happen for the 5.x version?

Thanks.