Title: Missing arguments for WooCommerce Last modified: August 31, 2016 --- # Missing arguments for WooCommerce * Resolved [John Chadwick](https://wordpress.org/support/users/chadwickjs/) * (@chadwickjs) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/) * Hi, Plugin working fine until I upgraded (v1.4) this evening. Every page of site now has various missing argument texts before the header of the site. I have had to activate maintenance mode as this is a live commerce site. * I am using the Divi theme and I understand there may be js issues with Divi however all the errors relate to; * Warning: Missing argument 4 for Popmake_Woocommerce_Integration::popup_is_loadable() in /home/bigburymint/public_html/wp-content/plugins/popup-maker/includes/integrations/ class-popmake-woocommerce-integration.php on line 104 * and * Warning: array_key_exists() expects parameter 2 to be array, null given in /home/ bigburymint/public_html/wp-content/plugins/popup-maker/includes/integrations/ class-popmake-woocommerce-integration.php on line 110 * the above lines repeat at least twenty times on each page before the site header. * Deactivating the popupmaker plugin stops the error but obviously I now have nil popups. * Hope the above is enough to go on. I cannot make the site live to show error as it would be viewed by potential customers. 8-( * Thank you * John Chadwick * [https://wordpress.org/plugins/popup-maker/](https://wordpress.org/plugins/popup-maker/) Viewing 15 replies - 1 through 15 (of 23 total) 1 [2](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/page/2/?output_format=md) [→](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/page/2/?output_format=md) * Thread Starter [John Chadwick](https://wordpress.org/support/users/chadwickjs/) * (@chadwickjs) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302393) * Update – I have successfully re-installed previous version – 1.3.9 and site is back live. * I realise the error will not be present but should any admin wish to see the site to see if there is anything obvious; * [bigburymint.com](https://bigburymint.com) * Thank you * John * Thread Starter [John Chadwick](https://wordpress.org/support/users/chadwickjs/) * (@chadwickjs) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302394) * Update 2 – Just in case anyone has to go through the above! * On rolling the plugin back to v1.3.9 I noticed that none of the popups were working. I had to go into each popup in the admin and reselect a “targeting Condition” and then reselect the theme for the popup in order to get them working again. * Sorry if all this is a bit long winded but I want to be as helpful as possible. 😎 * [doublesharp](https://wordpress.org/support/users/doublesharp/) * (@doublesharp) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302397) * This is being caused by the call to `return apply_filters( 'popmake_popup_is_loadable', $loadable, $popup_id );` in `includes/pum-deprecated.php`. The filter function in `includes/integrations/class-popmake-woocommerce-integration.php` takes 4 arguments and has no defaults. * Either need to change `includes/pum-deprecated.php` to: * ``` // or whatever the appropriate default values are for args 3 and 4 return apply_filters( 'popmake_popup_is_loadable', $loadable, $popup_id, array(), false ); ``` * Or change `includes/integrations/class-popmake-woocommerce-integration.php` to: * ``` // or whatever the appropriate default values are for args 3 and 4 public function popup_is_loadable( $is_loadable, $popup_id, $conditions=array(), $sitewide=false ) { ``` * This former seems like it would best make it backwards compatible. * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302401) * [@chadwickjs](https://wordpress.org/support/users/chadwickjs/) – Looking into this now. Will get this included in the next patch tonight. Planned to be online all night pushing patches as fast as issues get reported. Thanks for the report. Will let you know when I know the solution. * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302403) * [@doublesharp](https://wordpress.org/support/users/doublesharp/) – That looks about right, thanks for reporting the fix. I am about to push v1.4.2 shourtly which will include this once I double check it. * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302404) * Did either of you process the upgrade routine? Once processed that file shouldn’t be called, just wondering if that has anything to do with it. * In v1.4 after migration is done all new conditions are added and replace that entire class. * I just patched with both suggestions made by [@doublesharp](https://wordpress.org/support/users/doublesharp/), look for a release shortly. * [doublesharp](https://wordpress.org/support/users/doublesharp/) * (@doublesharp) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302408) * Not immediately, but the upgrade was processed… fwiw I am using nginx, php-fpm, php7, and mariadb so kind of but not really standard. * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302409) * [@doublesharp](https://wordpress.org/support/users/doublesharp/) – Ok I am pushing v1.4.2 including those patches, so if they worked for you should work for everyone else too. Not sure how I missed those :(.. * [doublesharp](https://wordpress.org/support/users/doublesharp/) * (@doublesharp) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302410) * I’m also using a cache, which I just cleared. It doesn’t always flush things as well as one might hope if that could be related. * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302411) * [@doublesharp](https://wordpress.org/support/users/doublesharp/) – No problem, please test v1.4.2 guys and let me know if that solves your problems. or at least this one. Long night ahead for me 🙂 * [adam.coppin](https://wordpress.org/support/users/adamcoppin/) * (@adamcoppin) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302425) * I am getting: * Warning: Cannot modify header information – headers already sent by (output started at /home/XXXXXX/public_html/wp-content/plugins/popup-maker/includes/integrations/ class-popmake-woocommerce-integration.php:1) in /home/smocpt/public_html/wp-includes/ pluggable.php on line 1171 * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302434) * [@adam](https://wordpress.org/support/users/adam/).coppin – Sorry about that, the last patch had an accidental space added somehwere it should’t be, patch is already pushed, should be available in your dash shortly. * [adam.coppin](https://wordpress.org/support/users/adamcoppin/) * (@adamcoppin) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302437) * I have had to roll back to 1.3.9 too.. thanks to [@chadwickjs](https://wordpress.org/support/users/chadwickjs/) for the pointer for resetting the targetting and theme… I ended up having to reset my theme background too, but that fixed everything… * [adam.coppin](https://wordpress.org/support/users/adamcoppin/) * (@adamcoppin) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302438) * [@danieliser](https://wordpress.org/support/users/danieliser/) – thanks for the extremely rapid response! * Bit late for me tonight, so I may try this again tomorrow… * Plugin Author [Daniel Iser](https://wordpress.org/support/users/danieliser/) * (@danieliser) * [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/#post-7302440) * [@adam](https://wordpress.org/support/users/adam/).coppin – Sorry about that, if you can bear with me through the next day or so, we will get every issue patched very rapidly. This was a massive update, over 5 months in the making, so I knew and planned for a few days of rapid bug patches. Viewing 15 replies - 1 through 15 (of 23 total) 1 [2](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/page/2/?output_format=md) [→](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/page/2/?output_format=md) The topic ‘Missing arguments for WooCommerce’ is closed to new replies. * ![](https://ps.w.org/popup-maker/assets/icon-256x256.gif?rev=3097653) * [Popup Maker - Boost Sales, Conversions, Optins, Subscribers with the Ultimate WP Popup Builder](https://wordpress.org/plugins/popup-maker/) * [Frequently Asked Questions](https://wordpress.org/plugins/popup-maker/#faq) * [Support Threads](https://wordpress.org/support/plugin/popup-maker/) * [Active Topics](https://wordpress.org/support/plugin/popup-maker/active/) * [Unresolved Topics](https://wordpress.org/support/plugin/popup-maker/unresolved/) * [Reviews](https://wordpress.org/support/plugin/popup-maker/reviews/) * 23 replies * 6 participants * Last reply from: [Daniel Iser](https://wordpress.org/support/users/danieliser/) * Last activity: [10 years, 2 months ago](https://wordpress.org/support/topic/missing-arguments-for-woocommerce/page/2/#post-7302690) * Status: resolved