PrivateCoffee/travelynx
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

month overview: distinguish between no stats and no journeys

Browse source
This commit is contained in:
Daniel Friesel 2020-11-21 15:39:21 +01:00
parent ccfdd8007e
commit 0addce7bff
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/Travelynx/Model/Journeys.pm
View file

@ -749,9 +749,9 @@ sub get_months_for_year {
month => $row->{month} month => $row->{month}
} }
)->expand->hash; )->expand->hash;
if ($stats) {
$ret[ $row->{month} - 1 ][2] = $stats->{data}; # undef -> no journeys for this month; empty hash -> no cached stats
} $ret[ $row->{month} - 1 ][2] = $stats->{data} // {};
} }
} }
return @ret; return @ret;

2
templates/_history_months_for_year.html.ep
View file

@ -4,7 +4,9 @@
% for my $month (journeys->get_months_for_year(uid => current_user->{id}, year => $year)) { % for my $month (journeys->get_months_for_year(uid => current_user->{id}, year => $year)) {
% if (defined $month->[2]) { % if (defined $month->[2]) {
<a class="collection-item" href="/history/<%= $month->[0] %>"><%= $month->[1] %> <a class="collection-item" href="/history/<%= $month->[0] %>"><%= $month->[1] %>
% if (defined $month->[2]{km_route}) {
<span class="secondary-content"><%= sprintf('%.f', $month->[2]{km_route}) %> km</span> <span class="secondary-content"><%= sprintf('%.f', $month->[2]{km_route}) %> km</span>
% }
</a> </a>
% } % }
% else { % else {