* Wed Nov 05 2025 Brian Read <brianr@koozali.org> 11.0.0-132.sme

- Supress webmail option on menu and in user accounts if webmail not enabled [SME: 12997]

root/usr/share/smanager/lib/SrvMngr/Controller/Emailsettings-Custom.pm

@@ -17,6 +17,8 @@ use Data::Validate::IP qw(is_ipv4 is_ipv6);
 use constant FALSE => 0;
 use constant TRUE  => 1;
 
+our @EXPORT_OK = qw(get_current_webmail_status);
+
 
 #The most common ones - open DB when required.
 our $cdb; 

root/usr/share/smanager/lib/SrvMngr/Controller/Hostentries.pm

@@ -116,16 +116,39 @@ sub do_update {
     $ndb = esmith::NetworksDB::UTF8->open || die "Couldn't open networks db";
     my $notif     = '';
     my $result    = '';
-    $hos_datas{'name'}       = lc $c->param('Name');
-    $hos_datas{'domain'}     = lc $c->param('Domain');
-    $hos_datas{'hostname'}   = $c->param('Hostname');
-    $hos_datas{'comment'}    = $c->param('Comment');
-    $hos_datas{'hosttype'}   = $c->param('Hosttype');
-    $hos_datas{'internalip'} = $c->param('Internalip');
-    $hos_datas{'macaddress'} = $c->param('Macaddress');
-    $hos_datas{'externalip'} = $c->param('Externalip');
+	# Fetch parameters with forced scalar context and default empty string if undefined
+	$hos_datas{'name'}       = lc(scalar $c->param('Name') // '');
+	$hos_datas{'domain'}     = lc(scalar $c->param('Domain') // '');
+	$hos_datas{'hostname'}   = scalar $c->param('Hostname') // '';
+	$hos_datas{'comment'}    = scalar $c->param('Comment') // '';
+	$hos_datas{'hosttype'}   = scalar $c->param('Hosttype') // '';
+	$hos_datas{'internalip'} = scalar $c->param('Internalip') // '';
+	$hos_datas{'externalip'} = scalar $c->param('Externalip') // '';
+
 	my $hostname = "$hos_datas{'name'}.$hos_datas{'domain'}";
 	
+
+	if (my $hostrec = $hdb->get($hostname)) {
+		my $hosttype = $hostrec->prop('HostType') // '';
+		#$c->app->log->info("$hosttype $hos_datas{'hosttype'} $hos_datas{'comment'} $hostrec->prop('Comment')");
+		# Clear comment if hosttype changes to 'self' and comment was not intentionally changed
+		if ($hosttype ne 'Self' 
+			&& $hos_datas{'hosttype'} eq 'Self' 
+			&& $hos_datas{'comment'} eq $hostrec->prop('Comment')) {
+			$hos_datas{'comment'} = '';
+		}
+	}
+
+	# Clear MAC address if hosttype is 'self', otherwise get from param
+	if ($hos_datas{'hosttype'} eq 'Self') {
+		$hos_datas{'macaddress'} = '';
+		$hos_datas{'internalip'} = '';
+		#$c->app->log->info("yes $hos_datas{'hosttype'} $hos_datas{'macaddress'}");
+	} else {
+		$hos_datas{'macaddress'} = scalar $c->param('Macaddress') // '';
+		#$c->app->log->info("no $hos_datas{'hosttype'} $hos_datas{'macaddress'}");
+	}
+
     if ($trt eq 'ADD') {
         $hos_datas{'hostname'} = $hostname;
 

root/usr/share/smanager/lib/SrvMngr/Controller/Useraccounts.pm

@@ -179,7 +179,7 @@ sub do_update {
         $res = $c->pseudonym_clash($first);
         $result .= $res unless $res eq 'OK';
 
-        if ($mail) {
+        if (defined $mail) {
             $res = $c->emailforward($mail);
             $result .= $res unless $res eq 'OK';
         }
@@ -217,7 +217,7 @@ sub do_update {
         $res = $c->pseudonym_clash($first);
         $result .= $res unless $res eq 'OK';
 
-        if ($mail) {
+        if (defined $mail) {
             $res = $c->emailforward($mail);
             $result .= $res unless $res eq 'OK';
         }
@@ -554,27 +554,31 @@ sub pseudonym_clash {
 
 sub emailforward {
     my ($c, $data) = @_;
-    my $response = $c->email_simple($data);
+    #$c->app->log->info("emailformward called with $data!");
 
-    if ($response eq "OK") {
-        return "OK";
-    } elsif ($data eq "") {
+    # Trim whitespace from $data
+    $data =~ s/^\s+|\s+$//g if defined $data;
 
-        # Blank is ok, only if we're not forwarding, which means that the
-        # EmailForward param must be set to 'local'.
+    # Check simple email validation first
+    return "OK" if $c->email_simple($data) eq "OK";
+
+    # If trimmed data is empty
+    if ($data eq "") {
         my $email_forward = $c->param('EmailForward') || '';
         $email_forward =~ s/^\s+|\s+$//g;
         return 'OK' if $email_forward eq 'local';
         return $c->l('usr_CANNOT_CONTAIN_WHITESPACE');
-    } else {
-        return $c->l('usr_CANNOT_CONTAIN_WHITESPACE')
-            if ($data =~ /\s+/);
+    }
 
-        # Permit a local address.
-        return "OK" if $data =~ /^[a-zA-Z][a-zA-Z0-9\._\-]*$/;
+    # Reject if $data contains any whitespace inside
+    return $c->l('usr_CANNOT_CONTAIN_WHITESPACE') if $data =~ /\s/;
+
+    # Allow local address pattern
+    return "OK" if $data =~ /^[a-zA-Z][a-zA-Z0-9._-]*$/;
+
+    # Otherwise reject for unacceptable chars
     return $c->l('usr_UNACCEPTABLE_CHARS');
-    } ## end else [ if ($response eq "OK")]
-} ## end sub emailforward
+}
 
 sub get_groups {
     my ($c) = shift;

root/usr/share/smanager/lib/SrvMngr/I18N/Modules/Roundcubepanel/roundcubepanel_en.lex

@@ -1 +1 @@
-
+'rc_WEBMAIL_DISABLED' => 'Webmail is disabled in Email Settings panel',

root/usr/share/smanager/themes/default/public/css/new-sme-main.css

@@ -519,3 +519,8 @@ div.roundcube #roundcube{
   margin-left: auto!important;
   margin-right: auto;
 }
+
+table.incolumn {
+	margin:auto;
+	color:red;
+}

root/usr/share/smanager/themes/default/public/css/new_sme.css

@@ -122,6 +122,7 @@
     max-width: 100%;
     position: relative;
     margin: auto;
+    margin-top:-20px;
 }
 
 #header2 {
@@ -291,3 +292,7 @@ background-color: #e8f3e1;
 .busy {
 	cursor: wait; /* Change the cursor to a 'wait' cursor */
 }
+
+#swt_theme {
+	margin-left:10px;
+}

root/usr/share/smanager/themes/default/public/js/flag-by-locale.js

@@ -263,8 +263,12 @@ document.addEventListener('DOMContentLoaded', () => {
 
 function displayLocaleAndFlag() {
     // Get the browser locale
-    const userLocale = navigator.language || navigator.userLanguage;
+    const userLocale = navigator.languages && navigator.languages.length
+    ? navigator.languages[0]
+    : navigator.language;
+        
     //alert(`User Locale: ${userLocale}`); // Alert the detected locale
+	console.log(navigator.languages); // Log language to console
 
     const { flag, isUnknown, countryCode } = getFlagEmoji(userLocale);
     

root/usr/share/smanager/themes/default/templates/partials/_grp_list.html.ep

@@ -35,13 +35,13 @@
 
         % foreach my $group (@$groups)
         % {
+		  % my $csrf_token = "TOKEN"; # CSRF token for security
+		  % my $group_name = $group->key; # group name extracted from the data structure
           <tr>
             %= t td => ( class => 'sme-border' ) => $group->key
             %= t td => ( class => 'sme-border' ) => $group->prop('Description')
             <td class='sme-border' style="min-width:15em">
               % my $modify_text = l('MODIFY'); # Localized text
-              % my $csrf_token = "TOKEN"; # CSRF token for security
-              % my $group_name = $group->key; # group name extracted from the data structure
               % my $actionModify = qq{
                 % <a href="groups2?CsrfDef=$csrf_token&trt=UPD&group=$group_name"
                 % class="sme-modify-button unset ui-button ui-corner-all ui-widget ui-button-icon-only"
@@ -52,8 +52,6 @@
                 % </a>
             % };
           % my $remove_text = l('REMOVE'); # Localized text
-          % my $csrf_token = "TOKEN"; # CSRF token for security
-          % my $group_name = $group->key; # group name extracted from the data structure
           % my $actionRemove = qq{
             % <a href="groups2?CsrfDef=$csrf_token&trt=DEL&group=$group_name"
             % class="sme-remove-button unset ui-button ui-corner-all ui-widget ui-button-icon-only"

root/usr/share/smanager/themes/default/templates/partials/_usr_list.html.ep

@@ -1,5 +1,7 @@
 <div id='usr_list'>
 
+% use constant FALSE => 0;
+% use constant TRUE  => 1;
 
   % my $btn = l('usr_ADD_USER');
 
@@ -153,7 +155,9 @@
   <%= $c->render_to_string( inline => $actionResetPw ) %>
   <%= $c->render_to_string( inline => $actionLock ) %>
   <%= $c->render_to_string( inline => $actionRemove ) %>
+  % if ($c->SrvMngr::Controller::Emailsettings::get_current_webmail_status(FALSE) ne  'disabled'){
 	<%= $c->render_to_string( inline => $actionroundcube ) %>
+  % }
 </td>
 </tr>
 % }

root/usr/share/smanager/themes/default/templates/partials/_usr_upd.html.ep

@@ -131,7 +131,7 @@
         %= l 'usr_GROUP_MEMBERSHIPS'
       </span>
       <span class=data>
-        <table class="sme-border "><thead>
+        <table class="sme-border incolumn"><thead>
             <tr><th class='sme-border'>
                 %= l 'usr_MEMBER'
               </th><th class='sme-border'>

root/usr/share/smanager/themes/default/templates/portforwarding.html.ep

@@ -5,11 +5,8 @@
 
     % if (config->{debug} == 1) {
       <p>
-        %= dumper "<pf>" . $c->current_route
-          %= dumper $c->stash("ret")
-          %= dumper $c->stash("portforwarding")
-          % my $ref = $pf_datas->{portforwarding};
-          %= dumper $ref->{TCP}->[0] . "</pf>"
+        %= dumper $c->current_route
+        %= dumper $pf_datas
       </p>
     % }
 

root/usr/share/smanager/themes/default/templates/roundcube.html.ep

@@ -1,8 +1,7 @@
 % layout 'default', title => "Sme server 2 - roundcube";
-
+% use constant FALSE => 0;
+% use constant TRUE  => 1;
 % content_for 'module' => begin
-
-
   <div id='roundcube' class='roundcube roundcube-panel module'>
     % if (config->{debug} == 1) {
       <p>
@@ -15,8 +14,10 @@
     </div>
   % }
   <h1><%= $title %></h1><br>
+  % if ($c->SrvMngr::Controller::Emailsettings::get_current_webmail_status(FALSE) eq 'disabled'){
+		<%= $c->render_to_string( inline => l('rc_WEBMAIL_DISABLED') ) %>
+  % } else {
 	<p>If the Webmail panel does not appear after logging in, then please check your password.</p><br />
-
 	% my $thisdomain = $c->req->url->to_abs->host;
 	% my $url = $c->stash('modul');
 	% if (!$url) {
@@ -29,12 +30,10 @@
 	% if (!($url =~ /https:/)) {
 		% $url = 'https://' . $url;
 	% }
-
 	<div class='roundcube'>
 		<a href="<%= "https://" . $thisdomain . "/" %>roundcube?_user=<%= $username %>"><button class="sme-fullwindow1-button" width=20em title="<%= l('Full Window') %>"><%= l('Full Window') %></button></a>
 		<object id="roundcube" data="<%= $url %>" title="<%= $c->stash('title') %>"  type="text/html" ><%= $c->stash('title') %> not found</object>
 	</div>
+  % }
 </div>
-
-
 % end

smeserver-manager.spec

@@ -2,7 +2,7 @@ Summary: Sme Server Configuration : Manager 2
 %define name smeserver-manager
 Name: %{name}
 %define version 11.0.0
-%define release 124
+%define release 132
 Version: %{version}
 Release: %{release}%{?dist}
 License: GPL
@@ -147,6 +147,30 @@ true
 %defattr(-,root,root)
 
 %changelog
+* Wed Nov 05 2025 Brian Read <brianr@koozali.org> 11.0.0-132.sme
+- Supress webmail option on menu and in user accounts if webmail not enabled [SME: 12997]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-131.sme
+- Add fix for IE browser local/language detect [SME: 13039]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-130.sme
+- Adjust heading so that no white line under theme selector and move theme button in [SME: 13057]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-129.sme
+- Useraccounts: Clean up forward email sub and make sure blank is errored  [SME: 13056]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-128.sme
+- Move group table to under others in User accounts setup panel [SME: 13068]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-127.sme
+- Group panel: Move same var declations to outside innner scope [SME: 13073]
+
+* Tue Nov 04 2025 Brian Read <brianr@koozali.org> 11.0.0-126.sme
+- Remove debugging dump in portforwarding which crashes if no data  [SME: 13243]
+
+* Mon Nov 03 2025 Brian Read <brianr@koozali.org> 11.0.0-125.sme
+- Arrange that Macaddress, InternalIP and comment cleared out when host entry switched to self [SME: 13207]
+
 * Fri Oct 24 2025 Brian Read <brianr@koozali.org> 11.0.0-124.sme
 - Adjust CSS for logout button to remove overlap of border and rounding [SME: 13247]