It took me a bit to learn just how many paths there are to admin on the backend. I found the menus and settings weren't up to normal drupal 5 arrangement. That has been updated.

I added an extra menu on the backend to provide an overview of the affiliate program to people browsing the "top affiliates". I also added an option to provide instructions for newbies, on the front end.

Attached is the patch, I will also attach a screenshot to show the results. Some of the coding may not be up to par, in drupal's terms, as the names of functions and having multiple functions for similar tasks, but I think it is a great start. I hope it is the beginning of making this module more simpler to use and admin.

CommentFileSizeAuthor
#2 aff_screenshot1.gif15.7 KBelvis2
#1 aff_screenshot2.gif14.95 KBelvis2
affiliates_patch_drupal_5_0.diff10.47 KBelvis2

Comments

elvis2’s picture

elvis2 commented
Component: Code » User interface
StatusFileSize
new aff_screenshot2.gif14.95 KB

Screenshot of the backend.

elvis2’s picture

elvis2 commented
StatusFileSize
new aff_screenshot1.gif15.7 KB

Screenshot of frontend.

One thing I did want to mention, I am not sure if it is the module of something with this drupal install I am working with, but I have notice that many times IE will give me a white page when loading the /affiliates links. The rest of the drupal site loads fine. No errors in watchdog are showing up. If I close IE and reopen, it sometimes works.

drupalgirl’s picture

drupalgirl commented

Thanks. I'm sure this might be a silly question for experienced Drupal developers. Sorry about that. Is there a method for patching a module? Noticed the -/+. What steps to take for getting patch applied?

Thanks
Drupalgirl

elvis2’s picture

elvis2 commented

There is some general info here:
http://drupal.org/patch/apply

You can always do it by hand, removing the lines with the negative (-) sign and adding the items with the plus (+) sign. Just be sure to remove the plus signs before saving the file.

Good luck

mrboliao’s picture

mrboliao commented

There are bugs in your argument number, $op and $id are actually argument 4 and 5 respectively, after you've changed to the new url format.
I would believe all instances of $op = arg(2); and $id = (int)arg(3); be changed to $op = arg(4); and $id = (int)arg(5); respectively?

edited:
Also changed $ad_id = (int)arg(3); to $ad_id = $id;...

Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »
Anonymous’s picture

Anonymous (not verified) commented
Category: support » feature
Anonymous’s picture

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

Thanks for all your work here.

Would you mind submitting your patches against the latest development branches and ill review
them promptly.

Best, Paul

mdowsett’s picture

mdowsett commented

will it go towards the D6 version as well?

Anonymous’s picture

Anonymous (not verified) commented

I will be happy to test recent patches for D5 & D6.

Best, Paul

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Closed (fixed)

Please reopen if you have any work against the latest development branches for D5 / D6 that needs to reviewed.

Thanks Paul