[Rt-devel] [PATCH] fix highlighting of site config values

Jesse Vincent jesse at bestpractical.com
Thu Sep 2 17:31:01 EDT 2010


Thanks. Applied to 3.9 (pushed soon)


On Mon 23.Aug'10 at 22:30:45 +0200, Christian Loos wrote:
> Hi,
> 
> attached an small fix to the site config highlighting on the
> configuration page.
> 
> -chris

> From 1151b079bb793e8ebbb6d657c2762968405e8b3b Mon Sep 17 00:00:00 2001
> From: Christian Loos <cloos at netsandbox.de>
> Date: Mon, 23 Aug 2010 22:17:53 +0200
> Subject: [PATCH] fix highlighting of site config values
> 
> Description contains the localized string for site config,
> so this worked only for englisch before.
> ---
>  share/html/Admin/Tools/Configuration.html |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/share/html/Admin/Tools/Configuration.html b/share/html/Admin/Tools/Configuration.html
> index 67ffa29..8fe4054 100644
> --- a/share/html/Admin/Tools/Configuration.html
> +++ b/share/html/Admin/Tools/Configuration.html
> @@ -103,7 +103,7 @@ foreach my $key ( RT->Config->Options( Overridable => undef, Sorted => 0 ) ) {
>  % }
>  </td>
>  <td class="collection-as-table" style="white-space: nowrap">
> -% if ( $description =~ /^.*site config$/ ) {
> +% if ( $meta->{'Source'}{'SiteConfig'} ) {
>  <span style="font-weight: bold"><% $description %></span>
>  % } else {
>  <% $description %>
> -- 
> 1.7.0.4
> 

> _______________________________________________
> List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel



More information about the rt-devel mailing list