While at it... I forgot to mention, that the CREATE_INVOICE action should also
contain a test at the very beginning,
to see if SHOPPER_MUST_REGISTER ! As if the customer, adds items into the
basket, and then clicks on "Checkout" the CREATE_INVOICE creates dummy
invoices with UserID=0 . No real harm there, because that is never set to
"Active" and the user once he registers or logs in, will reselect his product.
It just uses an invoice number, and then you have non-ascending invoice
numbers. Its kinda scary to see "missing"
invoices. (Well not really missing, just incomplete). but in any case we
should stop them from coming about
Again my .02$ worth.
PS: I don't have CVS write access. and I think this is within your "frame of
design" for FreeTrade. That's why I'm mentioning it. If you want the sources,
I'd be glad to sent you those 3 files. Thou, do a "diff" prior, cuz I did
quite of few changes. (Like currancy tests, skipping CC info, if the invoice
is $0.00 as I have some software that is free! I'm setting up a software
store. Not a widget shipping store. So I have lotsa file-access control and
download modules added) ...
Joseph Renda wrote:
> Perhaps my 2 cents worth...
>
> I noticed some incorrect behavior, in FreeTrade 1.0.3, it has to do with the
> global setting of
> "SHOPPER_MUST_REGISTER" that's defined in the global section.
>
> The only time that this is check (this definition) is in the
> "modules/content" file, thus when looking at your basket.
>
> If I hit, "Checkout" or "Express Checkout" I can skip this "requirement" I
> set.
>
> Meaning, I wish for all my customers to register & create a login name and
> password, so they can track their, orders and history.
>
> Thus, I added a fast test in the "order_info" and "order_express" screens...
> mainly like so...
>
> if ( (SHOPPER_MUST_REGISTER) AND !(isset($UserInfo["ID"] &&
> $UserInfo["ID"]>0)
> {
> //complain, and show them the 'register' & login links...
> }else {
>
> // Get item from invoice glob.
>
> }
>
> I 'elsed' the first glob of code, because from that point on, you use the
> $continueScript var to control if the code should continue or not.
>
> ------------------------------------------------------------
> To subscribe: [EMAIL PROTECTED]
> To unsubscribe: [EMAIL PROTECTED]
> Site: http://www.working-dogs.com/freetrade/
> Problems?: [EMAIL PROTECTED]
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Site: http://www.working-dogs.com/freetrade/
Problems?: [EMAIL PROTECTED]
