Page 15 of 16
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Mon Jun 22, 2020 8:02 am
by sw!tch
A non-numeric value encountered - From Voucher when a non-number is entered in amount..
Probably should patch this one since bots like to hit this form, not a huge issue, but it fills up error logs, etc.
Link to thread. -
viewtopic.php?t=215547#p773734
Was able to replicate on v3.0.3.4 a few versions back, not sure if you patched this one.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Mon Jun 22, 2020 11:10 am
by straightlight
sw!tch wrote: ↑Mon Jun 22, 2020 8:02 am
A non-numeric value encountered - From Voucher when a non-number is entered in amount..
Probably should patch this one since bots like to hit this form, not a huge issue, but it fills up error logs, etc.
Link to thread. -
viewtopic.php?t=215547#p773734
Was able to replicate on v3.0.3.4 a few versions back, not sure if you patched this one.
It was fixed based on the master branch but I think the solution we both worked on might be better as we already have the input from the OP of that topic indicating it did solved the issue. Let's see how it goes on this one.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Mon Jun 22, 2020 11:23 am
by straightlight
Package updated.
- Fixed catalog/controller/account/affiliate.php with custom fields.
- Fixed catalog/controller/account/forgotten.php
- Fixed catalog/controller/account/register.php
- Fixed catalog/controller/account/voucher.php as per post provided by sw!tch:
viewtopic.php?f=201&t=218203&p=791513#p791499 .
- Fixed catalog/controller/account/wishlist.php
- Fixed catalog/controller/affiliate/register.php
- Fixed catalog/controller/checkout/payment_address.php
- Fixed catalog/controller/checkout/shipping_address.php
- Fixed catalog/controller/extension/module/banner.php with html_entity_decode on the $result['image'] . Not yet in master branch
- Fixed catalog/controller/tool/upload.php as per master branch
- Fixed admin/controller/tool/upload.php as per master branch
- Fixed catalog/controller/extension/module/bestseller.php with review_status config; not yet on master branch
- Fixed admin/controller/catalog/filter.php as per this commit:
https://github.com/opencart/opencart/pull/8026
- Reverted to leafo vendor
- Replaced all JSON redirection / links with str_replace from & to '&' characters on the URL. Not yet on master branch
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Tue Jun 23, 2020 11:34 pm
by straightlight
Package updated
- Added more warnings in the OC admin as per this commit:
https://github.com/opencart/opencart/pu ... d825dcf60b .
- Fixed admin/controller/report/statistics.php as per this commit:
https://github.com/opencart/opencart/pu ... c28e6588ec
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 5:50 am
by straightlight
Package updated.
- Fixed wishlist conventional array names in catalog account/account and account/wishlist.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 6:50 am
by xxvirusxx
admin/controller/design/banner.php, change:
Code: Select all
if (is_file(DIR_IMAGE . $banner_image['image'])) {
to:
Code: Select all
if (is_file(DIR_IMAGE . html_entity_decode($banner_image['image'], ENT_QUOTES, 'UTF-8'))) {
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 10:10 am
by straightlight
xxvirusxx wrote: ↑Wed Jun 24, 2020 6:50 am
admin/controller/design/banner.php, change:
Code: Select all
if (is_file(DIR_IMAGE . $banner_image['image'])) {
to:
Code: Select all
if (is_file(DIR_IMAGE . html_entity_decode($banner_image['image'], ENT_QUOTES, 'UTF-8'))) {
Ok, fixed.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 1:57 pm
by xxvirusxx
Bank Transfer missing value:
admin/controller/extension/payment/bank_transfer.php
Change:
Code: Select all
$data['payment_bank_transfer'] = array();
to
Code: Select all
$data['payment_bank_transfer_bank'] = array();
https://github.com/opencart/opencart/pull/7936/files
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 5:53 pm
by straightlight
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Wed Jun 24, 2020 7:04 pm
by straightlight
Package updated.
- Fixed admin banner image, bank transfer array and removed text_empty in account/wishlist.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Thu Jun 25, 2020 11:35 pm
by straightlight
Package updated.
- Fixed all html_entity_decode with image post, database insertions and database updates.
- Fixed reserved terms with database with escaped quotes such as
type and
description.
- Fixed voucher as per this commit:
https://github.com/opencart/opencart/pull/8040 .
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 12:01 am
by straightlight
Package updated
- Completed description database fields with escaped quotes.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 1:22 am
by xxvirusxx
Language bug??
LE. You have updated Bootstrap? Because you use...
and
And same problem everywhere
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 1:54 am
by straightlight
Please provide the TWIG filenames!
Looked everywhere for fas and can't find any of it.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 4:01 am
by xxvirusxx
Is fking weird...
Now on Firefox I have this error..
Code: Select all
Parse error: syntax error, unexpected 'template' (T_STRING), expecting variable (T_VARIABLE) in catalog/controller/event/theme.php on line 3
Downloaded: opencart-3.0.3.4-core-pre.zip
The function is:
Code: Select all
public function index($route, $args, &template) {
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 6:53 am
by straightlight
xxvirusxx wrote: ↑Fri Jun 26, 2020 4:01 am
Is fking weird...
Now on Firefox I have this error..
Code: Select all
Parse error: syntax error, unexpected 'template' (T_STRING), expecting variable (T_VARIABLE) in catalog/controller/event/theme.php on line 3
Downloaded: opencart-3.0.3.4-core-pre.zip
The function is:
Code: Select all
public function index($route, $args, &template) {
Ok, fixed and package updated.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 1:38 pm
by xxvirusxx
Same issue.
Some text is missing. From Admin login page, Add to cart from catalog, some text from footer (informations)..etc
LE.
catalog/controller/event/language.php
When you change from:
Code: Select all
public function index($route, $args) {
to:
Code: Select all
public function index(&$route, &$args) {
Catalog language issue is fixed.
Same for admin.
admin/controller/event/language.php
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Fri Jun 26, 2020 10:11 pm
by ADD Creative
Missing Twig from composer.json. You need to correct that and then do a "composer update" to clean up the system/storage/vendor folder.
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Sat Jun 27, 2020 7:37 pm
by head_dunce
Thank you straightlight
Re: [SOLVED] - Opencart Pre-Release - Core v3.0.3.4
Posted: Sun Jun 28, 2020 2:42 pm
by xxvirusxx
-
admin/controller/customer/customer.php
Change this:
Code: Select all
'href' => $this->url->link('customer/customer/login', 'user_token=' . $this->session->data['user_token'] . '&customer_id=' . $result['customer_id'] . '&store_id=' . $result['store_id'], true)
To:
Code: Select all
'href' => $this->url->link('customer/customer/login', 'user_token=' . $this->session->data['user_token'] . '&customer_id=' . $result['customer_id'] . '&store_id=' . $store['store_id'], true)
https://github.com/opencart/opencart/bl ... r.php#L391
Same file. You forgot to change as Master
Code: Select all
$results = $this->model_customer_customer->getRewards($this->request->get['customer_id'], ($page - 1) * 10, 10);
to
Code: Select all
$results = $this->model_customer_customer->getRewards($customer_id, ($page - 1) * 10, 10);
-
admin/model/localisation/location.php
Change this:
Code: Select all
$this->db->query("DELETE FROM " . DB_PREFIX . "location WHERE location_id = " . (int)$location_id);
To this:
Code: Select all
$this->db->query("DELETE FROM " . DB_PREFIX . "location WHERE location_id = '" . (int)$location_id . "'");
-
/admin/view/template/marketing/marketing_form.twig
Change this:
Code: Select all
$('#input-code').on('keyup', function() {
To this:
Code: Select all
$('#input-code').on('keyup', function(e) {
Add this line after { % endif %} (on error_code)
Code: Select all
<small class="form-text text-muted">{{ help_code }}</small>
https://github.com/opencart/opencart/co ... 769ce6c400
-
admin/controller/sale/voucher.php, delete line 640
Code: Select all
$this->load->model('sale/voucher_theme');
-
catalog/controller/affiliate/register.php, missing brace after line 89 and hit Enter
-
catalog/controller/extension/total, remove credit.php file, is a model not controller.
- why you have changed from:
Code: Select all
$json['redirect'] = str_replace('&', '&', $this->url->link('checkout/cart', '', true));
to (line 155)
Code: Select all
$json['redirect'] = str_replace('&', '&', $this->url->link('checkout/checkout', '', true));
https://github.com/opencart/opencart/bl ... t.php#L155
-
/catalog/view/theme/default/template/checkout/register.twig
Change this: (line 322)
Code: Select all
{% if custom_field.type == 'time' %}
to this
Code: Select all
{% if custom_field.type == 'datetime' %}