2
0
mirror of https://github.com/fhem/fhem-mirror.git synced 2025-01-31 06:39:11 +00:00

74_Unifi: removed deprecated UnifiSwitch-functions!

git-svn-id: https://svn.fhem.de/fhem/trunk@18717 2b470e98-0d58-463d-a4d8-8e2adae1ed80
This commit is contained in:
wuehler 2019-02-24 17:53:04 +00:00
parent 31aaa6290b
commit 9596247eab
3 changed files with 70 additions and 117 deletions

View File

@ -1,5 +1,7 @@
# Add changes at the top of the list. Keep it in ASCII, and 80-char wide.
# Do not insert empty lines here, update check depends on it.
- changed: 74_Unifi: removed deprecated UnifiSwitch-functions!
- bugfix: 74_UnififSwitch: fixed poe restart
- feature: 57_Calendar.pm: new features and attributes
german commandref added
- bugfix: 00_SIGNALduino: Load protocolHash from web if file is

View File

@ -40,10 +40,12 @@
# - changed: 74_Unifi: Reading(name) for Utilization of APs with UC V 5.7/8.x
# V 3.0.5
# - feature: 74_Unifi: Added readings for wan_ip and results of speedtest
# V 3.1.0
# - changed: 74_Unifi: removed deprecated UnifiSwitch-functions!
package main;
my $version="3.0.4";
my $version="3.1.0";
use strict;
use warnings;
use HttpUtils;
@ -237,13 +239,13 @@ sub Unifi_Set($@) {
my $apNames = Unifi_ApNames($hash);
my $SSIDs = Unifi_SSIDs($hash);
if($setName !~ /archiveAlerts|restartAP|setLocateAP|unsetLocateAP|disconnectClient|update|updateClient|clear|poeMode|blockClient|unblockClient|enableWLAN|disableWLAN|switchSiteLEDs|createVoucher/) {
if($setName !~ /archiveAlerts|restartAP|setLocateAP|unsetLocateAP|disconnectClient|update|updateClient|clear|blockClient|unblockClient|enableWLAN|disableWLAN|switchSiteLEDs|createVoucher/) {
return "Unknown argument $setName, choose one of update:noArg "
."clear:all,readings,clientData,allData,voucherCache "
.((defined $hash->{alerts_unarchived}[0] && scalar @{$hash->{alerts_unarchived}}) ? "archiveAlerts:noArg " : "")
.(($apNames && Unifi_CONNECTED($hash)) ? "restartAP:all,$apNames setLocateAP:all,$apNames unsetLocateAP:all,$apNames " : "")
.(($clientNames && Unifi_CONNECTED($hash)) ? "disconnectClient:all,$clientNames " : "")
."poeMode createVoucher enableWLAN:$SSIDs disableWLAN:$SSIDs "
."createVoucher enableWLAN:$SSIDs disableWLAN:$SSIDs "
."blockClient:$clientNames unblockClient:$clientNames switchSiteLEDs:on,off updateClient";
}
else {
@ -328,73 +330,6 @@ sub Unifi_Set($@) {
return "enter mac of client" if( ! defined $setVal);
Unifi_UpdateClient_Send($hash,$setVal);
}
elsif ($setName eq 'poeMode') {
Log3 $name, 2, "$name: deprecated use of set poeMode. Use same feature in autocreated UnifiSwitch-Device.";
return "usage: $setName <name|mac|id> <port> <off|auto|passive|passthrough|restart>" if( !$setVal3 );
my $apRef;
for my $apID (keys %{$hash->{accespoints}}) {
my $ap = $hash->{accespoints}->{$apID};
next if( !$ap->{port_table} );
next if( $ap->{type} ne 'usw' );
next if( $setVal ne $ap->{mac} && $setVal ne $ap->{device_id} && $ap->{name} !~ $setVal );
return "multiple switches found for $setVal" if( $apRef );
$apRef = $ap;
}
return "no switch $setVal found" if( !$apRef );
if( $setVal2 !~ m/\d+/ ) {
for my $port (@{$apRef->{port_table}}) {
next if( $port->{name} !~ $setVal2 );
$setVal2 = $port->{port_idx};
last;
}
}
return "port musst be numeric" if( $setVal2 !~ m/\d+/ );
return "port musst be in [1..". scalar @{$apRef->{port_table}} ."] " if( $setVal2 < 1 || $setVal2 > scalar @{$apRef->{port_table}} );
return "switch '$apRef->{name}' has no port $setVal2" if( !defined(@{$apRef->{port_table}}[$setVal2-1] ) );
return "port $setVal2 of switch '$apRef->{name}' is not poe capable" if( !@{$apRef->{port_table}}[$setVal2-1]->{port_poe} );
my $port_overrides = $apRef->{port_overrides};
my $idx;
my $i = 0;
for my $entry (@{$port_overrides}) {
if( $entry->{port_idx} eq $setVal2 ) {
$idx = $i;
last;
}
++$i;
}
if( !defined($idx) ) {
push @{$port_overrides}, {port_idx => $setVal2+0};
$idx = scalar @{$port_overrides};
}
if( $setVal3 eq 'off' ) {
$port_overrides->[$idx]{poe_mode} = "off";
Unifi_RestJson_Send($hash, $apRef->{device_id}, {port_overrides => $port_overrides });
} elsif( $setVal3 eq 'auto' || $setVal3 eq 'poe+' ) {
#return "port $setVal2 not auto poe capable" if( @{$apRef->{port_table}}[$setVal2-1]->{poe_caps} & 0x03 ) ;
$port_overrides->[$idx]{poe_mode} = "auto";
Unifi_RestJson_Send($hash, $apRef->{device_id}, {port_overrides => $port_overrides });
} elsif( $setVal3 eq 'passive' ) {
#return "port $setVal2 not passive poe capable" if( @{$apRef->{port_table}}[$setVal2-1]->{poe_caps} & 0x04 ) ;
$port_overrides->[$idx]{poe_mode} = "pasv24";
Unifi_RestJson_Send($hash, $apRef->{device_id}, {port_overrides => $port_overrides });
} elsif( $setVal3 eq 'passthrough' ) {
#return "port $setVal2 not passthrough poe capable" if( @{$apRef->{port_table}}[$setVal2-1]->{poe_caps} & 0x08 ) ;
$port_overrides->[$idx]{poe_mode} = "passthrough";
Unifi_RestJson_Send($hash, $apRef->{device_id}, {port_overrides => $port_overrides });
} elsif( $setVal3 eq 'restart' ) {
Unifi_ApJson_Send($hash,{cmd => 'power-cycle', mac => $apRef->{mac}, port_idx => $setVal2+0});
} else {
return "unknwon poe mode $setVal3";
}
}
elsif ($setName eq 'archiveAlerts' && defined $hash->{alerts_unarchived}[0]) {
Unifi_ArchiveAlerts_Send($hash);
undef @{$hash->{alerts_unarchived}};
@ -516,37 +451,12 @@ sub Unifi_Get($@) {
my $clientNames = Unifi_ClientNames($hash);
if($getName !~ /events|clientData|unarchivedAlerts|poeState|voucherList|voucher|showAccount/) {
if($getName !~ /events|clientData|unarchivedAlerts|voucherList|voucher|showAccount/) {
return "Unknown argument $getName, choose one of "
.((defined $hash->{events}[0] && scalar @{$hash->{events}}) ? "events:noArg " : "")
.((defined $hash->{alerts_unarchived}[0] && scalar @{$hash->{alerts_unarchived}}) ? "unarchivedAlerts:noArg " : "")
.(($clientNames) ? "clientData:all,$clientNames " : "")
."poeState voucherList:all,$voucherNotes voucher:$voucherNotes showAccount";
}
elsif ($getName eq 'poeState') {
Log3 $name, 2, "$name: deprecated use of get poeState. Use readings in autocreated UnifiSwitch-Device instead.";
my $poeState;
for my $apID (keys %{$hash->{accespoints}}) {
my $apRef = $hash->{accespoints}->{$apID};
next if( $apRef->{type} ne 'usw' );
next if( !$apRef->{port_table} );
next if( $getVal && $getVal ne $apRef->{mac} && $getVal ne $apRef->{device_id} && $apRef->{name} !~ $getVal );
$poeState .= "\n" if( $poeState );
$poeState .= sprintf( "%-20s (mac:%-17s, id:%s)\n", $apRef->{name}, $apRef->{mac}, $apRef->{device_id} );
$poeState .= sprintf( " %2s %-15s", "id", "name" );
$poeState .= sprintf( " %s %s %-6s %-4s %-10s", "", "on", "mode", "", "class" );
$poeState .= "\n";
for my $port (@{$apRef->{port_table}}) {
#next if( !$port->{port_poe} );
$poeState .= sprintf( " %2i %-15s", $port->{port_idx}, $port->{name} );
$poeState .= sprintf( " %s %s %-6s %-4s %-10s", $port->{poe_caps}, $port->{poe_enable}, $port->{poe_mode}, defined($port->{poe_good})?($port->{poe_good}?"good":""):"", defined($port->{poe_class})?$port->{poe_class}:"" ) if( $port->{port_poe} );
$poeState .= sprintf( " %5.2fW %5.2fV %5.2fmA", $port->{poe_power}?$port->{poe_power}:0, $port->{poe_voltage}, $port->{poe_current}?$port->{poe_current}:0 ) if( $port->{port_poe} );
$poeState .= "\n";
}
}
$poeState = "====================================================\n". $poeState;
$poeState .= "====================================================\n";
return $poeState;
."voucherList:all,$voucherNotes voucher:$voucherNotes showAccount";
}
elsif ($getName eq 'unarchivedAlerts' && defined $hash->{alerts_unarchived}[0] && scalar @{$hash->{alerts_unarchived}}) {
my $alerts = "====================================================\n";
@ -734,12 +644,15 @@ sub Unifi_Attr(@) {
###############################################################################
sub Unifi_Write($$){
my ( $hash, $type, $ap_id, $port_overrides) = @_; #TODO: ap_id und port_overrides in @a, damit es für andere $type auch geht.
my ( $hash, $type, $ap_id, $data) = @_; #TODO: ap_id und port_overrides in @a, damit es für andere $type auch geht.
my ($name,$self) = ($hash->{NAME},Unifi_Whoami());
Log3 $name, 5, "$name ($self) - executed with ".$type;
if($type eq "Unifi_RestJson_Send"){
Unifi_RestJson_Send($hash, $ap_id, {port_overrides => $port_overrides });
Unifi_RestJson_Send($hash, $ap_id, {port_overrides => $data });
}
if($type eq "Unifi_ApJson_Send"){
Unifi_ApJson_Send($hash, $data );
}
return undef;
@ -1329,12 +1242,12 @@ sub Unifi_SetAccesspointReadings($) {
readingsBulkUpdate($hash,'-AP_'.$apName.'_utilizationNG',$apRef->{'ng_cu_total'}) if( defined($apRef->{'ng_cu_total'}) );
}
readingsBulkUpdate($hash,'-AP_'.$apName.'_locate',(!defined $apRef->{locating}) ? 'unknown' : ($apRef->{locating}) ? 'on' : 'off');
my $poe_power;
for my $port (@{$apRef->{port_table}}) {
next if( !$port->{port_poe} );
$poe_power += $port->{poe_power} if( defined($port->{poe_power}) );
}
readingsBulkUpdate($hash,'-AP_'.$apName.'_poePower', $poe_power) if( defined($poe_power) );
#my $poe_power;
#for my $port (@{$apRef->{port_table}}) {
# next if( !$port->{port_poe} );
# $poe_power += $port->{poe_power} if( defined($port->{poe_power}) );
#}
#readingsBulkUpdate($hash,'-AP_'.$apName.'_poePower', $poe_power) if( defined($poe_power) );
# readingsBulkUpdate($hash,'-AP_'.$apName.'_guests',$apRef->{'guest-num_sta'});
# readingsBulkUpdate($hash,'-AP_'.$apName.'_users',$apRef->{'user-num_sta'});
@ -2093,7 +2006,7 @@ sub Unifi_ReceiveFailure($$) {
my ($name,$self) = ($hash->{NAME},Unifi_Whowasi());
if (defined $meta->{msg}) {
if ($meta->{msg} eq 'api.err.LoginRequired') {
if (($meta->{msg} eq 'api.err.LoginRequired') || ($meta->{msg} =~ /Connection refused$/) || ($meta->{msg} =~ /SSL connect attempt failed$/)){
Log3 $name, 5, "$name ($self) - LoginRequired detected...";
if(Unifi_CONNECTED($hash)) {
Log3 $name, 5, "$name ($self) - I am the first who detected LoginRequired. Do re-login...";
@ -2294,9 +2207,6 @@ Or you can use the other readings or set and get features to control your unifi-
<li><code>set &lt;name&gt; unsetLocateAP &lt;all|_id|name|ip&gt;</code><br>
Stop 'locate' on one or all accesspoints. </li>
<br>
<li><code>set &lt;name&gt; poeMode &lt;name|mac|id&gt; &lt;port&gt; &lt;off|auto|passive|passthrough|restart&gt;</code><br>
Set PoE mode for &lt;port&gt;. </li>
<br>
<li><code>set &lt;name&gt; blockClient &lt;clientname&gt;</code><br>
Block the &lt;clientname&gt;. Can also be called with the mac-address of the client.</li>
<br>
@ -2331,9 +2241,6 @@ Or you can use the other readings or set and get features to control your unifi-
<li><code>get &lt;name&gt; unarchivedAlerts</code><br>
Show all unarchived Alerts.</li>
<br>
<li><code>get &lt;name&gt; poeState [name|mac|id]</code><br>
Show port PoE state.</li>
<br>
<li><code>get &lt;name&gt; voucher [note]</code><br>
Show next voucher-code with specified note. If &lt;note&gt; is used in voucherCache the voucher will be marked as delivered</li>
<br>

View File

@ -19,12 +19,14 @@
# added new state-mappings
# V 0.92
# - fixed: 74_UnififSwitch: fixed possible log-error in eq in line 135
# V 0.93
# - bugfix: 74_UnififSwitch: fixed poe restart
#
# TODOs:
# - state des USW für weiter state-Numbers korrekt in Worte übersetzen
package main;
my $version="0.92";
my $version="0.93";
# Laden evtl. abhängiger Perl- bzw. FHEM-Module
use strict;
use warnings;
@ -109,7 +111,7 @@ sub UnifiSwitch_Set($@){
#}
if($setName !~ /clear|poeMode/) {
return "Unknown argument $setName, choose one of "
."clear:readings poeMode "; #TODO: PortNamen sowie die Modes als Auswahl anhängen
."clear:all,readings poeMode "; #TODO: PortNamen sowie die Modes als Auswahl anhängen
} else {
Log3 $name, 4, "$name: set $setName";
if ($setName eq 'poeMode') {
@ -165,7 +167,7 @@ sub UnifiSwitch_Set($@){
IOWrite($hash, "Unifi_RestJson_Send", $apRef->{device_id}, $port_overrides);
} elsif( $setVal2 eq 'restart' ) {#TODO: Was wir hier gemacht? Funktioniert das noch?
IOWrite($hash, "Unifi_RestJson_Send", {cmd => 'power-cycle', mac => $apRef->{mac}, port_idx => $setVal+0});
IOWrite($hash, "Unifi_ApJson_Send", $apRef->{device_id}, {cmd => 'power-cycle', mac => $apRef->{mac}, port_idx => $setVal});
} else {
return "unknwon poe mode $setVal2";
@ -183,8 +185,42 @@ sub UnifiSwitch_Set($@){
sub UnifiSwitch_Get($@){
my @a = @_;
my ($hash,@a) = @_;
return "\"get $hash->{NAME}\" needs at least one argument" if ( @a < 2 );
my ($name,$getName,$getVal) = @a;
if (defined $getVal){
Log3 $name, 5, "$name: get called with $getName $getVal." ;
}else{
Log3 $name, 5, "$name: get called with $getName.";
}
if($getName !~ /poeState/) {
return "Unknown argument $getName, choose one of poeState";
}
elsif ($getName eq 'poeState') {
my $poeState;
my $apRef = $hash->{usw};
next if( $apRef->{type} ne 'usw' );
next if( !$apRef->{port_table} );
next if( $getVal && $getVal ne $apRef->{mac} && $getVal ne $apRef->{device_id} && $apRef->{name} !~ $getVal );
$poeState .= "\n" if( $poeState );
$poeState .= sprintf( "%-20s (mac:%-17s, id:%s)\n", $apRef->{name}, $apRef->{mac}, $apRef->{device_id} );
$poeState .= sprintf( " %2s %-15s", "id", "name" );
$poeState .= sprintf( " %s %s %-6s %-4s %-10s", "", "on", "mode", "", "class" );
$poeState .= "\n";
for my $port (@{$apRef->{port_table}}) {
#next if( !$port->{port_poe} );
$poeState .= sprintf( " %2i %-15s", $port->{port_idx}, $port->{name} );
$poeState .= sprintf( " %s %s %-6s %-4s %-10s", $port->{poe_caps}, $port->{poe_enable}, $port->{poe_mode}, defined($port->{poe_good})?($port->{poe_good}?"good":""):"", defined($port->{poe_class})?$port->{poe_class}:"" ) if( $port->{port_poe} );
$poeState .= sprintf( " %5.2fW %5.2fV %5.2fmA", $port->{poe_power}?$port->{poe_power}:0, $port->{poe_voltage}, $port->{poe_current}?$port->{poe_current}:0 ) if( $port->{port_poe} );
$poeState .= "\n";
}
$poeState = "====================================================\n". $poeState;
$poeState .= "====================================================\n";
return $poeState;
}
return undef;
}
@ -310,6 +346,14 @@ You can use the readings or set features to control your unifi-switch.
Set PoE mode for &lt;port&gt;. </li>
</ul>
<h4>Get</h4>
<ul>
<li><code>get &lt;name&gt; poeState</code><br>
Show more details about the ports of the switch.</li>
<br>
</ul>
<h4>Readings</h4>
<ul>
Note: All readings generate events. You can control this with <a href="#readingFnAttributes">these global attributes</a>.