If you encounter any problems or bugs with that version, please check first the issue section at Github (especially if you are a developer).
If you can't find there an entry or you are not firm with Github, post the issue here.
Full Stack Web Developer :: Dedicated OpenCart Development & Support DACH Region
Contact for Custom Work / Fast Support.
Edit catalog/model/account/customer.php
And change this line: ( 130 )
Code: Select all
$this->db->query("INSERT INTO " . DB_PREFIX . "customer_affiliate SET `customer_id` = '" . (int)$customer_id . "', `company` = '" . $this->db->escape($data['company']) . "', `website` = '" . $this->db->escape($data['website']) . "', `tracking` = '" . $this->db->escape(token(64)) . "', `commission` = '" . (float)$this->config->get('config_affiliate_commission') . "', `tax` = '" . $this->db->escape($data['tax']) . "', `payment` = '" . $this->db->escape($data['payment']) . "', `cheque` = '" . $this->db->escape($data['cheque']) . "', `paypal` = '" . $this->db->escape($data['paypal']) . "', `bank_name` = '" . $this->db->escape($data['bank_name']) . "', `bank_branch_number` = '" . $this->db->escape($data['bank_branch_number']) . "', `bank_swift_code` = '" . $this->db->escape($data['bank_swift_code']) . "', `bank_account_name` = '" . $this->db->escape($data['bank_account_name']) . "', `bank_account_number` = '" . $this->db->escape($data['bank_account_number']) . "', `status` = '" . (int)!$this->config->get('config_affiliate_approval') . "'");
Code: Select all
$this->db->query("INSERT INTO " . DB_PREFIX . "customer_affiliate SET `customer_id` = '" . (int)$customer_id . "', `company` = '" . $this->db->escape($data['company']) . "', `website` = '" . $this->db->escape($data['website']) . "', `tracking` = '" . $this->db->escape(token(64)) . "', `commission` = '" . (float)$this->config->get('config_affiliate_commission') . "', `tax` = '" . $this->db->escape($data['tax']) . "', `payment` = '" . $this->db->escape($data['payment']) . "', `cheque` = '" . $this->db->escape($data['cheque']) . "', `paypal` = '" . $this->db->escape($data['paypal']) . "', `bank_name` = '" . $this->db->escape($data['bank_name']) . "', `bank_branch_number` = '" . $this->db->escape($data['bank_branch_number']) . "', `bank_swift_code` = '" . $this->db->escape($data['bank_swift_code']) . "', `bank_account_name` = '" . $this->db->escape($data['bank_account_name']) . "', `bank_account_number` = '" . $this->db->escape($data['bank_account_number']) . "', custom_field = '" . $this->db->escape(isset($data['custom_field']['affiliate']) ? json_encode($data['custom_field']['affiliate']) : json_encode(array())) . "', `status` = '" . (int)!$this->config->get('config_affiliate_approval') . "'");
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
https://github.com/opencart/opencart/pull/10323/files
This was changed from 64 to 10 like in Master Branch Admin. If you don't like it...change 10 from admin to 64.
https://github.com/opencart/opencart/pull/10324/files
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
https://github.com/opencart/opencart/pull/10325/files
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
Unknown: Return type of Twig\Node\Node::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in D:\wamp64\www\workpage\magazin\system\storage\vendor\twig\twig\src\Node\Node.php on line 161Unknown: Return type of Twig\Node\Node::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in D:\wamp64\www\market\system\storage\vendor\twig\twig\src\Node\Node.php on line 166After refreshing page , the error disappears.
When i go to my market after instalation have error:
Unknown: mysqli::real_escape_string(): Passing null to parameter #1 ($string) of type string is deprecated in D:\wamp64\www\market\system\library\db\mysqli.php on line 53Unknown: Automatic conversion of false to array is deprecated in D:\wamp64\www\market\catalog\controller\startup\startup.php on line 103Unknown: Return type of Twig\Node\Node::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in D:\wamp64\www\market\system\storage\vendor\twig\twig\src\Node\Node.php on line 161Unknown: Return type of Twig\Node\Node::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in D:\wamp64\www\market\system\storage\vendor\twig\twig\src\Node\Node.php on line 166Unknown: round(): Passing null to parameter #1 ($num) of type int|float is deprecated in D:\wamp64\www\market\catalog\model\catalog\product.php on line 45Unknown: round(): Passing null to parameter #1 ($num) of type int|float is deprecated in D:\wamp64\www\market\catalog\model\catalog\product.php on line 45Unknown: round(): Passing null to parameter #1 ($num) of type int|float is deprecated in D:\wamp64\www\market\catalog\model\catalog\product.php on line 45Unknown: round(): Passing null to parameter #1 ($num) of type int|float is deprecated in D:\wamp64\www\market\catalog\model\catalog\product.php on line 45
- After edit price and rating from the D:\wamp64\www\market\catalog\model\catalog\product.php remaining just one error:
Unknown: mysqli::real_escape_string(): Passing null to parameter #1 ($string) of type string is deprecated in D:\wamp64\www\market\system\library\db\mysqli.php on line 53
In line 53 have that:
public function escape($value) {
return $this->connection->real_escape_string($value); }
After that all i try to do nothing happened just show me error Unknown: mysqli::real_escape_string(): Passing null to parameter #1 ($string) of type string is deprecated in D:\wamp64\www\market\system\library\db\mysqli.php on line 53
Thx
viewtopic.php?f=199&t=227216&p=836849#p836876
And is not a bug...just not compatible with PHP 8.1.
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
warning: googleshopping\googleshopping::integer(): argument #1 ($product_id) must be passed by reference, value given in /home/admin/public_html/system/library/googleshopping/googleshopping.php
Try. https://github.com/opencart/opencart/pull/10836jerry1mundy wrote: ↑Sun May 22, 2022 7:29 amI can't delete products that come with a new install using php8.0 the error I get
warning: googleshopping\googleshopping::integer(): argument #1 ($product_id) must be passed by reference, value given in /home/admin/public_html/system/library/googleshopping/googleshopping.php
If a customer has 2 custom_fields: one is for Account (named custom_field_account) second is for Affiliate (named custom_field_affiliate)
When customer is edited and both custom_fields are filled in with values, these values are inserted together in database in both tables.
customer table
customer_affiliate table
When custom_field_account value should be inserted only in customer table and custom_field_affiliate value should be inserted only in customer_affiliate table.
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
Please create an incident request or create a pull request on the opencart-3 repository on Github about it.xxvirusxx wrote: ↑Sun Jun 12, 2022 4:50 amAdmin custom_fields bug when a Customer is edited
If a customer has 2 custom_fields: one is for Account (named custom_field_account) second is for Affiliate (named custom_field_affiliate)
When customer is edited and both custom_fields are filled in with values, these values are inserted together in database in both tables.
customer table
customer_affiliate table
When custom_field_account value should be inserted only in customer table and custom_field_affiliate value should be inserted only in customer_affiliate table.
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Doesn't matter.straightlight wrote: ↑Sun Jun 12, 2022 10:17 amPlease create an incident request or create a pull request on the opencart-3 repository on Github about it.
Will be unfixed like Time Zone bug and there are not developers here
But can be added structure from 4.0
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
The timezone function has been integrated into the opencart-3 already since that fix portion was also integrated in the master branch for the admin settings recently.xxvirusxx wrote: ↑Sun Jun 12, 2022 5:13 pmDoesn't matter.straightlight wrote: ↑Sun Jun 12, 2022 10:17 amPlease create an incident request or create a pull request on the opencart-3 repository on Github about it.
Will be unfixed like Time Zone bug and there are not developers here
But can be added structure from 4.0
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Yes, after 1 yearstraightlight wrote: ↑Sun Jun 12, 2022 7:08 pmThe timezone function has been integrated into the opencart-3 already since that fix portion was also integrated in the master branch for the admin settings recently.
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
Could of been longer. At least, now, it's in.xxvirusxx wrote: ↑Sun Jun 12, 2022 9:27 pmYes, after 1 yearstraightlight wrote: ↑Sun Jun 12, 2022 7:08 pmThe timezone function has been integrated into the opencart-3 already since that fix portion was also integrated in the master branch for the admin settings recently.
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Yes, for 4.0.0.0 is ok.
Not for 3.0.3.8
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Yes, on Catalog side but not Adminstraightlight wrote: ↑Mon Jun 13, 2022 5:57 amThe custom fields have now been officially fixed on the opencart-3 repository.
Upgrade Service | OC 2.3.0.2 PHP 8 | My Custom OC 3.0.3.8 | Buy me a beer
It's in progress.xxvirusxx wrote: ↑Mon Jun 13, 2022 4:15 pmYes, on Catalog side but not Adminstraightlight wrote: ↑Mon Jun 13, 2022 5:57 amThe custom fields have now been officially fixed on the opencart-3 repository.
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Users browsing this forum: No registered users and 118 guests