- CPAN Bug# 23587: _pre_submit() would return success if fraud detection is
  used but not found. Fix: croak() if unable to load the fraud detection class.

bump version to 3.00_05, fix, fix inclusion of B:FS:preCharge in "make install" and "make dist", add back in preCharge tests, add all Phil's recent work to the changelog

0.05?

- _risk_detect(): explicity set return value
- _pre_submit(): explicity set return value
- BUG: _pre_submit() returns success if fraud detection is used but not found!
  this behavior is unchanged but this is probably a bug / bad behavior (Ivan?)

- Used feedback from MSCHWERN / Bug #22074 to cleanup new/_pre_submit more
  Definitely agree having modules use SUPER would be cleaner/safer/better.
  Ivan, we should consider this idea very seriously.
- Now always wrap submit() method with _pre_submit() (but only once)
- no longer populate _child_submit, code in anon sub was cleaned up
- use return values from _pre_submit to determine if real submit is called
  the return values from _pre_submit should be reviewed/verified still

- minor cleanup of test drivers
- fix "submit unchanged" test case: now wrapped with _pre_submit() always

- removed fraud_detect w/preCharge test, dist doesn't contain preCharge yet

* Add Class data %WrappedSubmitClassMethod to remember "wrapped" submits
* Fix new() to check %WrappedSubmitClassMethod to avoid creating deep recursion
* Minor documentation change in print statements use ',' instead of '.'

* 8 new tests and 1 new "MOCK3" driver for testing _pre_submit() functionality
* test for new() replacing subclass submit and causing deep recursion

3.00_04 for real

add missing files to MANIFEST, move failure_statuses to top of the changelog as its the most important

DOC: s/exp_date/expiration/; #closes CPAN#21082

3.00_04

removed TODO since it doesn't exist in CVS

- remove extra block in code that redefines submit for subclass

- Add a second test driver (this one has a submit method)
- update submit() test cases

Doh!  Forgot to fix the test count in bop.t

Changed the logic in the symbol-table magic to punt if the processor
class does not have a submit() method.

This bug *only* affected testing -- properly written processors that
declared a submit() method would not have experienced the
deep-recursion failure.

bop.t reverted to continue testing with the MOCK1 processor.

I see opportunities for more tests....

Changes to code formatting, POD and other non-functional changes.

- Lots of general cleanup, no functional changes. "fill paragraphs" in
  POD, remove extra whitespace, sorted %fields, dump_contents now
  sorts %contents, added myself to AUTHORS.

- test that required_fields() returns 0 if no required fields are missing

More changes per discussions on bop-devel

- Removed some old commented out 2.x statements
- set $VERSION using eval per modperlstyle
- new() now will strip off multiple leading dashes before calling build_subs
  NOTE: we should probably have it remove any \W to make perl happy
- required_fields() now croaks with a list of missing fields

- created test cases for (all?) methods except _risk_detect _pre_submit

adding Phil's changes to Changes

- content() now returns empty hash if no content is defined
- build_subs now uses can() so it doesn't redefine subs (and generate warnings)
- DOC: SYNOPSIS now uses correct "card_number" and not "cardnumber"

ch ch ch chaaanges

don't want to necessarily depend on Data::Dumper...

Fixup parameter passing, finish changing names & doco to match

added POD

New module for preCharge

New modules for preCharge

tenatively adding inactive status

add failure_status field and documentation

need post_https from upstream now (yay!)

eliminate warnings about redefined subroutes

HTTPS: require Net::SSLeay 1.30 and remove _my_https_post kludge

oops, forgot _03 changelog in _03

new raw post

quick bug fix for OpenECHO

3.00_02!

removing

first changes for 3.00_01 test release

Import of Business::OnlinePayment 2.01