| View previous topic :: View next topic |
| Author |
Message |
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
 Posted: Tue May 01, 2007 9:04 pm Post subject: |
 |
|
Yes, assuming your question is not instructory. It seemed simple to me at first. Data recorded to transaction table fine, so just borrow that code I thought. Got tripped up though. Because it appears at time of reconcile it is looking to transactions table for data query and writing data received via IPN script as well? Probably something simple, I'm sure.
About code comparison from your work today too. Your thoughts on my IPN code update? I used the IPN manual at the Club here and replaced snippets of code.
-Kev
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
 |
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
|
| Back to top |
|
|
emilacosta
dadaAdmin
Joined: Sep 19, 2005
Posts: 219
Location: Miami, FL
|
| Posted: Wed May 02, 2007 9:13 am Post subject: |
|
|
Situation like this make me wish PHP-Nuke has a CSV or a Code module where you can see the last version of the modified code. I have been thinking about this a lot lately so provably I will make one module for that.
Now coming back to the NukeTreasury module I have to say that I like your new version a lot more than the original. I have use this module in the pass and it was a pain to manage. My question is simple:
From where the table in the Treasury Financial Register should fill after it reconcile the IPN from Paypal? Should it pull data from the nuke_treasury_transactions table? if so:
Date = payment_date
Num = txn_id
Name = first_name + last_name
Decsription = txn_type
Amount = mc_gross
I would like to suggest add to that table at least one more column call payment status and also concatenate txn_type with memo to have a full description. Also it is always a good idea to have in the same module the thank you and the cancellation page with the choice to use other if you want.
Your thoughts?
_________________
"In the computer business you’re either a one or a zero and I am determined never to be zero." |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Wed May 02, 2007 9:44 am Post subject: |
|
|
| emilacosta wrote: | | Situation like this make me wish PHP-Nuke has a CSV or a Code module where you can see the last version of the modified code. I have been thinking about this a lot lately so provably I will make one module for that. |
What about NSN Repository Module from NukeScripts?
| emilacosta wrote: | | I like your new version a lot more than the original. I have use this module in the pass and it was a pain to manage |
Thanks! It is a modded version of original TechGFX release. I thought much cleaner than the current 1.1 available at http://www.code-authors.com. The IPN script was the main focus with me. More a 'Frankenstein' script now  Borrowed from IPN manual and two other web pages I searched out. Notice I added more variables for future expansion?
| emilacosta wrote: | From where the table in the Treasury Financial Register should fill after it reconcile the IPN from PayPal? Should it pull data from the nuke_treasury_transactions table? if so:
Date = payment_date
Num = txn_id
Name = first_name + last_name
Decsription = txn_type
Amount = mc_gross
I would like to suggest add to that table at least one more column call payment status and also concatenate txn_type with memo to have a full description. Also it is always a good idea to have in the same module the thank you and the cancellation page with the choice to use other if you want.
Your thoughts? |
Yes, it appears it is doing that now, with pulling data from transactions table. So it should stay. Maintain the spirit of original script without deviating (exception for updating legacy code) is my philosophy. I would prefer that the current columns display the info they are supposed to, i.e...item number, amount, etc... I know the other items are hard coded, but could we not pass along the same variables that the transactions table is using? That data is recording correctly for transactions. And with regards to improvements, it appears the author left room for more options in DB table for transactions.
-Kev
_________________
Visit the DaDaNuke Wish List
Last edited by dadaBIT on Mon Aug 06, 2007 12:32 pm; edited 1 time in total |
|
| Back to top |
|
|
blackacid
dadaClub
Joined: Feb 21, 2007
Posts: 23
|
| Posted: Thu May 03, 2007 9:42 am Post subject: |
|
|
have you guys fixed the pull data of number not showing on admin panel? if not let me know so i can work on it too emil you know hwere to find me thanks  |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Thu May 03, 2007 9:55 am Post subject: |
|
|
Hi blackacid and thank you for your patience in this matter. Still working on it. I'm sure it is some thing simple like reusing variables already present. The data logged to transaction table is correct and more than enough to populate financial table. Any ideas would be appreciated. Did you download the version I just uploaded to Club?
Regards,
Kevin
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
|
blackacid
dadaClub
Joined: Feb 21, 2007
Posts: 23
|
| Posted: Thu May 03, 2007 9:57 am Post subject: |
|
|
| i didnt download yet but ill get it now so i can work with yah hehehe |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Thu May 03, 2007 10:04 am Post subject: |
|
|
Great! What I'll do now is add NukeTreasury to our CVS, that way we can keep up to date with latest changes.
-Kev
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
|
blackacid
dadaClub
Joined: Feb 21, 2007
Posts: 23
|
| Posted: Fri May 11, 2007 6:56 am Post subject: |
|
|
hey yall im back and look what i got here hehehe
hit me back on yahoo EMIL if you want this fix file on nuketreasury |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Fri May 11, 2007 7:38 am Post subject: |
|
|
Hello, thanks for the contribution! Attach the fix here or if you wish, I'll give you access to the CVS for update (ChangeLog needs to be appended).
-Kev
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
|
blackacid
dadaClub
Joined: Feb 21, 2007
Posts: 23
|
| Posted: Fri May 11, 2007 7:42 am Post subject: |
|
|
| can you pm me on yahoo messenger or msn messenger? |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Fri May 11, 2007 7:49 am Post subject: |
|
|
Good Morning from Southeast Michigan and thanks for the speedy response! If I may, suggest using any one of IM choices just below this message to reach me. 
-Kevin
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
|
blackacid
dadaClub
Joined: Feb 21, 2007
Posts: 23
|
| Posted: Fri May 11, 2007 7:53 am Post subject: |
|
|
| well wont let me pm you on yahoo messenger and ill pm you here on site pm my yahoo id cuz emil is not online right now lol he have me added on yahoo :p |
|
| Back to top |
|
|
dadaBIT
General Manager
Joined: Jun 23, 2005
Posts: 582
Location: United States
|
| Posted: Fri May 11, 2007 6:23 pm Post subject: |
|
|
Greetings my ever erstwhile friend. Check your private messages. 
_________________
Visit the DaDaNuke Wish List |
|
| Back to top |
|
|
|
Forum FAQ
Search
Usergroups
Profile
Log in
