• Resolved pulsedigit

    (@pulsedigit)


    Hello,

    i saw that previous conversation is not anymore visible on the old topic because it’s resolved. We have a client WP 5.9 with PHP 7.1 (due to client restriction on his server, and when we install the plugin we have this issue (i don’t know how to share with you the sceen capture).

    Can you help us on that issue ?

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Jeff Starr

    (@specialk)

    Yes glad to help. In order to investigate and try to resolve the issue, it needs to be repeatable on default WordPress site. If you can provide the basic steps to do so, I will take a look asap.

    Thread Starter pulsedigit

    (@pulsedigit)

    Easy to reproduce :

    install docker images from this github repository : https://github.com/gquittet/php-mariadb-apache (You will need to commentdelete le line about the XDebug installation from php Docker file)

    copy wordpress 5.9 fresh install into php folder (don’t forget to change owner and group to www-data) : https://www.ads-software.com/wordpress-5.9.8.tar.gz

    go to 127.0.0.1:7000 and install WordPress (DBname, User and Password is tdi )

    Once login as ADMIN install simple download counter

    When you activate the Plugin you will see the error message ??

    • This reply was modified 1 year, 1 month ago by pulsedigit.
    Thread Starter pulsedigit

    (@pulsedigit)

    Hello again,

    i find a way to fix it :

    ??????? add_submenu_page(
    ??????????????? 'edit.php?post_type=sdc_download',
    ??????????????? esc_html__('Settings', 'simple-download-counter'),
    ??????????????? esc_html__('Settings', 'simple-download-counter'),
    ??????????????? 'manage_options', 'download-counter-settings',
    ??? ???????? 'simple_download_counter_display_settings',
    
    ??????? );
    
    Should be replaced by this :
    
    ??????? add_submenu_page(
    
    ??????????????? 'edit.php?post_type=sdc_download',
    ??????????????? esc_html__('Settings', 'simple-download-counter'),
    ??????????????? esc_html__('Settings', 'simple-download-counter'),
    ??????????????? 'manage_options', 'download-counter-settings',
    ??????? ????????'simple_download_counter_display_settings'
    
    ??????? );
    
    ( i remove the last “,” after 'simple_download_counter_display_settings' )
    Plugin Author Jeff Starr

    (@specialk)

    Hey @pulsedigit, so if I understand correctly, the entire issue boils down to the trailing comma..? If so I can fix that up asap no problem.

    Thread Starter pulsedigit

    (@pulsedigit)

    Yes a very simple issue that can be fix ?? let me know when you publish a new version.

    Thanks,

    Plugin Author Jeff Starr

    (@specialk)

    Just to follow up, the trailing comma is removed in the upcoming version (1.7) of SDC, should be available today or tomorrow. Thank you for reporting and solution, much appreciated @pulsedigit.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘WP 5.9 + PHP 7.1 -> Parse error: syntax error, unexpected ‘)’’ is closed to new replies.