Bug when changing Buy It Now

ktmtalk · October 27th, 2009, 07:38 PM

A person lists an ad, and originally chooses NO on the Buy It Now option. They see their error and try to edit the ad to change it to YES.

They are allowed to change it to YES but, there is no prompt for the Paypal address. When the edit is saved, the Buy It Now button does not appear (probably because they've never entered a PayPal address).

Bug??

Chuck S · October 27th, 2009, 09:25 PM

well maybe not a bug. Maybe more so an omission as it was never considered to be an edit option. A user on upload is given a choice to enter a paypal email but I can look at what it would take to add it to the edit screen.

ktmtalk · October 27th, 2009, 09:29 PM

Quote:

Originally Posted by Chuck S

well maybe not a bug. Maybe more so an omission as it was never considered to be an edit option. A user on upload is given a choice to enter a paypal email but I can look at what it would take to add it to the edit screen.

I know you're busy... lemme take a qwik look and see if it's something simple - I'll get back to you.

Chuck S · October 27th, 2009, 09:32 PM

Yeah it probally is. adding a paypal email is a users table thing hense when I coded this feature it made sense to do it on the upload screen. Now since editing an ad only dealt with the edit of the actual ad content it as stated was never considered an option to update a paypal email but I dont think it would be hard. Just might have to be something I look at when I get a few.

Chuck S · October 27th, 2009, 10:00 PM

Try redownloading the build and uploading the editproduct php script and templates.

ktmtalk · October 27th, 2009, 10:15 PM

Quote:

Originally Posted by Chuck S

Try redownloading the build and uploading the editproduct php script and templates.

gad you're fast... I was just getting back to tell you I was done... I'll grab your build.

be back in a sec....

PERFECT.... THANK YOU!!

Now I'm off to do some searching on how to make the PROFILE pull up the profile from the invision tables.

.
.

Chuck S · October 27th, 2009, 10:30 PM

LOL it was nothing to do. Later at night I can churn out all kinds of things when things slow down.

ktmtalk · October 27th, 2009, 10:57 PM

Quote:

Originally Posted by Chuck S

LOL it was nothing to do. Later at night I can churn out all kinds of things when things slow down.

thank you for taking the time... you did good... naw.. you did GREAT!

ktmtalk · October 27th, 2009, 11:07 PM

ooops.... not quite done.... need to add code to actually save the new paypal if it was entered.

Chuck S · October 28th, 2009, 07:39 AM

Thats in there my friend from what I see. You say that doesnt work?

Code:

Content visible to verified customers only.

ktmtalk · October 28th, 2009, 12:17 PM

Quote:

Originally Posted by Chuck S

Thats in there my friend from what I see. You say that doesnt work?

Code:

Content visible to verified customers only.

Correct... not working. Here is the culprit:

Code:

Content visible to verified customers only.

$theuser is not being set - it's blank.

.

Chuck S · October 28th, 2009, 12:29 PM

Try changing that instance of $theuser to $userid

Chuck S · October 28th, 2009, 12:33 PM

Rather just download and upload editproduct.php

ktmtalk · October 28th, 2009, 01:15 PM

Quote:

Originally Posted by Chuck S

Try changing that instance of $theuser to $userid

still no workie... userid is also blank

Code:

Content visible to verified customers only.

i did not download a new build, I just edited editproduct.php and changed that instance of $theuser to $userid

Chuck S · October 28th, 2009, 01:27 PM

well it works for me and I am using the build I uploaded and its confirmed tested

ktmtalk · October 28th, 2009, 01:50 PM

Quote:

Originally Posted by Chuck S

well it works for me and I am using the build I uploaded and its confirmed tested

not for me.

Inside editproduct.php, there are only two instances of $userid

one is here:

Code:

Content visible to verified customers only.

and the other is the new buyit routine

Code:

Content visible to verified customers only.

or should I just go take a warm bath and open up one of my veins ?

try this in an existing installation... find a seller, any seller that has a product for sale. Use phmyadmin and clear out the paypal for that seller. Now use admin to edit one of the sellers ads... stick in a fake paypal address.. save the edit... then see if the "buy now" shows up.

Chuck S · October 28th, 2009, 01:57 PM

why not simply redownload the build and upload the editproduct.php that I know I have tested and is working.

You noted you had not downloaded the build

ktmtalk · October 28th, 2009, 02:03 PM

Quote:

Originally Posted by Chuck S

why not simply redownload the build and upload the editproduct.php that I know I have tested and is working.

You noted you had not downloaded the build

my first post said that yes.... but I then redownloaded the new build and that's where I checked editproduct for all instances of $userid. One will be in the "if auction" and the only other one that I see is in the buyit routine.

my current editproduct.php, dated 10/28/09, 43,757 bytes

please do a check of editproduct.php and see if you find more than the two instances of $userid.

sorry for the hassle.

.

Chuck S · October 28th, 2009, 02:10 PM

userid is passed in the actual form that is sent when you hit submit to do changes.

Then there is this in the typecast function that grabs the userid

Code:

Content visible to verified customers only.

I have tested the code and it works fine I can most definately change a users paypal email. Userid is not empty.

So unless we are talking something else you should be able to change a users paypal email

Chuck S · October 28th, 2009, 02:14 PM

test highlight - PhotoPost Classifieds

See for yourself login with test and password testing