[v2,3/5] services.cgi: minor cosmetics

Message ID 20221006175958.11036-4-robin.roevens@disroot.org
State Superseded
Headers
Series Fix Bug#12935 + cosmetic changes/enhancements |

Commit Message

Robin Roevens Oct. 6, 2022, 5:59 p.m. UTC
  * Singular 'Service' instead of plural 'Services' as column header of
  services table
* Sort list of services

Signed-off-by: Robin Roevens <robin.roevens@disroot.org>
---
 html/cgi-bin/services.cgi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Bernhard Bitsch Oct. 6, 2022, 6:52 p.m. UTC | #1
Reviewed-by: Bernhard Bitsch <bbitsch@ipfire.org>

Am 06.10.2022 um 19:59 schrieb Robin Roevens:
> * Singular 'Service' instead of plural 'Services' as column header of
>    services table
> * Sort list of services
> 
> Signed-off-by: Robin Roevens <robin.roevens@disroot.org>
> ---
>   html/cgi-bin/services.cgi | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/html/cgi-bin/services.cgi b/html/cgi-bin/services.cgi
> index de946d755..e35b04cae 100644
> --- a/html/cgi-bin/services.cgi
> +++ b/html/cgi-bin/services.cgi
> @@ -105,7 +105,7 @@ if ( $querry[0] =~ "processescpu"){
>   <div align='center'>
>   <table width='80%' cellspacing='1' class='tbl'>
>   <tr>
> -	<th align='left'><b>$Lang::tr{'services'}</b></th>
> +	<th align='left'><b>$Lang::tr{'service'}</b></th>
>   	<th align='center' ><b>$Lang::tr{'status'}</b></th>
>   	<th align='center'><b>PID</b></th>
>   	<th align='center'><b>$Lang::tr{'memory'}</b></th>
> @@ -174,7 +174,7 @@ END
>   	# Generate list of installed addon pak services
>   	my %paklist = &Pakfire::dblist("installed");
>   
> -	foreach my $pak (keys %paklist) {
> +	foreach my $pak (sort keys %paklist) {
>   		my %metadata = &Pakfire::getmetadata($pak, "installed");
>   			
>   		my $service;
  

Patch

diff --git a/html/cgi-bin/services.cgi b/html/cgi-bin/services.cgi
index de946d755..e35b04cae 100644
--- a/html/cgi-bin/services.cgi
+++ b/html/cgi-bin/services.cgi
@@ -105,7 +105,7 @@  if ( $querry[0] =~ "processescpu"){
 <div align='center'>
 <table width='80%' cellspacing='1' class='tbl'>
 <tr>
-	<th align='left'><b>$Lang::tr{'services'}</b></th>
+	<th align='left'><b>$Lang::tr{'service'}</b></th>
 	<th align='center' ><b>$Lang::tr{'status'}</b></th>
 	<th align='center'><b>PID</b></th>
 	<th align='center'><b>$Lang::tr{'memory'}</b></th>
@@ -174,7 +174,7 @@  END
 	# Generate list of installed addon pak services
 	my %paklist = &Pakfire::dblist("installed");
 
-	foreach my $pak (keys %paklist) {
+	foreach my $pak (sort keys %paklist) {
 		my %metadata = &Pakfire::getmetadata($pak, "installed");
 			
 		my $service;