public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
* [PATCH] services.cgi: Remove links from addon services
@ 2024-11-11  9:37 Adolf Belka
  2024-11-12 15:56 ` Bernhard Bitsch
  2024-11-18 20:07 ` Michael Tremer
  0 siblings, 2 replies; 3+ messages in thread
From: Adolf Belka @ 2024-11-11  9:37 UTC (permalink / raw)
  To: development

[-- Attachment #1: Type: text/plain, Size: 1031 bytes --]

- As discussed in Video Conf Call on Oct 7th 2024
- This patch removes the links from the addon services, in line with the change made
   to the core services.

Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
---
 html/cgi-bin/services.cgi | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/html/cgi-bin/services.cgi b/html/cgi-bin/services.cgi
index a3ae13c78..43babf522 100644
--- a/html/cgi-bin/services.cgi
+++ b/html/cgi-bin/services.cgi
@@ -161,9 +161,6 @@ END
 			foreach $service (split(/ /, "$metadata{'Services'}")) {
 				# Add addon name to displayname of service if servicename differs from addon
 				my $displayname = ($pak ne $service) ? "$service ($pak)" : $service;
-				if ( -e "/srv/web/ipfire/cgi-bin/$pak.cgi" ) {
-					$displayname = ($pak ne $service) ? "$service (<a href=\'$pak.cgi\'>$pak</a>)" : "<a href=\'$pak.cgi\'>$service</a>";
-				}
 
 				print "<td align='left' width='31%'>$displayname</td> ";
 
-- 
2.47.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] services.cgi: Remove links from addon services
  2024-11-11  9:37 [PATCH] services.cgi: Remove links from addon services Adolf Belka
@ 2024-11-12 15:56 ` Bernhard Bitsch
  2024-11-18 20:07 ` Michael Tremer
  1 sibling, 0 replies; 3+ messages in thread
From: Bernhard Bitsch @ 2024-11-12 15:56 UTC (permalink / raw)
  To: development

[-- Attachment #1: Type: text/plain, Size: 1226 bytes --]

Reviewed-by: Bernhard Bitsch <bbitsch(a)ipfire.org>
Tested-by: Bernhard Bitsch <bbitsch(a)ipfire.org>

Am 11.11.2024 um 10:37 schrieb Adolf Belka:
> - As discussed in Video Conf Call on Oct 7th 2024
> - This patch removes the links from the addon services, in line with the change made
>     to the core services.
> 
> Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
> ---
>   html/cgi-bin/services.cgi | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/html/cgi-bin/services.cgi b/html/cgi-bin/services.cgi
> index a3ae13c78..43babf522 100644
> --- a/html/cgi-bin/services.cgi
> +++ b/html/cgi-bin/services.cgi
> @@ -161,9 +161,6 @@ END
>   			foreach $service (split(/ /, "$metadata{'Services'}")) {
>   				# Add addon name to displayname of service if servicename differs from addon
>   				my $displayname = ($pak ne $service) ? "$service ($pak)" : $service;
> -				if ( -e "/srv/web/ipfire/cgi-bin/$pak.cgi" ) {
> -					$displayname = ($pak ne $service) ? "$service (<a href=\'$pak.cgi\'>$pak</a>)" : "<a href=\'$pak.cgi\'>$service</a>";
> -				}
>   
>   				print "<td align='left' width='31%'>$displayname</td> ";
>   


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] services.cgi: Remove links from addon services
  2024-11-11  9:37 [PATCH] services.cgi: Remove links from addon services Adolf Belka
  2024-11-12 15:56 ` Bernhard Bitsch
@ 2024-11-18 20:07 ` Michael Tremer
  1 sibling, 0 replies; 3+ messages in thread
From: Michael Tremer @ 2024-11-18 20:07 UTC (permalink / raw)
  To: development

[-- Attachment #1: Type: text/plain, Size: 1190 bytes --]

Acked-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 11 Nov 2024, at 09:37, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
> 
> - As discussed in Video Conf Call on Oct 7th 2024
> - This patch removes the links from the addon services, in line with the change made
>   to the core services.
> 
> Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
> ---
> html/cgi-bin/services.cgi | 3 ---
> 1 file changed, 3 deletions(-)
> 
> diff --git a/html/cgi-bin/services.cgi b/html/cgi-bin/services.cgi
> index a3ae13c78..43babf522 100644
> --- a/html/cgi-bin/services.cgi
> +++ b/html/cgi-bin/services.cgi
> @@ -161,9 +161,6 @@ END
> foreach $service (split(/ /, "$metadata{'Services'}")) {
> # Add addon name to displayname of service if servicename differs from addon
> my $displayname = ($pak ne $service) ? "$service ($pak)" : $service;
> - if ( -e "/srv/web/ipfire/cgi-bin/$pak.cgi" ) {
> - $displayname = ($pak ne $service) ? "$service (<a href=\'$pak.cgi\'>$pak</a>)" : "<a href=\'$pak.cgi\'>$service</a>";
> - }
> 
> print "<td align='left' width='31%'>$displayname</td> ";
> 
> -- 
> 2.47.0
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-11-18 20:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-11  9:37 [PATCH] services.cgi: Remove links from addon services Adolf Belka
2024-11-12 15:56 ` Bernhard Bitsch
2024-11-18 20:07 ` Michael Tremer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox