IDStore – PHP 7 compatibility - by Cram

This topic has 4 replies, 2 voices, and was last updated 6 years, 7 months ago ago by Rose Tyler

  • Avatar: Cram
    Cram
    Participant
    September 14, 2017 at 09:31

    Hi,

    I passed the wpengine plugin php 7 compatibility check and it return the following errors:

    Name: IDStore – Responsive WordPress Theme

    FILE: /nas/content/live/cram/wp-content/themes/idstore/code/wordpress-importer/wordpress-importer.php
    —————————————————————————————————————————————————————————
    FOUND 1 ERROR AFFECTING 1 LINE
    —————————————————————————————————————————————————————————
    795 | ERROR | Indirect access to variables, properties and methods will be evaluated strictly in left-to-right order since PHP 7.0. Use curly braces to remove ambiguity.
    —————————————————————————————————————————————————————————

    FILE: /nas/content/live/cram/wp-content/themes/idstore/option-tree/includes/ot-functions-admin.php
    ————————————————————————————————————
    FOUND 2 ERRORS AFFECTING 1 LINE
    ————————————————————————————————————
    274 | ERROR | Extension ‘mysql_’ is deprecated since PHP 5.5 and removed since PHP 7.0; Use mysqli instead
    274 | ERROR | Extension ‘mysql_’ is deprecated since PHP 5.5 and removed since PHP 7.0; Use mysqli instead
    ————————————————————————————————————

    Do you have any time estimation to solve these problems with php 7?

    I read in an april post of the forum that you don’t have date estimated to prepare idstore to be compatible with php 7 and you recommended use XStore theme, but i have a question about that. If i have IDStore on my web, can i change it for XStore without loose all the customization of my pages ?

    Thanks in advance!.

    3 Answers
    Avatar: Rose Tyler
    Rose Tyler
    Support staff
    September 14, 2017 at 10:05

    Hello,

    We can not say the deadlines for solving this problem.
    We may offer you XStore theme, if you decide to buy XStore we’ll be able to give you refund for the IDStore theme. Your content of pages won’t be lost after change theme, but styles of site and pages will be changed because Xstore is another theme with own layouts and options. You may contact us at Live Chat to get the test details to check all the XStore features https://www.8theme.com/demo/xstore/preview-new/

    Regards

    Avatar: Cram
    Cram
    Participant
    September 14, 2017 at 16:54

    Hi,

    Taking into account that it was only two errors to solve, I applied the code modifications for myself. Here are the changes I made in the code:

    /idstore/code/wordpress-importer/wordpress-importer.php

    OLD LINE 795: $$meta[‘key’] = $meta[‘value’];
    NEW LINE 795: ${$meta[‘key’]} = $meta[‘value’];

    /idstore/option-tree/includes/ot-functions-admin.php

    OLD LINE 274: if ( mysql_num_rows( mysql_query( “SHOW TABLES LIKE ‘{$table_prefix}option_tree'” ) ) == 1 && $old_settings = $wpdb->get_results( “SELECT * FROM {$table_prefix}option_tree ORDER BY item_sort ASC” ) ) {

    NEW LINE 274: if ( mysqli_num_rows( mysqli_query( “SHOW TABLES LIKE ‘{$table_prefix}option_tree'” ) ) == 1 && $old_settings = $wpdb->get_results( “SELECT * FROM {$table_prefix}option_tree ORDER BY item_sort ASC” ) ) {

    I try these changes on staging web and after check that all is ok, i pass the compatibility php 7 check plugin and now IDStore appears like compatible, so, my production web will be migrated to PHP 7 during next hours.

    I am aware that I will have to repeat these changes every time the theme IDStore is updated, but I prefer to do so in order to pass the web to php 7. I appreciate any comments or clarification on this topic.

    Thanks for your help!.

    Avatar: Rose Tyler
    Rose Tyler
    Support staff
    September 15, 2017 at 12:30

    Hello,

    I am glad that you sorted out.
    I will send this topic to our developers, hope it will be useful in realizing compatibility our theme with php 7 and will be added in one of next theme updates.

    Regards

  • Viewing 4 results - 1 through 4 (of 4 total)

You must be logged in to reply to this topic.Log in/Sign up

We're using our own and third-party cookies to improve your experience and our website. Keep on browsing to accept our cookie policy.