X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2Fsvc_dsl.pm;h=dcd6d1dbecbf9e89d57911eb3251c8eefec1ac93;hp=89827e8d75476e2a515516ab703bc567b7f9cda3;hb=674cb2d9d7105f4cc2871539b2e9f7088cdaa750;hpb=878e77215666e79591eff1ac67eb119994a08d05 diff --git a/FS/FS/svc_dsl.pm b/FS/FS/svc_dsl.pm index 89827e8d7..dcd6d1dbe 100644 --- a/FS/FS/svc_dsl.pm +++ b/FS/FS/svc_dsl.pm @@ -1,13 +1,15 @@ package FS::svc_dsl; +use base qw(FS::Password_Mixin + FS::svc_Common); use strict; -use vars qw( @ISA $conf $DEBUG $me ); -use FS::Record qw( qsearch qsearchs ); +use vars qw( $conf $DEBUG $me ); +use FS::UID; +use FS::Record qw( qsearch qsearchs dbh ); use FS::svc_Common; use FS::dsl_note; use FS::qual; -@ISA = qw( FS::svc_Common ); $DEBUG = 0; $me = '[FS::svc_dsl]'; @@ -75,6 +77,8 @@ Vendor/telco DSL order status (e.g. (N)ew, (A)ssigned, (R)ejected, (M)revised, =item phonenum - DSL Telephone Number +=item gateway_access_number - Gateway access number, if different + =item loop_type - Loop-type - vendor/telco-specific =item local_voice_provider - Local Voice Provider's name @@ -128,54 +132,56 @@ sub table_info { my %dis2 = ( disable_inventory=>1, disable_select=>1 ); { - 'name' => 'DSL', - 'sorts' => [ 'phonenum' ], - 'display_weight' => 55, - 'cancel_weight' => 75, - 'fields' => { - 'pushed' => { label => 'Pushed', - type => 'disabled' }, - 'desired_due_date' => { label => 'Desired Due Date', %dis2, }, - 'due_date' => { label => 'Due Date', %dis2, }, - 'vendor_order_id' => { label => 'Vendor Order Id', %dis2, }, - 'vendor_qual_id' => { label => 'Vendor Qualification Id', - type => 'disabled' }, - 'vendor_order_type' => { label => 'Vendor Order Type', - disable_inventory => 1, - }, - 'vendor_order_status' => { label => 'Vendor Order Status', - disable_inventory => 1, - }, - 'first' => { label => 'First Name', %dis2, }, - 'last' => { label => 'Last Name', %dis2, }, - 'company' => { label => 'Company Name', %dis2, }, - 'phonenum' => { label => 'Service Telephone Number', }, - 'loop_type' => { label => 'Loop Type', - disable_inventory => 1, - }, - 'local_voice_provider' => { label => 'Local Voice Provider', - disable_inventory => 1, - }, - 'circuitnum' => { label => 'Circuit #', }, - 'rate_band' => { label => 'Rate Band', - disable_inventory => 1, - }, - 'vpi' => { label => 'VPI', disable_inventory => 1 }, - 'vci' => { label => 'VCI', disable_inventory => 1 }, - 'isp_chg' => { label => 'ISP Changing?', - type => 'checkbox', %dis2 }, - 'isp_prev' => { label => 'Current or Previous ISP', - disable_inventory => 1, - }, - 'username' => { label => 'PPPoE Username', - type => 'text', - }, - 'password' => { label => 'PPPoE Password', %dis2 }, - 'staticips' => { label => 'Static IPs', %dis1 }, - 'monitored' => { label => 'Monitored', - type => 'checkbox', %dis2 }, - 'last_pull' => { label => 'Last Pull', type => 'disabled' }, - }, + 'name' => 'DSL', + 'name_plural' => 'DSLs', + 'lcname_plural' => 'DSLs', + 'sorts' => [ 'phonenum' ], + 'display_weight' => 55, + 'cancel_weight' => 75, + 'ip_field' => 'staticips', + 'fields' => { + 'pushed' => { label => 'Pushed', + type => 'disabled' }, + 'desired_due_date' => { label => 'Desired Due Date', %dis2, }, + 'due_date' => { label => 'Due Date', %dis2, }, + 'vendor_order_id' => { label => 'Vendor Order ID', %dis2, }, + 'vendor_qual_id' => { label => 'Vendor Qualification ID', + type => 'disabled' }, + 'vendor_order_type' => { label => 'Vendor Order Type', + disable_inventory => 1, }, + 'vendor_order_status' => { label => 'Vendor Order Status', + disable_inventory => 1, }, + 'first' => { label => 'First Name', %dis2, }, + 'last' => { label => 'Last Name', %dis2, }, + 'company' => { label => 'Company Name', %dis2, }, + 'phonenum' => { label => 'Service Telephone Number', }, + 'gateway_access_number' => { label => 'Gateway Access Number'. + ' (if different)', }, + 'loop_type' => { label => 'Loop Type', + disable_inventory => 1, }, + 'local_voice_provider' => { label => 'Local Voice Provider', + disable_inventory => 1, }, + 'circuitnum' => { label => 'Circuit #', }, + 'rate_band' => { label => 'Rate Band', + disable_inventory => 1, }, + 'vpi' => { label => 'VPI', + disable_inventory => 1 }, + 'vci' => { label => 'VCI', + disable_inventory => 1 }, + 'isp_chg' => { label => 'ISP Changing?', + type => 'checkbox', + %dis2, }, + 'isp_prev' => { label => 'Current or Previous ISP', + disable_inventory => 1, }, + 'username' => { label => 'PPPoE Username', + type => 'text', }, + 'password' => { label => 'PPPoE Password', %dis2 }, + 'staticips' => { label => 'Static IPs', %dis1 }, + 'monitored' => { label => 'Monitored', + type => 'checkbox', %dis2 }, + 'last_pull' => { label => 'Last Pull', + type => 'disabled' }, + }, }; } @@ -206,7 +212,25 @@ otherwise returns false. =cut -# the insert method can be inherited from FS::Record +sub insert { + my $self = shift; + my $dbh = dbh; + my $oldAutoCommit = $FS::UID::AutoCommit; + local $FS::UID::AutoCommit = 0; + + my $error = $self->SUPER::insert(@_); + if ( length($self->password) ) { + $error ||= $self->insert_password_history; + } + + if ( $error ) { + $dbh->rollback if $oldAutoCommit; + return $error; + } + + $dbh->commit if $oldAutoCommit; + ''; +} =item delete @@ -223,6 +247,27 @@ returns the error, otherwise returns false. =cut +sub replace { + my $new = shift; + my $old = shift || $new->replace_old; + my $dbh = dbh; + my $oldAutoCommit = $FS::UID::AutoCommit; + local $FS::UID::AutoCommit = 0; + + my $error = $new->SUPER::replace($old, @_); + if ( $old->password ne $new->password ) { + $error ||= $new->insert_password_history; + } + + if ( $error ) { + $dbh->rollback if $oldAutoCommit; + return $error; + } + + $dbh->commit if $oldAutoCommit; + ''; +} + # the replace method can be inherited from FS::Record =item check @@ -248,10 +293,11 @@ sub check { || $self->ut_textn('vendor_qual_id') || $self->ut_alphan('vendor_order_type') || $self->ut_alphan('vendor_order_status') - || $self->ut_text('first') - || $self->ut_text('last') + || $self->ut_textn('first') + || $self->ut_textn('last') || $self->ut_textn('company') || $self->ut_numbern('phonenum') + || $self->ut_numbern('gateway_access_number') || $self->ut_alphasn('loop_type') || $self->ut_textn('local_voice_provider') || $self->ut_textn('circuitnum') @@ -268,9 +314,28 @@ sub check { ; return $error if $error; + $self->gateway_access_number('') + if $self->phonenum && $self->phonenum eq $self->gateway_access_number; + $self->SUPER::check; } +=item gateway_access_or_phonenum + +=cut + +sub gateway_access_or_phonenum { + my $self = shift; + $self->gateway_access_number || $self->phonenum; +} + +=item dsl_device + +Returns the MAC addresses associated with this DSL service, as FS::dsl_device +objects. + +=cut + sub predelete_hook_first { my $self = shift; my @exports = $self->part_svc->part_export_dsl_pull; @@ -292,6 +357,15 @@ sub predelete_hook { ''; } +# password_history compatibility + +sub _password { + my $self = shift; + $self->get('password'); +} + +sub _password_encoding { 'plain'; } + =back =head1 SEE ALSO