Description:
fix bug on calculating best of each lanuage
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
-
r438:1c02c7fc8a89 - - 1 file changed: 1 inserted, 1 deleted
| @@ -143,47 +143,47 | |||||
|
|
143 | value: sub.submitted_at, |
|
143 | value: sub.submitted_at, |
|
|
144 | sub_id: sub.id |
|
144 | sub_id: sub.id |
|
|
145 | } |
|
145 | } |
|
|
146 | end |
|
146 | end |
|
|
147 |
|
147 | ||
|
|
148 | if @by_lang[lang.pretty_name][:length][:value] > sub.effective_code_length |
|
148 | if @by_lang[lang.pretty_name][:length][:value] > sub.effective_code_length |
|
|
149 | @by_lang[lang.pretty_name][:length] = { |
|
149 | @by_lang[lang.pretty_name][:length] = { |
|
|
150 | avail: true, |
|
150 | avail: true, |
|
|
151 | user_id: sub.user_id, |
|
151 | user_id: sub.user_id, |
|
|
152 | value: sub.effective_code_length, |
|
152 | value: sub.effective_code_length, |
|
|
153 | sub_id: sub.id |
|
153 | sub_id: sub.id |
|
|
154 | } |
|
154 | } |
|
|
155 | end |
|
155 | end |
|
|
156 | end |
|
156 | end |
|
|
157 |
|
157 | ||
|
|
158 | #process user_id |
|
158 | #process user_id |
|
|
159 | @by_lang.each do |lang,prop| |
|
159 | @by_lang.each do |lang,prop| |
|
|
160 | prop.each do |k,v| |
|
160 | prop.each do |k,v| |
|
|
161 | v[:user] = User.exists?(v[:user_id]) ? User.find(v[:user_id]).full_name : "(NULL)" |
|
161 | v[:user] = User.exists?(v[:user_id]) ? User.find(v[:user_id]).full_name : "(NULL)" |
|
|
162 | end |
|
162 | end |
|
|
163 | end |
|
163 | end |
|
|
164 |
|
164 | ||
|
|
165 | #sum into best |
|
165 | #sum into best |
|
|
166 | if @by_lang and @by_lang.first |
|
166 | if @by_lang and @by_lang.first |
|
|
167 | - @best = @by_lang.first[1] |
|
167 | + @best = @by_lang.first[1].clone |
|
|
168 | @by_lang.each do |lang,prop| |
|
168 | @by_lang.each do |lang,prop| |
|
|
169 | if @best[:runtime][:value] >= prop[:runtime][:value] |
|
169 | if @best[:runtime][:value] >= prop[:runtime][:value] |
|
|
170 | @best[:runtime] = prop[:runtime] |
|
170 | @best[:runtime] = prop[:runtime] |
|
|
171 | @best[:runtime][:lang] = lang |
|
171 | @best[:runtime][:lang] = lang |
|
|
172 | end |
|
172 | end |
|
|
173 | if @best[:memory][:value] >= prop[:memory][:value] |
|
173 | if @best[:memory][:value] >= prop[:memory][:value] |
|
|
174 | @best[:memory] = prop[:memory] |
|
174 | @best[:memory] = prop[:memory] |
|
|
175 | @best[:memory][:lang] = lang |
|
175 | @best[:memory][:lang] = lang |
|
|
176 | end |
|
176 | end |
|
|
177 | if @best[:length][:value] >= prop[:length][:value] |
|
177 | if @best[:length][:value] >= prop[:length][:value] |
|
|
178 | @best[:length] = prop[:length] |
|
178 | @best[:length] = prop[:length] |
|
|
179 | @best[:length][:lang] = lang |
|
179 | @best[:length][:lang] = lang |
|
|
180 | end |
|
180 | end |
|
|
181 | if @best[:first][:value] >= prop[:first][:value] |
|
181 | if @best[:first][:value] >= prop[:first][:value] |
|
|
182 | @best[:first] = prop[:first] |
|
182 | @best[:first] = prop[:first] |
|
|
183 | @best[:first][:lang] = lang |
|
183 | @best[:first][:lang] = lang |
|
|
184 | end |
|
184 | end |
|
|
185 | end |
|
185 | end |
|
|
186 | end |
|
186 | end |
|
|
187 | end |
|
187 | end |
|
|
188 | end |
|
188 | end |
|
|
189 | end |
|
189 | end |
You need to be logged in to leave comments.
Login now