diff options
author | Ivan Kohler <ivan@freeside.biz> | 2015-07-21 15:25:01 -0700 |
---|---|---|
committer | Ivan Kohler <ivan@freeside.biz> | 2015-07-21 15:25:01 -0700 |
commit | 164e0da4c66b354bb35bf0af54a1f28555f0bf2a (patch) | |
tree | fcd043850b067001953e26600c75ccb6de70a6f1 /FS | |
parent | 49c58715c06a5150805fd3019a72dc1bc6361514 (diff) |
remove external cust_bill::delete method, RT#37157
Diffstat (limited to 'FS')
-rw-r--r-- | FS/FS/cust_bill.pm | 31 |
1 files changed, 17 insertions, 14 deletions
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<void> method. - -This is only for internal use by V<void>, which is what you should be using. - -DO NOT USE THIS METHOD. Whatever reason you think you have is almost certainly -wrong. Use B<void>, 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<void> method. +# +# This is only for internal use by V<void>, which is what you should be using. +# +# DO NOT USE THIS METHOD. Whatever reason you think you have is almost certainly +# wrong. Use B<void>, 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; |