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.
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://ww.wp.xz.cn/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 2 years, 7 months ago by
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' )
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.
Yes a very simple issue that can be fix 🙂 let me know when you publish a new version.
Thanks,
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.