From: Adolf Belka <adolf.belka@ipfire.org>
Date: Thu, 11 Feb 2021 12:41:29 +0000 (+0100)
Subject: wirelessclient.cgi: Fix for bug #12571
X-Git-Tag: v2.25-core155~100
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f90b0f24a21e95fea350262967558557b89145b8;p=ipfire-2.x.git

wirelessclient.cgi: Fix for bug #12571

- Wirelessclient shows priority 0 to be most preferred and priority 4 as
least preferred. Based on forum posters experience and the wpa_supplicant
man page it is the other way round.
- This patch moves the least preferred title to priority 0 and vice versa
- Will ask bug reporter to test out the patch and confirm it works. The page
is only shown if you have a wifi connection on red.

Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---

diff --git a/html/cgi-bin/wirelessclient.cgi b/html/cgi-bin/wirelessclient.cgi
index e8c3c96284..b1679c1285 100644
--- a/html/cgi-bin/wirelessclient.cgi
+++ b/html/cgi-bin/wirelessclient.cgi
@@ -602,11 +602,11 @@ sub showEditBox() {
 					</td>
 					<td width='40%'>
 						<select name='PRIO'>
-							<option value="0" $selected{'PRIO'}{'0'}>0 ($Lang::tr{'most preferred'})</option>
+							<option value="0" $selected{'PRIO'}{'0'}>0 ($Lang::tr{'least preferred'})</option>
 							<option value="1" $selected{'PRIO'}{'1'}>1</option>
 							<option value="2" $selected{'PRIO'}{'2'}>2</option>
 							<option value="3" $selected{'PRIO'}{'3'}>3</option>
-							<option value="4" $selected{'PRIO'}{'4'}>4 ($Lang::tr{'least preferred'})</option>
+							<option value="4" $selected{'PRIO'}{'4'}>4 ($Lang::tr{'most preferred'})</option>
 						</select>
 					</td>
 					<td colspan="2" width='40%'></td>