Post by ipthena » Thu Sep 27, 2018 7:14 am

OC 3.0.2.0
When a new customer chooses to register during the checkout process, they enter in their payment address and click continue. By default the "My delivery and billing addresses are the same." is checked, BUT it still asks for shipping address. At that point it seems the customer account is created/saved to the database, and the payment address they entered is saved to customer_ID = 1! (not to their new customer_ID). The checkout asks for the shipping address and still allows them to proceed with checkout, but the order comes in without a payment address. BUt i can see that customer's address in the very first customer's account (customer_ID = 1). It also throws an error in my PayPal Payflow Pro iFrame gateway (not even sure why it still accepts the payment without a payment address but that's a different issue):
2018-09-26 18:06:10 - PHP Notice: Undefined index: code in catalog/controller/extension/payment/pp_payflow_iframe.php on line 46
2018-09-26 18:06:10 - PHP Notice: Undefined index: iso_code_2 in catalog/controller/extension/payment/pp_payflow_iframe.php on line 48

I have turned off my themes but it still does the same. I have installed i on 2 difference servers and again, same issue. Am I the only one???

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am

Post by ipthena » Thu Sep 27, 2018 10:12 pm

There seems to be a problem with obtaining the last customer ID added to the database in file catalog\model\account\customer.php line 16:
$customer_id = $this->db->getLastId();

I also get an error when I try my Affiliate Registration:
PHP Fatal error: Uncaught Exception: Error: Duplicate entry '1' for key 'PRIMARY'<br />Error No: 1062<br />INSERT INTO customer_affiliate SET `customer_id` = '1'

So that function getLastId() is pulling the first ID instead of the last. I'm not the most skilled PHP developer, so help is appreciated thanks!

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am

Post by straightlight » Fri Sep 28, 2018 7:14 am

The last ID method does pull the last inserted ID rather than the first as it is supposed to. However, what you might be looking for would be the next un-existing one next to the latest added ID on the database. Let's try this.

In catalog/model/account/customer.php file,

find:

Code: Select all

$customer_id = $this->db->getLastId();
replace with:

Code: Select all

$customer_id = $this->db->getLastId('customer_id', DB_PREFIX . 'customer');
Then, in system/library/db/mysqli.php file (if using mySQLi extension), replace:

Code: Select all

public function getLastId() {
		return $this->connection->insert_id;
	}
with:

Code: Select all

public function getLastId($column = '', $table = '') {
		if (!empty($column) && !empty($table)) {
			$sql = "SELECT MAX(`" . $column . "`) AS `max_id` FROM `" . $table . "`";
			
			$row = $this->connection->query($sql);
			
			return (($row['max_id'] + 1) > 0 ? (int)$row['max_id'] + 1 : 1);		
		} else {
			return $this->connection->insert_id;
		}
	}
Then, try again with the checkout customer registration. This modification also works for the customer registration without checkout on the store-front end.

Dedication and passion goes to those who are able to push and merge a project.

Regards,
Straightlight
Programmer / Opencart Tester


Legendary Member

Posts

Joined
Mon Nov 14, 2011 11:38 pm
Location - Canada, ON

Post by ipthena » Fri Sep 28, 2018 9:10 pm

Thanks straightlight! Is this a known bug in OC 3.0.2.0? I would think more people would have noticed it. I hesitate changing that piece of code if it's not the real cause of the issue. I've been doing a lot of googling. Does it matter that i'm hosted on GoDaddy and maybe something isn't turned on to all that code to run properly?

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am

Post by straightlight » Sat Sep 29, 2018 4:21 am

If you hesitate to follow the suggested code change, then you'll never know if it does actually resolved the issue. However, it is not an OC bug.

Dedication and passion goes to those who are able to push and merge a project.

Regards,
Straightlight
Programmer / Opencart Tester


Legendary Member

Posts

Joined
Mon Nov 14, 2011 11:38 pm
Location - Canada, ON

Post by ipthena » Sat Sep 29, 2018 5:14 am

Well I'm just concerned that this issue may be caused by some other factor (not the OC code itself) - maybe where i'm hosted, maybe a plugin. Anyway, i did test your suggestions but it's still doing the same thing. I really had hoped that your code would have patched it up for me so i could finalize this project! :) I cleared my OC cache as well as my browser cache and still got the same result: new customer registration payment address was saved to customer_ID 1

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am

Post by straightlight » Sat Sep 29, 2018 5:33 am

If the new customer ID was saved as value 1, then it means that the last ID from your database could not be tracked in from your server. Not an OC issue as all OC does at this point is to gather the last inserted ID and with my modifications, it adds a + 1 to it. If the + 1 result cannot be returned, it's an issue from your server.

Dedication and passion goes to those who are able to push and merge a project.

Regards,
Straightlight
Programmer / Opencart Tester


Legendary Member

Posts

Joined
Mon Nov 14, 2011 11:38 pm
Location - Canada, ON

Post by ipthena » Sat Sep 29, 2018 5:48 am

i appreciate your help! I'm on GoDaddy. I have OC 1.5.6.1 hosted on there as well (same hosting account) - and that did not have an issue. I compared both mysqli.php files and they both get the last inserted ID the same - same function. So now I'm even more perplexed.

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am

Post by ipthena » Tue Oct 02, 2018 3:17 am

SOLVED IT! After a TON of testing, I discovered that the GetResponse Module was causing the issue. I have not yet investigated as to why it was causing it, but i have confirmed from uninstalling the module the cart worked properly, then re-installed it and it again failed. I will be contacting GetResponse to report the issue. I very much appreciate your help! Hope this thread helps someone else that may be having the same issue.

Newbie

Posts

Joined
Sun Dec 07, 2014 3:18 am
Who is online

Users browsing this forum: No registered users and 33 guests