Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[-] IN : Fix #PSCSX-6535 + PSCSX-6346 adding id_cart to id_product_2 … #3714

Merged
merged 1 commit into from Aug 17, 2015

Conversation

@gRoussac
Copy link
Contributor

commented Aug 17, 2015

…key in specific_price

@jnadaud

This comment has been minimized.

Copy link
Contributor

commented Aug 17, 2015

👍

jnadaud pushed a commit that referenced this pull request Aug 17, 2015
Jérôme Nadaud
Merge pull request #3714 from gRoussac/PSCSX-6535
[-] IN : Fix #PSCSX-6535 + PSCSX-6346 adding id_cart to id_product_2 …

@jnadaud jnadaud merged commit 467b74c into PrestaShop:1.6.1.x Aug 17, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@gRoussac gRoussac deleted the gRoussac:PSCSX-6535 branch Aug 17, 2015

jnadaud pushed a commit to jnadaud/PrestaShop that referenced this pull request Aug 17, 2015
Jérôme Nadaud Jérôme Nadaud
Merge pull request PrestaShop#3714 from gRoussac/PSCSX-6535
[-] IN : Fix #PSCSX-6535 + PSCSX-6346 adding id_cart to id_product_2 …
jnadaud pushed a commit that referenced this pull request Aug 20, 2015
Jérôme Nadaud Jérôme Nadaud
Merge pull request #3714 from gRoussac/PSCSX-6535
[-] IN : Fix #PSCSX-6535 + PSCSX-6346 adding id_cart to id_product_2 …
jnadaud pushed a commit that referenced this pull request Aug 20, 2015
Jérôme Nadaud Jérôme Nadaud
Merge pull request #3714 from gRoussac/PSCSX-6535
[-] IN : Fix #PSCSX-6535 + PSCSX-6346 adding id_cart to id_product_2 …
@sagenterprises

This comment has been minimized.

Copy link

commented on 56744d3 Sep 3, 2015

I am facing same concern.... Can you please confirm in which files these changed need to be incorporated.... as unable to find any such files
install-dev/data/db_structure.sql
install-dev/upgrade/sql/1.6.1.0.sql
install-dev/upgrade/sql/1.6.1.1.sql

This comment has been minimized.

Copy link
Contributor Author

replied Sep 3, 2015

Hi,

If you are in 1.6.1.0 or 1.6.1.1 you need to run thses queries against your database directly

See http://forge.prestashop.com/browse/PSCSX-6430

Regards

This comment has been minimized.

Copy link

replied Sep 3, 2015

Queries Only with + like we are on 1.6.1.1, so we run 2 queries one by one.... Is it right

This comment has been minimized.

Copy link
Contributor Author

replied Sep 3, 2015

yep

This comment has been minimized.

Copy link

replied Sep 3, 2015

ALTER TABLE PREFIX_specific_price DROP KEY id_product_2;
RESULT >> #1091 - Can't DROP 'id_product_2'; check that column/key exists

This comment has been minimized.

Copy link

replied Sep 3, 2015

Disabled all overrides

Deprecated: mysql_connect(): The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead in /home/esadar/public_html/classes/db/MySQL.php on line 51

Warning: Cannot modify header information - headers already sent by (output started at /home/esadar/public_html/classes/db/MySQL.php:51) in /home/esadar/public_html/classes/controller/FrontController.php on line 790

Warning: Cannot modify header information - headers already sent by (output started at /home/esadar/public_html/classes/db/MySQL.php:51) in /home/esadar/public_html/classes/controller/FrontController.php on line 791

Warning: Cannot modify header information - headers already sent by (output started at /home/esadar/public_html/classes/db/MySQL.php:51) in /home/esadar/public_html/classes/controller/FrontController.php on line 793

This comment has been minimized.

Copy link
Contributor Author

replied Sep 3, 2015

Hi,

ALTER TABLE nmkz_specific_price ENGINE=MyISAM;
then execute the query again
ALTER IGNORE TABLE nmkz_specific_price ADD UNIQUE KEY id_product_2 (id_cart, id_product,id_shop,id_shop_group,id_currency,id_country,id_group,id_customer,id_product_attribute,from_quantity,id_specific_price_rule,from,to);
(http://forge.prestashop.com/browse/NM-540)

then ALTER TABLE nmkz_specific_price ENGINE=InnoDB;
__

mysql_connect is deprecated, we can not do anything about that expect bypassing (like in PrestaShop/autoupgrade@c9bafc7) it as I already told you.

Regards

This comment has been minimized.

Copy link

replied Sep 3, 2015

This comment has been minimized.

Copy link

replied Sep 3, 2015

This comment has been minimized.

Copy link

replied Nov 25, 2015

Same problem passing from 1.6.0.1 to 1.6.1.2

Specific prices are not showing up in front office.
I add the index id_product_2 but no success.
I wonder if the specificprice_priority is not the problem because the value of the field has completely changed with shop_id added.

What I can see is that $specificprice array is empty in product.php classe.
Where can I find where this object is read ?

Jean-Marie

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.