Fix 2210

[antgonza]

Adds sample counts by prep in the listing. Deciding to go via PrepTemplate.keys() cause it and cache the values as it saves us some code/queries.

Now, the study list looks like this:

The study/sample selected like this:

[josenavas]

Given test pass

[codecov-io]

Codecov Report

Merging #2246 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #2246      +/-   ##
==========================================
+ Coverage   92.81%   92.81%   +<.01%     
==========================================
  Files         171      171              
  Lines       18906    18917      +11     
==========================================
+ Hits        17547    17558      +11     
  Misses       1359     1359

Impacted Files	Coverage Δ
qiita_db/util.py	96.55% <100%> (+0.03%)	⬆️
...iita_pet/handlers/api_proxy/tests/test_artifact.py	98.22% <100%> (+0.03%)	⬆️
...pet/handlers/study_handlers/tests/test_artifact.py	97.19% <100%> (+0.1%)	⬆️
qiita_db/test/test_util.py	99.78% <100%> (ø)	⬆️
qiita_db/metadata_template/util.py	90.97% <0%> (-0.07%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a369cad...b7e2cc9. Read the comment docs.

[josenavas]

Thanks @tanaes 🍺

[tanaes]

I inevitably make a bunch of 'wait I don't understand why...' comments on these PRs but then spend a bunch of time scrolling back and forth on the diffs and then I'm like 'ahhh oh I see.' :P

…

On Mon, Aug 21, 2017 at 12:44 PM Jose Navas ***@***.***> wrote: Thanks @tanaes <https://github.com/tanaes> 🍺 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2246 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AH6JALYCVOGewXeQ5mKydElF6iFl0eZsks5sad4rgaJpZM4O8w75> .