Andreas/tracks
1
0
Fork 0
mirror of https://github.com/TracksApp/tracks.git synced 2025-12-28 12:58:48 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Shave off slight redundancy

Browse source
This commit is contained in:
Katrina Owen 2013-03-17 19:47:35 -06:00
parent f3a076c2af
commit 9f3470f9dc
1 changed files with 7 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

15
app/controllers/stats_controller.rb
View file

@ -347,26 +347,25 @@ class StatsController < ApplicationController
def prep_context_data_for_view(all_actions_per_context)
sum = all_actions_per_context.inject(0){|sum, apc| sum += apc['total'].to_i }
sum = all_actions_per_context.inject(0){|sum, apc| sum + apc['total']}
pie_cutoff=10
size = [all_actions_per_context.size, pie_cutoff].min
# explicitely copy contents of hash to avoid ending up with two arrays pointing to same hashes
# explicitly copy contents of hash to avoid ending up with two arrays pointing to same hashes
actions_per_context = Array.new(size){|i| {
'name' => all_actions_per_context[i]['name'],
'total' => all_actions_per_context[i]['total'].to_i,
'total' => all_actions_per_context[i]['total'],
'id' => all_actions_per_context[i]['id']
} }
if all_actions_per_context.size > pie_cutoff
actions_per_context[size-1]['name']=t('stats.other_actions_label')
actions_per_context[size-1]['total']=actions_per_context[size-1]['total']
actions_per_context[size-1]['id']=-1
size.upto(all_actions_per_context.size-1){ |i| actions_per_context[size-1]['total']+=(all_actions_per_context[i]['total'].to_i) }
actions_per_context[-1]['name']=t('stats.other_actions_label')
actions_per_context[-1]['id']=-1
size.upto(all_actions_per_context.size-1){ |i| actions_per_context[-1]['total']+=(all_actions_per_context[i]['total']) }
end
@pie_slices = Array.new(size){|i| actions_per_context[i]['total'].to_i*100/sum }
@pie_slices = Array.new(size){|i| actions_per_context[i]['total']*100/sum }
@pie_labels = Array.new(size){|i| actions_per_context[i]['name'].truncate(15, :omission => '...') }
@pie_links = Array.new(size){|i| context_path(actions_per_context[i]['id'])}
end