elueckel / Unifi-Toolbox

Modul for Symcon integrating UniFi Network devices
2 stars 2 forks source link

Kleinere Rechtschreibkorrekturen in der Readme #9

Closed Brovning closed 3 years ago

Brovning commented 3 years ago

siehe Commits

Brovning commented 3 years ago

@elueckel : Die "error messages" die du im Form entfernt hast, werden verwendet und funktionier(t)en auch, sofern der Controller/DreamMachine nicht erreichbar war oder der Login fehlgeschlagen hat:

        if(false === $data)
        {
            $this->SendDebug($this->Translate("Authentication"), $this->Translate('Error: Not reachable / No response!'),0);

            // IP or Port not reachable / no response
            $this->SetStatus(200);

            return false;
        }
...
            else {
                $this->SendDebug($this->Translate("UniFi API Call"),$this->Translate("API could not be called - check the login data. Do you see a Cookie?"),0); 

                // login seems to be not successful
                $this->SetStatus(201);
            }

Willst du diese Fehlerüberprüfung entfernen? Oder hast du diesen Check nur übersehen?

elueckel commented 3 years ago

Bei mir hatte es "unbegründet Fehler" geworfen - ging alles, aber es war auf Rot. Deswegen hatte ich es entfernt. Am 2021-09-10 09:52, Brovning @.***> schrieb: > > > @elueckel (https://github.com/elueckel) : > Die "error messages" die du im Form entfernt hast, werden verwendet und funktionier(t)en auch, sofern der Controller/DreamMachine nicht erreichbar war oder der Login fehlgeschlagen hat: > > > if(false === $data) { $this->SendDebug($this->Translate("Authentication"), $this->Translate('Error: Not reachable / No response!'),0); // IP or Port not reachable / no response $this->SetStatus(200); return false; } ... else { $this->SendDebug($this->Translate("UniFi API Call"),$this->Translate("API could not be called - check the login data. Do you see a Cookie?"),0); // login seems to be not successful $this->SetStatus(201); } > > > Willst du diese Fehlerüberprüfung entfernen? > Oder hast du diesen Check nur übersehen? > > > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub (https://github.com/elueckel/Unifi-Toolbox/pull/9#issuecomment-916706522), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AHI57OY7QRV6TE6KDQQCGLTUBG2KNANCNFSM5DYWQ6UQ). > >

Brovning commented 3 years ago

Ah, ok. Siehe dir mal folgende Änderung an: https://github.com/Brovning/Unifi-Toolbox/commit/d0224875f3e4c3b5dde0be3329ce4b76da244e87#diff-9af1ac9620b9bb273778f2146d28440cea1e5051e7cb64767f5e42864dfba202

Brovning commented 3 years ago

@elueckel Funktioniert es mit der obigen Änderung bei dir?

elueckel commented 3 years ago

Ich hatte es jetzt nicht mehr probiert. Ich warte jetzt mal Feedback ab und finalisiere am WE. Dann schaue ich es mir nochmal an. Ich schmeisse dann die Auswahl für den Controller raus - geht sowieso nur mit der DM und dann kommt es mal in den Store. Am 2021-09-10 16:18, Brovning @.***> schrieb: > > > @elueckel (https://github.com/elueckel) > Funktioniert es mit der obigen Änderung bei dir? > > > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub (https://github.com/elueckel/Unifi-Toolbox/pull/9#issuecomment-916940706), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AHI57O43A74KIWI36OSAXHLUBIHT7ANCNFSM5DYWQ6UQ). > >

Brovning commented 3 years ago

Verstehe aber nicht, warum du die Controller-Auswahl rauswerfen willst.

Die nachfolgenden Werte funktionieren doch mit jedem Controller, oder?

Zumindest funktionierten diese in meinem Docker-Controller.

elueckel commented 3 years ago

Die Controller laufen alle auf einem Docker oder dem Cloud Key und bekommen nicht die Internetdaten. Wir bekommen sie auf der DM, da hier der Controller embedded ist. Der Kollege bei dem es nicht geht, hat eine USG und einen Controller ... da wird es nicht gehen - da es 2 getrennte Geräte sind. Am 2021-09-10 17:04, Brovning @.***> schrieb: > > > Verstehe aber nicht, warum du die Controller-Auswahl rauswerfen willst. > > > > Die nachfolgenden Werte funktionieren doch mit jedem Controller, oder? > > > > WAN1 External IP Address > > WAN2 External IP Address > > Unifi Network Version > > Update available > > Update downloaded > > Uptime > > > > Zumindest funktionierten diese in meinem Docker-Controller. > > > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub (https://github.com/elueckel/Unifi-Toolbox/pull/9#issuecomment-916974314), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AHI57O4XA5GUQTUT776AQB3UBINAXANCNFSM5DYWQ6UQ). > >