Bugfix für verschobene Headerleiste bei Befehlen

[Blackcat]

Hallo ihr Lieben,

nachdem mehreren Usern aufgefallen ist, dass bei Styles, die keine absolute Position des Headers setzen, der Header nach unten rutscht. Habe ich nach der Ursache gesucht:

Das HTML Dokument erzeugt für cmds erst den content und dann den Rest. Lösung wäre deshalb den Aufruf der Headerinit einfach vor die cmd Logik zu schieben:

Code Auswählen Erweitern

Index: C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm
===================================================================
--- C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm (revision 6053)
+++ C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm (working copy)
@@ -661,7 +661,9 @@
     $FW_cmdret = $FW_activateInform = "";
     $cmd = "style eventMonitor";
   }
-
+ 
+  FW_roomOverview($cmd);
+ 
   if($FW_cmdret) {
     $FW_detail = "";
     $FW_room = "";
@@ -693,7 +695,6 @@

   }

-  FW_roomOverview($cmd);
   if($FW_contentFunc) {
     no strict "refs";
     my $ret = &{$FW_contentFunc}($arg);


Kann das jemand noch prüfen und ggf. einchecken, oder mir die Freigabe zum einchecken geben? Ist ja nicht meine Ecke

PS: Beispiel des Fehlverhaltens:
http://forum.fhem.de/index.php/topic,23406.msg174004.html#msg174004

[maxritti]

Hi Sandra,

gerade habe ich mal Deinen Patch in meiner Entwicklungsumgebung eingespielt.
Das sieht gut aus.
Nun verschwindet nichts mehr an den unteren Rand.

Wobei ich diese 01_FHEMWEB.pm habe.

Code Auswählen Erweitern

# $Id: 01_FHEMWEB.pm 6031 2014-06-02 07:19:40Z rudolfkoenig $

Oder hat die Revision 6053 in deinem Patchfile nichts mit der $Id zu tun?

Beim einchecken kann ich allerdings nicht helfen 

[Blackcat]

Wobei ich diese 01_FHEMWEB.pm habe.

Code Auswählen Erweitern

# $Id: 01_FHEMWEB.pm 6031 2014-06-02 07:19:40Z rudolfkoenig $

Oder hat die Revision 6053 in deinem Patchfile nichts mit der $Id zu tun?

Hm.. habe vorher extra geupdatet ... denke mal die Id ist vielleicht eine andere?
Aber Rudolf wird wahrscheinlich die Änderungen einspielen, da er alle commits gemacht hatte, daher denke ich wird er das mit der Id wissen (kann ja auch sein, dass mein SVN das diff file nicht richtig benannt hat)

[betateilchen]

Die 6053 (aktueller SVN Stand zum Zeitpunkt der Erstellung des patch) aus dem patch hat nichts direkt mit der 6031 der tatsächlichen FHEMWEB Version zu tun. Kein Grund zur Sorge.

[rudolfkoenig]

Habs eingecheckt.

[Blackcat]

Danke