Incorporated all changes that were made in SBA V1.5.3 for ZC 1.5.1.

This did include revision of determining previously ordered products that
have attributes and are tracked by SBA.
Wanted to get some of the testing started.  Still have some delete variant(s) based on other actions to do in attributes_controller; however, need to know if have missed anything in the rest of the code.
This too probably should be a break instead of a continue... When reviewing the code and providing improvements, I chose the wrong action to get "out" of the loop.  Continue will initiate the next iteration, break will get out of it.  The code surrounding it should be reviewed before committing that new revision.  Primarily important/an issue when dealing with a loop that increments to the next database result through ->MoveNext().