{"action":"created","issue":{"url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/16","labels_url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/16/comments","events_url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/16/events","html_url":"https://github.com/BlueAcornInc/UniversalAnalytics/issues/16","id":98296707,"number":16,"title":"Category page - ec:addImpression fires for all products regardless of \"Products per Page\" value","user":{"login":"fweb","id":5377757,"avatar_url":"https://avatars.githubusercontent.com/u/5377757?v=3","gravatar_id":"","url":"https://api.github.com/users/fweb","html_url":"https://github.com/fweb","followers_url":"https://api.github.com/users/fweb/followers","following_url":"https://api.github.com/users/fweb/following{/other_user}","gists_url":"https://api.github.com/users/fweb/gists{/gist_id}","starred_url":"https://api.github.com/users/fweb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fweb/subscriptions","organizations_url":"https://api.github.com/users/fweb/orgs","repos_url":"https://api.github.com/users/fweb/repos","events_url":"https://api.github.com/users/fweb/events{/privacy}","received_events_url":"https://api.github.com/users/fweb/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-07-31T01:22:12Z","updated_at":"2015-09-21T19:27:10Z","closed_at":"2015-09-21T19:27:10Z","body":"Magento CE 1.8 with baua 0.5.4\r\n\r\nOn the category page, ec:addImpression is firing for all products in the category, regardless of the \"products per page\" value set in Configuration > Catalog > Products per Page [List & Grid]\r\n\r\nec:addProduct has this behavior as well.\r\n\r\nVisitor sees: Items 1 to 32 of 421\r\nbaua reports: 421 impressions\r\n\r\nThis artificially inflates the impression metric and probably negative affects performance.\r\n\r\nHave I done something wrong in the implementation? Any ideas?\r\n\r\nThank you =]"},"comment":{"url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/comments/142085109","html_url":"https://github.com/BlueAcornInc/UniversalAnalytics/issues/16#issuecomment-142085109","issue_url":"https://api.github.com/repos/BlueAcornInc/UniversalAnalytics/issues/16","id":142085109,"user":{"login":"fweb","id":5377757,"avatar_url":"https://avatars.githubusercontent.com/u/5377757?v=3","gravatar_id":"","url":"https://api.github.com/users/fweb","html_url":"https://github.com/fweb","followers_url":"https://api.github.com/users/fweb/followers","following_url":"https://api.github.com/users/fweb/following{/other_user}","gists_url":"https://api.github.com/users/fweb/gists{/gist_id}","starred_url":"https://api.github.com/users/fweb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fweb/subscriptions","organizations_url":"https://api.github.com/users/fweb/orgs","repos_url":"https://api.github.com/users/fweb/repos","events_url":"https://api.github.com/users/fweb/events{/privacy}","received_events_url":"https://api.github.com/users/fweb/received_events","type":"User","site_admin":false},"created_at":"2015-09-21T19:27:10Z","updated_at":"2015-09-21T19:27:10Z","body":"Alright, closing. The issue is caused by AITOC Manufacturers plugin v0.2.6 (now called \"Shop by Brands\"). Sharing the info below for future reference:\r\n\r\nAitoc_Aitmanufacturers_Block_Manufacturers_Brief::getCategoryItems() is getting called and returning a collection with all the products in the category on line 63\r\n\r\napp/code/local/Aitoc/Aitmanufacturers/Block/Manufacturers/Brief.php\r\n\r\nReplace lines 63-68 with:\r\n`$productIds = $productCollection->addCategoryFilter(Mage::registry('current_category'))->getAllIds();`\r\n\r\nThis update will have it use getAllIds() instead of loading the collection and looping over the items to get the ids. This prevents the UA event from firing and should prevent all the products from appearing at the bottom."}}