X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2Fcust_bill.pm;h=7ea586a90dc70cca232a057912794b6f1b84d795;hp=b694924f3ade2860ae482f7580c5cacb0e73d995;hb=164e0da4c66b354bb35bf0af54a1f28555f0bf2a;hpb=49c58715c06a5150805fd3019a72dc1bc6361514 diff --git a/FS/FS/cust_bill.pm b/FS/FS/cust_bill.pm index b694924f3..7ea586a90 100644 --- a/FS/FS/cust_bill.pm +++ b/FS/FS/cust_bill.pm @@ -253,7 +253,7 @@ sub void { } } - $error = $self->delete; + $error = $self->_delete; if ( $error ) { $dbh->rollback if $oldAutoCommit; return $error; @@ -265,19 +265,22 @@ sub void { } -=item delete - -DO NOT USE THIS METHOD. Instead, apply a credit against the invoice, or use -the B method. - -This is only for internal use by V, which is what you should be using. - -DO NOT USE THIS METHOD. Whatever reason you think you have is almost certainly -wrong. Use B, that's what it is for. Really. This means you. - -=cut - -sub delete { +# removed docs entirely and renamed method to _delete to further indicate it is +# internal-only and discourage use +# +# =item delete +# +# DO NOT USE THIS METHOD. Instead, apply a credit against the invoice, or use +# the B method. +# +# This is only for internal use by V, which is what you should be using. +# +# DO NOT USE THIS METHOD. Whatever reason you think you have is almost certainly +# wrong. Use B, that's what it is for. Really. This means you. +# +# =cut + +sub _delete { my $self = shift; return "Can't delete closed invoice" if $self->closed =~ /^Y/i;