Hi,
At every level of the product partition tree, all product partitions need
to have the same type of caseValue
<https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductPartition#caseValue>
(ProductDimension
subclass) -- see our Shopping Campaigns guide
<https://developers.google.com/adwords/api/docs/guides/shopping#product_dimensions>
for
details.
In your example, you are trying to create a tree that looks like this:
ROOT <-- subdivision
-- ProductCanonicalCondition condition NEW <-- unit
-- ProductCanonicalCondition condition USED <-- unit
-- ProductType type null, value null <-- subdivision
-- ProductType type L1, value 'shirts' <-- unit
-- ProductType type L1, value 'footwear' <-- unit
There are two issues with this tree:
1. You have a combination of ProductCanonicalConditions and ProductType at
the first level below the root. For the 'other' condition at level one, you
need to have a ProductCanonicalCondition with a null condition
<https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductCanonicalCondition#condition>
because
all partitions at a level need to have the same type of *caseValue*.
2. The ProductType you have at level one needs to have a type
<https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductType#type>
value
(one of L1, L2, L3, L4, L5).
One example of a valid tree you could create using the dimensions in your
example is:
ROOT <-- subdivision
-- ProductCanonicalCondition condition NEW <-- unit
-- ProductCanonicalCondition condition USED <-- unit
-- ProductCanonicalCondition condition null <-- subdivision
-- ProductType type L1, value 'shirts' <-- unit
-- ProductType type L1, value 'footwear' <-- unit
-- ProductType type L1, value null <-- unit
You could also create a tree like this if you wanted to break down each of
NEW and USED by product type L1 'shirts' and 'footwear'.
ROOT <-- subdivision
-- ProductCanonicalCondition condition NEW <-- subdivision
-- ProductType type L1, value 'shirts' <-- unit
-- ProductType type L1, value 'footwear' <-- unit
-- ProductCanonicalCondition condition USED <-- subdivision
-- ProductType type L1, value 'shirts' <-- unit
-- ProductType type L1, value 'footwear' <-- unit
-- ProductType type L1, value null <-- unit
-- ProductCanonicalCondition condition null <-- unit
Please give that a try and let me know if you still have questions.
Thanks,
Josh, AdWords API Team
On Monday, November 24, 2014 5:42:55 AM UTC-5, Sarah Gilmore wrote:
>
> Hi Josh
>
> I've tried adding product level into the constuctor but get the following
> error:
>
> An error has occurred: [RequiredError.REQUIRED @
> operations[3].operand.criterion.caseValue.type]
>
> I've tried reducing it down to the bare minimum (as below) but stillt this
> error. All I really want is a tree to divide by product type but when I do
> this, it says 'product partition already exists' (despite being a brand new
> adgroup created within the API)
>
> Here is my current code, any other help appreciated
>
> Thanks
>
> * $adGroupCriterionService = $user->GetService('AdGroupCriterionService',*
> * ADWORDS_VERSION);*
> * $helper = new ProductPartitionHelper($adGroupId);*
> * $root = $helper->createSubdivision();*
> * $helper->createUnit($root, new ProductCanonicalCondition('NEW'),
> 200000);*
> * $helper->createUnit($root, new ProductCanonicalCondition('USED'),
> 100000);*
>
> * $otherCondition = $helper->createSubdivision($root, new ProductType());*
> * $helper->createUnit($otherCondition, new
> ProductType('PRODUCT_TYPE_L1','shirts'), 200000);*
> * $helper->createUnit($otherCondition, new
> ProductType('PRODUCT_TYPE_L1','footwear'), 100000);*
>
> * $result = $adGroupCriterionService->mutate($helper->getOperations());*
> * $children = array();*
> * $rootNode = null;*
>
>
>
>
> On Friday, November 21, 2014 4:52:00 PM UTC, Josh Radcliff (AdWords API
> Team) wrote:
>>
>> Hi,
>>
>> Similar to *ProductBiddingCategory*, when creating a *ProductType*
>> criterion,
>> you'll want to pass both:
>>
>> 1. The product type level (enum) - this is on the type
>>
>> <https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductType#type>
>> attribute
>> and should match one of the *PRODUCT_TYPE_Lx* enums of
>> ProductDimensionType
>>
>> <https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductDimensionType>
>> 2. The product type value (string) - this is on the value
>>
>> <https://developers.google.com/adwords/api/docs/reference/v201409/AdGroupCriterionService.ProductType#value>
>> attribute
>>
>> In your calls to the *ProductType* constructor you are only passing the
>> *value*, but the constructor expects the type and the value, in that
>> order. For example, for your "shirts" product type, you could modify your
>> constructor call to:
>>
>> new ProductType('PRODUCT_TYPE_L1', 'shirts')
>>
>> Please give that a try and let me know if you're still running into
>> errors.
>>
>> Cheers,
>> Josh, AdWords API Team
>>
>> On Friday, November 21, 2014 9:21:30 AM UTC-5, Sarah Gilmore wrote:
>>>
>>> Hi there
>>>
>>> I'm having trouble getting the Product Partition script to work in PHP.
>>> I'm using the example in the SDK. I've solved some errors by creating a new
>>> Adgroup from scratch within the API, but I get the following exception when
>>> I run the script:
>>>
>>> An error has occurred: Unmarshalling Error: cvc-enumeration-valid: Value
>>> 'shirts' is not facet-valid with respect to enumeration '[UNKNOWN,
>>> BIDDING_CATEGORY_L1, BIDDING_CATEGORY_L2, BIDDING_CATEGORY_L3,
>>> BIDDING_CATEGORY_L4, BIDDING_CATEGORY_L5, BRAND, CANONICAL_CONDITION,
>>> CUSTOM_ATTRIBUTE_0, CUSTOM_ATTRIBUTE_1, CUSTOM_ATTRIBUTE_2,
>>> CUSTOM_ATTRIBUTE_3, CUSTOM_ATTRIBUTE_4, OFFER_ID, PRODUCT_TYPE_L1,
>>> PRODUCT_TYPE_L2, PRODUCT_TYPE_L3, PRODUCT_TYPE_L4, PRODUCT_TYPE_L5]'
>>>
>>> I'm assuming this means that I'm entering invalid categories/making an
>>> invalid tree but I can't work out how. I've checked that the categories I'm
>>> using are in the product feed. I don't work much with Adwords itself - just
>>> the API - so any advice would be appreciated
>>>
>>>
>>> *function addProductPartitionTreeExample(AdWordsUser $user, $adGroupId)
>>> {*
>>>
>>> * // Get the AdGroupCriterionService, which loads the required classes.*
>>> * $adGroupCriterionService =
>>> $user->GetService('AdGroupCriterionService',*
>>> * ADWORDS_VERSION);*
>>> * $helper = new ProductPartitionHelper($adGroupId);*
>>> * // The most trivial partition tree has only a unit node as the root:*
>>> * // $helper->createUnit(null, null, 100000);*
>>> * $root = $helper->createSubdivision();*
>>> * $helper->createUnit($root, new ProductCanonicalCondition('NEW'),
>>> 200000);*
>>> * $helper->createUnit($root, new ProductCanonicalCondition('USED'),
>>> 100000);*
>>> * $otherCondition = $helper->createSubdivision($root,*
>>> * new ProductCanonicalCondition());*
>>> * $helper->createUnit($otherCondition, new ProductType('shirts'),
>>> 200000);*
>>> * $helper->createUnit($otherCondition, new ProductType('footwear'),
>>> 100000);*
>>> * $otherBrand =*
>>> * $helper->createSubdivision($otherCondition, new ProductType());*
>>> * // The value for the bidding category is a fixed ID for the 'Luggage
>>> & Bags'*
>>> * // category. You can retrieve IDs for categories from the
>>> ConstantDataService.*
>>> * // See the 'GetProductCategoryTaxonomy' example for more details.*
>>> * $helper->createUnit($otherBrand,*
>>> * new ProductBiddingCategory('BIDDING_CATEGORY_L1',*
>>> * '-3817140941569278349'), 750000);//for clothes*
>>> * $helper->createUnit($otherBrand,*
>>> * new ProductBiddingCategory('BIDDING_CATEGORY_L1'), 110000);*
>>> * // Make the mutate request.*
>>> * $result = $adGroupCriterionService->mutate($helper->getOperations());*
>>> * $children = array();*
>>> * $rootNode = null;*
>>> * // For each criterion, make an array containing each of its children*
>>> * // We always create the parent before the child, so we can rely on
>>> that here*
>>> * foreach ($result->value as $adGroupCriterion) {*
>>> * $children[$adGroupCriterion->criterion->id] = array();*
>>> * if (isset($adGroupCriterion->criterion->parentCriterionId)) {*
>>> * $children[$adGroupCriterion->criterion->parentCriterionId][] =*
>>> * $adGroupCriterion->criterion;*
>>> * } else {*
>>> * $rootNode = $adGroupCriterion->criterion;*
>>> * }*
>>> * }*
>>> * // Show the tree*
>>> * displayTree($rootNode, $children);*
>>>
>>> *}*
>>>
>>> Thanks
>>>
>>
--
--
=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~
Also find us on our blog and Google+:
https://googleadsdeveloper.blogspot.com/
https://plus.google.com/+GoogleAdsDevelopers/posts
=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~=~
You received this message because you are subscribed to the Google
Groups "AdWords API Forum" group.
To post to this group, send email to adwords-api@googlegroups.com
To unsubscribe from this group, send email to
adwords-api+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/adwords-api?hl=en
---
You received this message because you are subscribed to the Google Groups
"AdWords API Forum" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to adwords-api+unsubscr...@googlegroups.com.
Visit this group at http://groups.google.com/group/adwords-api.
To view this discussion on the web visit
https://groups.google.com/d/msgid/adwords-api/841db69f-651e-4849-8126-247da7a51793%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
