[RFC PATCH v4 08/11] fff-web: Include new options into settings.html

Tim Niemeyer tim at tn-x.org
So Aug 20 14:56:14 CEST 2017


Hi

Insbesondere da es sich um ein RFC Patch handelt, wäre eine ausfürliche
Commit-Message sehr hilfreich.

Darüber hinaus sehe ich keinen wirklichen Grund das als RFC zu
schicken. ;)

Da es aber noch RFC ist, hier nur eine grobe Meinung zu dem Patch, ohne
mir die Umsetzung im Detail angeschaut zu haben:
Sehr gut.

Im Moment scheinen wir vor lauter Patches unter zu gehen, da benötigt
es leider etwas mehr "Struktur". Wenn du dann den Patch nochmal als
nicht RFC Patch schickst, mach bitte deutlich welche Patches zu dem Set
dazu gehören und welche Abhängigkeiten das Patchset zu möglicherweise
voherigen Patchsets hat.

Tim

Am Freitag, den 21.07.2017, 15:08 +0200 schrieb Adrian Schmutzler:
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> 
> Tested-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> ---
>  .../fff-web/files/www/ssl/cgi-bin/settings.html    | 72
> ++++++++++++++++++++--
>  1 file changed, 67 insertions(+), 5 deletions(-)
> 
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-
> bin/settings.html b/src/packages/fff/fff-web/files/www/ssl/cgi-
> bin/settings.html
> index 2ccc756..779e7eb 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
> @@ -37,7 +37,33 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then
>  		uci -q set "simple-tc.example.ifname=$(uci -q get
> network.wan.ifname)"
>  		uci -q set "simple-
> tc.example.limit_ingress=${POST_limit_ingress}"
>  		uci -q set "simple-
> tc.example.limit_egress=${POST_limit_egress}"
> -
> +		
> +		# Restliche Einstellungen
> +		uci -q set "fff.notifyupdate=webui"
> +		if [ "$POST_upgrade_notification" == "on" ] ; then
> +			uci -q set "fff.notifyupdate.value=1"
> +		else
> +			uci -q set "fff.notifyupdate.value=0"
> +			rm -f /tmp/fwcheck
> +			rm -f /tmp/isupdate
> +		fi
> +		if [ "$POST_upgrade_reset" == "on" ] ; then
> +			uci -q set
> "fff.notifyupdate.resetonupgrade=1"
> +		else
> +			uci -q set
> "fff.notifyupdate.resetonupgrade=0"
> +		fi
> +		
> +		if uci -q get system.poe_passthrough > /dev/null ;
> then
> +			uci -q set "fff.poe_passthrough=gpio"
> +			if [ "$POST_poe_passthrough" == "on" ] ;
> then
> +				uci -q set
> "fff.poe_passthrough.active=1"
> +				uci -q set
> "system.poe_passthrough.value=1"
> +			else
> +				uci -q set
> "fff.poe_passthrough.active=0"
> +				uci -q set
> "system.poe_passthrough.value=0"
> +			fi
> +		fi
> +		
>  		uci commit
>  		
>  		/usr/sbin/applysimpletc
> @@ -50,11 +76,22 @@ fi
>  <%in /www/ssl/cgi-bin/helpers %>
>  <%
>  # read
> -if [ "$(uci -q get 'fff. at trafficcontrol[0].enabled')" == "1" ] ;
> then
> -	traffic_checked='checked="checked" '
> +chkd='checked="checked" '
> +if [ "$(uci -q get 'fff. at trafficcontrol[0].enabled')" == "1" ] ;
> then # not set=0
> +	traffic_checked="$chkd"
>  else
>  	traffic_checked=""
>  fi
> +if [ "$(uci -q get 'fff.notifyupdate.value')" == "0" ] ; then # not
> set=1
> +	upgrade_checked=""
> +else
> +	upgrade_checked="$chkd"
> +fi
> +if [ "$(uci -q get 'fff.notifyupdate.resetonupgrade')" == "0" ] ;
> then # not set=1
> +	upgrade_reset_checked=""
> +else
> +	upgrade_reset_checked="$chkd"
> +fi
>  %>
>  <% show_msg %>
>  <form method="post">
> @@ -76,8 +113,9 @@ fi
>  		<tr><th>E-Mail Adresse:</th><td><input type="email"
> name="contact" value="<%= $(uci -q get 'fff. at system[0].contact') %>"
> /></td></tr>
>  	</table>
>  </fieldset>
> +<p><input type="submit" style="margin-top: 5px; margin-left: 3px;"
> /></p>
>  </td><td>
> -<fieldset style="min-height: 13em;" class="smallinput">
> +<fieldset style="min-height: 7em;" class="smallinput">
>  	<legend>Bitratenbegrenzung</legend>
>  	<table>
>  		<tr><th>Aktiviert:</th><td colspan="2">
> @@ -87,8 +125,32 @@ fi
>  		<tr><th>Freifunk Upload:</th><td><input
> type="text" name="limit_egress" value="<%= $(uci -q get 'fff. at traffic
> control[0].limit_out') %>" /></td><td>kbit/s</td></tr>
>  	</table>
>  </fieldset>
> +<fieldset style="min-height: 8em;" class="smallinput">
> +	<legend>Weitere Einstellungen</legend>
> +	<table>
> +		<tr><th>Show upgrade notification:</th><td
> colspan="2">
> +			<input type="checkbox"
> name="upgrade_notification" <%= ${upgrade_checked} %>/>
> +		</td></tr>
> +		<tr><th>Reset notification setting after
> upgrade:</th><td colspan="2">
> +			<input type="checkbox" name="upgrade_reset"
> <%= ${upgrade_reset_checked} %>/>
> +		</td></tr>
> +		
> +		<%
> +		if uci -q get system.poe_passthrough > /dev/null ;
> then
> +			if [ "$(uci -q get
> 'fff.poe_passthrough.active')" == "1" ] ; then # not set=0
> +				poe_pt_checked="$chkd"
> +			else
> +				poe_pt_checked=""
> +			fi
> +		%>
> +		<tr><th>Enable PoE passthrough:</th><td colspan="2">
> +			<input type="checkbox"
> name="poe_passthrough" <%= ${poe_pt_checked} %>/>
> +		</td></tr>
> +		<% fi %>
> +		
> +	</table>
> +</fieldset>
>  </td></tr>
>  </table>
> -<p><input type="submit" style="margin-top: 5px; margin-left: 3px;"
> /></p>
>  </form>
>  <%in /www/ssl/cgi-bin/footer %>
> -- 
> 2.7.4
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20170820/e6d044da/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev