'note', 'text', 'NULL', '', '', '',
],
'primary_key' => 'svcnum',
- 'unique' => [],
+ 'unique' => [ ['typenum', 'serial'], ['ip_addr'], ['hw_addr'] ],
'index' => [],
'foreign_keys' => [
{ columns => [ 'svcnum' ],
'mac_addr', 'varchar', 'NULL', 12, '', '',
],
'primary_key' => 'svcnum',
- 'unique' => [ ['serialnum'] , ['mac_addr'] ],
+ 'unique' => [ ['modelnum', 'serialnum'] , ['mac_addr'] ],
'index' => [],
'foreign_keys' => [
{ columns => [ 'svcnum' ],
if ( $self->serialnum ) {
my @dup_serial;
- @dup_serial = $self->find_duplicates('global', 'serialnum');
+ @dup_serial = $self->find_duplicates('global', 'modelnum', 'serialnum');
if ( @dup_serial ) {
return "Serial number in use (svcnum ".$dup_serial[0]->svcnum.")";
}
$self->SUPER::check;
}
+#false laziness w/svc_cable
+sub _check_duplicate {
+ my $self = shift;
+
+ # Not reliable checks because the table isn't locked, but that's why we have
+ # unique indices. These are just to give friendlier error messages.
+
+ if ( $self->hw_addr ) {
+ my @dup_mac;
+ @dup_mac = $self->find_duplicates('global', 'hw_addr');
+ if ( @dup_mac ) {
+ return "MAC address in use (svcnum ".$dup_mac[0]->svcnum.")";
+ }
+ }
+
+ if ( $self->ip_addr ) {
+ my @dup_ip;
+ @dup_ip = $self->find_duplicates('global', 'ip_addr');
+ if ( @dup_ip ) {
+ return "IP address in use (svcnum ".$dup_ip[0]->svcnum.")";
+ }
+ }
+
+ if ( $self->serialnum ) {
+ my @dup_serial;
+ @dup_serial = $self->find_duplicates('global', 'typenum', 'serialnum');
+ if ( @dup_serial ) {
+ return "Serial number in use (svcnum ".$dup_serial[0]->svcnum.")";
+ }
+ }
+
+ '';
+}
+
=item hardware_type
Returns the L<FS::hardware_type> object associated with this installation.