My favorites | Sign in
Project Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 134: account_id and assoc_id not set in TarzanCore constructor
1 person starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  r...@tarzan-aws.com
Closed:  Apr 2009
Type-Defect
Priority-Medium
Milestone-2.0.4
Module-AAWS
Usability


Sign in to add a comment
 
Reported by hammjazz, Apr 01, 2009
1. put your accociateid in the define('AWS_ASSOC_ID', 'your-id') in
config.inc.php

2. issue: 
   
   $aaws = new AmazonAAWS();
   $list = $aaws->similarity_lookup('B00000JIOY'); 

3. Have a look at body->Items->Item->DetailPageURL field of the returned XML. 

The URL should contain "&tag=your-id" but it doesn't. 

I have changed the constructor of TarzanCore to set assoc_id and account_id
based on variables passed in or defines. Look here http://www.pastie.org/433609


Comment 1 by r...@tarzan-aws.com, Apr 01, 2009
(No comment was entered for this change.)
Status: Accepted
Owner: r...@tarzan-aws.com
Labels: Milestone-2.0.4 Module-AAWS Usability
Comment 2 by tommusic, Apr 02, 2009
I also ran across this. The code to which hammjazz linked has an error on line 48, by the way. It tests for 
AWS_ACCOUNT_ID when it probably ought to test AMS_ASSOC_ID.
Comment 3 by r...@tarzan-aws.com, Apr 29, 2009
Fixed in branch and trunk @ r299.
Comment 4 by r...@tarzan-aws.com, Apr 29, 2009
(No comment was entered for this change.)
Status: Fixed
Sign in to add a comment

Hosted by Google Code