ApplicationStatsExtractor should aggregate earnings, not purchase_price

Bug #788133 reported by Anthony Lenton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Developer registration portal
Fix Released
Medium
Danny Tamez

Bug Description

ApplicationStatsExtractor.purchase_stats_summary aggregates subscription's purchase_price, so the "Purchases" column in the application overview list shows an incorrectly high value.

Same goes for ApplicationStatsExtractor.subscriptions_by_distroseries, which causes the application detailed stats page to show (consistently with the application overview list) the same incorrect high value.

Until we run a batch script to pull in tax information from pay, these values should remain on $0. Longer term, the pay api should provide tax information so that we can generate them automatically.

Steps for QA:
 - Create an application
 - Create a couple of subscriptions via the admin interface for an application and manually set their earnings field
 - Check the list of applications and the stats page for the application itself. In both cases the reported value should be the total sum of the "earnings" field.
 - Check the accumulated revenue graph in the stats page. It too should be showing the correct accumulated earnings.

Tags: kb-defect sp-1
Changed in developer-portal:
importance: Undecided → Medium
status: New → Confirmed
tags: added: kb-defect sp-1
Changed in developer-portal:
assignee: nobody → Danny Tamez (zematynnad)
Changed in developer-portal:
status: Confirmed → In Progress
Danny Tamez (zematynnad)
Changed in developer-portal:
status: In Progress → Fix Committed
description: updated
Revision history for this message
Dave Morley (davmor2) wrote :

Reminder to check the math on production we don't want users thinking canonical is taking 70%

Changed in developer-portal:
milestone: none → 11.06
status: Fix Committed → Fix Released
Changed in developer-portal:
status: Fix Released → Fix Committed
Changed in developer-portal:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.