diff --git a/lib/App/Netdisco/Web/Plugin/Inventory.pm b/lib/App/Netdisco/Web/Plugin/Inventory.pm index 71dc40e3..cd5cd9ba 100644 --- a/lib/App/Netdisco/Web/Plugin/Inventory.pm +++ b/lib/App/Netdisco/Web/Plugin/Inventory.pm @@ -16,19 +16,53 @@ get '/inventory' => require_login sub { my $platforms = schema(vars->{'tenant'})->resultset('Device')->get_platforms(); my $releases = schema(vars->{'tenant'})->resultset('Device')->get_releases(); - my %release_map = ( + my %release_version_map = ( map { (join '', map {sprintf '%05s', $_} split m/(\D)/, ($_->{os_ver} || '')) => $_ } $releases->hri->all ); - my @release_list = - map { $release_map{$_} } - sort {(($release_map{$a}->{os} || '') cmp ($release_map{$b}->{os} || '')) || ($a cmp $b)} + + my %release_map = (); + map { push @{ $release_map{ $release_version_map{$_}->{os} } }, $release_version_map{$_} } + grep { $release_version_map{$_}->{os} } + grep { $_ } + sort {(lc($release_version_map{$a}->{os} || '') cmp lc($release_version_map{$b}->{os} || '')) || ($a cmp $b)} + keys %release_version_map; + + my %release_totals = + map { $_ => {rows => scalar @{ $release_map{$_} }, count => 0} } + grep { $_ } keys %release_map; + foreach my $r (keys %release_totals) { + map { $release_totals{$r}->{count} += $_->{count} } + @{ $release_map{ $r } }; + } + + my %platform_map = (); + map { push @{ $platform_map{$_->{vendor}} }, $_ } + grep { $_->{vendor} } + grep { $_ } + sort {(lc($a->{vendor} || '') cmp lc($b->{vendor} || '')) || (lc($a->{model} || '') cmp lc($b->{model} || ''))} + $platforms->hri->all; + + my %platform_totals = + map { $_ => {rows => scalar @{ $platform_map{$_} }, count => 0} } + grep { $_ } + keys %platform_map; + + foreach my $r (keys %platform_totals) { + map { $platform_totals{$r}->{count} += $_->{count} } + @{ $platform_map{ $r } }; + } + var(nav => 'inventory'); template 'inventory', { - platforms => [ $platforms->hri->all ], - releases => [ @release_list ], + platforms => [ sort keys %platform_totals ], + releases => [ sort keys %release_totals ], + platform_map => \%platform_map, + release_map => \%release_map, + platform_totals => \%platform_totals, + release_totals => \%release_totals, }, { layout => 'main' }; }; diff --git a/share/config.yml b/share/config.yml index 964ffd4a..72b966dd 100644 --- a/share/config.yml +++ b/share/config.yml @@ -244,6 +244,9 @@ table_pagesize: 10 table_showrecordsmenu: - [10, 25, 50, 100, '-1'] - [10, 25, 50, 100, 'All'] +inventory_collapse_threshold: 0 +inventory_platforms_collapse_threshold: 5 +inventory_releases_collapse_threshold: 5 vlanctl: true portctl_nameonly: false portctl_no: [] diff --git a/share/views/inventory.tt b/share/views/inventory.tt index c7cd8082..aa58815e 100644 --- a/share/views/inventory.tt +++ b/share/views/inventory.tt @@ -12,26 +12,55 @@ - [% FOREACH platform IN platforms %] - [% NEXT UNLESS platform.vendor OR platform.model %] + [% SET vendorcount = 0 %] + [% FOREACH vendor IN platforms %] + [% SET vendorcount = vendorcount + 1 %] + [% IF settings.inventory_collapse_threshold + AND platforms.size > settings.inventory_collapse_threshold + AND platform_totals.$vendor.rows > settings.inventory_platforms_collapse_threshold %] - [% IF platform.vendor %] - [% platform.vendor | html_entity %] + href="[% search_device | none %]&q=[% vendor | uri %]&vendor=[% vendor | uri %]"> + [% vendor | html_entity %] + + + + + Show [% platform_totals.$vendor.rows | html_entity %] + + + [% platform_totals.$vendor.count | html_entity %] + + [% END %] + [% FOREACH row IN platform_map.$vendor %] + [% IF settings.inventory_collapse_threshold + AND platforms.size > settings.inventory_collapse_threshold + AND platform_totals.$vendor.rows > settings.inventory_platforms_collapse_threshold %] + + [% ELSE %] + + [% END %] + + [% IF row.vendor %] + + [% row.vendor | html_entity %] [% ELSE %]unknown[% END %] - [% IF platform.model %] + [% IF row.model %] - [% platform.model | html_entity %] + href="[% search_device | none %]&q=[% row.model | uri %]&model=[% row.model | uri %]"> + [% row.model | html_entity %] [% ELSE %]unknown[% END %] - [% platform.count | html_entity %] + [% row.count | html_entity %] [% END %] + [% END %] @@ -46,26 +75,56 @@ - [% FOREACH release IN releases %] - [% NEXT UNLESS release.os OR release.os_ver %] + [% SET oscount = 0 %] + [% FOREACH os IN releases %] + [% SET oscount = oscount + 1 %] + [% IF settings.inventory_collapse_threshold + AND releases.size > settings.inventory_collapse_threshold + AND release_totals.$os.rows > settings.inventory_releases_collapse_threshold %] - [% IF release.os %] - [% release.os | html_entity %] + href="[% search_device | none %]&q=[% os | uri %]&vendor=[% os | uri %]"> + [% os | html_entity %] + + + + + Show [% release_totals.$os.rows | html_entity %] + + + [% release_totals.$os.count | html_entity %] + +
+ [% END %] + [% FOREACH row IN release_map.$os %] + [% IF settings.inventory_collapse_threshold + AND releases.size > settings.inventory_collapse_threshold + AND release_totals.$os.rows > settings.inventory_releases_collapse_threshold %] + + [% ELSE %] + + [% END %] + + [% IF row.os %] + + [% row.os | html_entity %] [% ELSE %]unknown[% END %] - [% IF release.os_ver %] + [% IF row.os_ver %] - [% release.os_ver | html_entity %] + href="[% search_device | none %]&q=[% row.os_ver | uri %]&os_ver=[% row.os_ver | uri %]"> + [% row.os_ver | html_entity %] [% ELSE %]unknown[% END %] - [% release.count | html_entity %] + [% row.count | html_entity %] [% END %] + [% END %]
@@ -76,3 +135,22 @@ [% END %] +