[gelöst] freetz-devel 3603: WebGUI-Breite laesst sich nicht aendern

[cuma]

Eigentlich doch nur wenn die Variable leer ist

[olistudent]

Die Variable hat noch den alten Wert und wird daher nicht auf den neuen Wert gesetzt. Ohne die if-Abfrage wird der neue Wert gleich übernommen.

MfG Oliver

[cuma]

Getestet hatte ich das nicht.. Aber ist das Problem nicht, dass der Webserver nicht neu startet (die pid gleich bleibt)?

[olistudent]

Bei mir startet der neu.

MfG Oliver

[cuma]

Bei mir auch, aber nur manchmal. Wenn das Löschen vom if hilft, mach es. Ich hatte aber in Erinnerung dass es da irgendwo Seiteneffekte gibt.
Hab gerade noch etwas beobachtet. Wenn man im Freetz-Webinterface etws lädt, spawnt httpd ein child. Evtl geht deshalb ein Reload per Wbinterface nicht immer richtig, bzw die falsche pid ist in der .pid Datei?

/var/mod/root # ps|grep 81|grep htt
22712 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
22714 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22712 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
22844 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
22957 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
23268 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
23494 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
/var/mod/root # ps|grep 81|grep htt
22842 root      1452 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz
23614 root      1468 S    httpd -P /var/run/webcfg.pid -p 81 -c /mod/etc/httpd.conf -h /usr/mww/ -r Freetz

[olistudent]

Index: root/usr/lib/libmodcgi.sh
===================================================================
--- root/usr/lib/libmodcgi.sh (revision 3607)
+++ root/usr/lib/libmodcgi.sh (working copy)
@@ -83,7 +83,7 @@
 EOF
 
 # custom style for fieldset and div.body
-if [ ! "$_cgi_width" ]; then
+if [ ! "$_cgi_width" ] || [ "$_cgi_width" != "$MOD_CGI_WIDTH" ]; then
  let _cgi_width=$MOD_CGI_WIDTH
 fi
 export _cgi_width

Oder so?

MfG Oliver

[cuma]

Ist auch 'ne gute Idee. Wenn es bei jemanden nicht klappt, wird sich dieser schon melden. Kann momentan nicht viel testen

[ao]

Vielen Dank für das Changeset 3624!

[buehmann]

-if [ ! "$_cgi_width" ]; then
+if [ ! "$_cgi_width" ] || [ "$_cgi_width" != "$MOD_CGI_WIDTH" ]; then
  let _cgi_width=$MOD_CGI_WIDTH
 fi

Dann könnte man die Bedingung auch komplett weglassen. So, wie es dasteht, hat _cgi_width in jedem Fall hinterher den Wert von MOD_CGI_WIDTH.

War der Sinn der _cgi_width-Geschichte bei seiner Einführung nicht, dass man die Standardbreite seitenweise überschreiben können sollte? (Ein grep hat ergeben, dass z.B. das Paket avm-firewall seine eigene Breite einstellt.)

Andreas

[olistudent]

Da hast du wohl Recht. Es gab da irgendein Problem mit dem Setzen von der Variable. Deshalb wurde die if-Abfrage eingeführt. Ich weiß aber nicht mehr genau was da los war. Wir können die Abfrage natürlich rausnehmen und schauen, ob jemand einen Fehler meldet.

MfG Oliver

[SaschaBr]

Ist es richtig, dass die Seite der AVM-Firewall eigentlich eine eigene Breite haben sollte? Die Breite dieser Seite wird aber jetzt mit geändert, wodurch es hier zu einem Darstellungsproblemen kommt.

[cuma]

Stimmt, das passt bei mir bei der "AVM-Firewall" auch nicht. Außerdem hängt rechts das Menü über den "Firewall-Regeln".
Ist es nicht sinnvoller das Problem selbst zu lösen, also den richtigen Restart vom httpd?