X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=FS%2FFS%2FUI%2FWeb%2Fsmall_custview.pm;h=1e9ddb172306806ed06ebb02af1ec419745190c6;hb=622c72770c971ae44d37dfb59a0f25972051a25a;hp=e4b5421a2c04df5dc657606c9a1826e17c2b4635;hpb=f06a0610477b0ba8e1931722c3105b880fbc35c3;p=freeside.git diff --git a/FS/FS/UI/Web/small_custview.pm b/FS/FS/UI/Web/small_custview.pm index e4b5421a2..1e9ddb172 100644 --- a/FS/FS/UI/Web/small_custview.pm +++ b/FS/FS/UI/Web/small_custview.pm @@ -11,32 +11,97 @@ use FS::cust_main; @ISA = qw(Exporter); @EXPORT_OK = qw( small_custview ); +=head1 NAME + +FS::UI::Web::small_custview + +=head1 SYNOPSIS + + use FS::UI::Web::small_custview qw( small_custview ); + + #new-style + $html = small_custview( + { 'cust_main' => $cust_main, #or 'custnum' => $custnum, + 'countrydefault' => 'US', + 'nobalance' => 1, + 'url' => 'http://freeside.machine/freeside/view/cust_main.cgi', + 'nopkg' => 1, + } + ); + + #old-style (deprecated) + $html = small_custview( $cust_main, $countrydefault, $nobalance, $url ); + +=head1 DESCRIPTION + +A subroutine for displaying customer information. + +=head1 SUBROUTINES + +=over 4 + +=item small_custview HASHREF + +New-style interface. Keys are: + +=over 4 + +=item cust_main + +Customer (as a FS::cust_main object) + +=item custnum + +Customer number (if cust_main is not provided). + +=item countrydefault + +=item nobalance + +=item url + +=back + =item small_custview CUSTNUM || CUST_MAIN_OBJECT, COUNTRYDEFAULT, NOBALANCE_FLAG, URL -Sheesh. I did switch to mason, but this is still hanging around. Figure out -some better way to sling mason components to self-service & RT. +Old-style (deprecated) interface. =cut sub small_custview { + my( $cust_main, $countrydefault, $nobalance, $url, $nopkg ); + if ( ref($_[0]) eq 'HASH' ) { + my $opt = shift; + $cust_main = $opt->{cust_main} + || qsearchs('cust_main', { 'custnum' => $opt->{custnum} } ); + $countrydefault = $opt->{countrydefault} || 'US'; + $nobalance = $opt->{nobalance}; + $url = $opt->{url}; + $nopkg = $opt->{nopkg}; + } else { + my $arg = shift; + $countrydefault = shift || 'US'; + $nobalance = shift; + $url = shift; + $nopkg = 0; + + $cust_main = ref($arg) ? $arg + : qsearchs('cust_main', { 'custnum' => $arg } ) + or die "unknown custnum $arg"; + } - my $arg = shift; - my $countrydefault = shift || 'US'; - my $nobalance = shift; - my $url = shift; - - my $cust_main = ref($arg) ? $arg - : qsearchs('cust_main', { 'custnum' => $arg } ) - or die "unknown custnum $arg"; - - my $html = '
'; + my $html = '
'; - $html = qq!View ' + $html = qq!' if $url; - $html .= 'Customer #'. $cust_main->display_custnum. ''. - ' - '. - ucfirst($cust_main->status). ''; + $html .= 'Customer #'. $cust_main->display_custnum. + ': '. encode_entities($cust_main->name). ''; + ' - '. + ucfirst($cust_main->status). ''; + + $html .= ' (Balance: $'. $cust_main->balance. ')' + unless $nobalance; my @part_tag = $cust_main->part_tag; if ( @part_tag ) { @@ -57,14 +122,15 @@ sub small_custview { $html .= ntable('#e8e8e8'). ''. ntable("#cccccc",2). - 'Billing
Address'. - encode_entities($cust_main->getfield('last')). ', '. - encode_entities($cust_main->first). '
'; + 'Billing
Address'; - $html .= encode_entities($cust_main->company). '
' if $cust_main->company; $html .= encode_entities($cust_main->address1). '
'; - $html .= encode_entities($cust_main->address2). '
' if $cust_main->address2; - $html .= encode_entities($cust_main->city). ', '. $cust_main->state. ' '. $cust_main->zip. '
'; + $html .= encode_entities($cust_main->address2). '
' + if $cust_main->address2; + $html .= encode_entities($cust_main->city) . ', ' if $cust_main->city; + $html .= $cust_main->state. ' '. + $cust_main->zip. '
'; + $cust_main->zip. '
'; $html .= $cust_main->country. '
' if $cust_main->country && $cust_main->country ne $countrydefault; @@ -89,11 +155,10 @@ sub small_custview { 'Service
Address'; $html .= join('
', map encode_entities($_), grep $_, - $cust_main->contact, - $cust_main->company, + $cust_main->ship_company, $ship->address1, $ship->address2, - ($ship->city . ', ' . $ship->state . ' ' . $ship->zip), + (($ship->city ? $ship->city . ', ' : '') . $ship->state . ' ' . $ship->zip), ($ship->country eq $countrydefault ? '' : $ship->country ), ); @@ -101,10 +166,25 @@ sub small_custview { $html .= ''; - $html .= ''; + $html .= ''; - $html .= '
Balance: $'. $cust_main->balance. '
' - unless $nobalance; + #would be better to use ncancelled_active_pkgs, but that doesn't have an + # optimization to just count them yet, so it would be a perf problem on + # tons-of-package customers + if ( !$nopkg && scalar($cust_main->ncancelled_pkgs) < 20 ) { + + foreach my $cust_pkg ( $cust_main->ncancelled_active_pkgs ) { + + $html .= ''. + ''. + ucfirst($cust_pkg->status). ' - '. + encode_entities($cust_pkg->part_pkg->pkg_comment_only(nopkgpart=>1)). + ''; + } + + } + + $html .= ''; # last payment might be good here too? @@ -126,5 +206,22 @@ sub ntable { } +=back + +=head1 BUGS + +Sheesh. I did switch to mason, but this is still hanging around. Figure out +some better way to sling mason components to self-service & RT. + +(Or, is it useful to have this without depending on the regular back-office UI +and Mason stuff to be in place? So we have something suitable for displaying +customer information in other external systems, not just RT?) + +=head1 SEE ALSO + +L + +=cut + 1;