vafour / vafpress-framework

Wordpress theme options framework, check out the demo.
http://demo.vafpress.com/vafpress-framework
Other
240 stars 98 forks source link

Is it possible fix those error messages when set WP_DEBUG to true? #105

Open ghost opened 9 years ago

ghost commented 9 years ago

Hi

No doubt that it's an awesome framework! Only have one problem, Is it possible fix those error messages when set WP_DEBUG to true? Because those error messages make me inconvenient to debug the theme, and the other side, I want to sumbit my theme to themeforest, so they requires that fix the error messages as far as I can.

Thanks

s43 commented 9 years ago

Hi Johnny,

It seems that you have something else wrong, cause I'm using the framework under the WP_DEBUG mode and I don't have any errors coming from the framework

otherwise you'll need to check those errors you're getting carefully to see from where it comes exactly and fix them for yourself.

Enjoy

ghost commented 9 years ago

I'm not quite sure what I was missing. The error message only appeared when I go to theme option page.

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'vp_bind_bigcontinents' not found or invalid function name in /Users/Johnny/projects/ThemeVan/Offical_Site/demo/wp-content/themes/Miao/framework/classes/control/fieldmulti.php on line 35

Warning: array_merge(): Argument #2 is not an array in /Users/Johnny/projects/ThemeVan/Offical_Site/demo/wp-content/themes/Miao/framework/classes/control/fieldmulti.php on line 36

Notice: Trying to get property of non-object in /Users/Johnny/projects/ThemeVan/Offical_Site/demo/wp-content/themes/Miao/framework/classes/control/fieldmulti.php on line 91

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'vp_bind_continents' not found or invalid function name in /Users/Johnny/projects/ThemeVan/Offical_Site/demo/wp-content/themes/Miao/framework/classes/option/control/set.php on line 250

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'vp_bind_countries' not found or invalid function name in /Users/Johnny/projects/ThemeVan/Offical_Site/demo/wp-content/themes/Miao/framework/classes/option/control/set.php on line 250

s43 commented 9 years ago

Hmm, for the first error, I think it's related to the VP_Security's whitelist function .. If I am not wrong you removed the function vp_bind_bigcontinents from the data_sources.php, but you kept the whitelist trigger ..

VP_Security::instance()->whitelist_function('vp_bind_bigcontinents');

This function executes a callback of vp_bind_bigcontinents, which actually doesn't exist on your project ...

it'll be fixed once you remove that line too, and I think the other errors are caused of the same issue.

ghost commented 9 years ago

I haven't created data_source.php file in currently, maybe I see! After I create the data source file, the error will be fixed, right?

s43 notifications@github.com于2015年2月3日星期二写道:

Hmm, for the first error, I think it's related to the VP_Security's whitelist function .. If I am not wrong you removed the function vp_bind_bigcontinents from the data_sources.php, but you kept the whitelist trigger ..

VP_Security::instance()->whitelist_function('vp_bind_bigcontinents');

This function executes a callback of vp_bind_bigcontinents, which actually doesn't exist on your project ...

it'll be fixed once you remove that line too, and I think the other errors are caused of the same issue.

— Reply to this email directly or view it on GitHub https://github.com/vafour/vafpress-framework/issues/105#issuecomment-72563782 .

Cheers!

BadJohnny

“The support service is not include customization or third-party plugin problem, it's only include trouble using and bug fixing. Thanks!.”

ThemeVan- Premium WordPress Themes Website: http://www.themevan.com Support Forum: http://www.themevan.com/support Google+ Group: https://plus.google.com/communities/114840886520756430726 Facebook Page: http://facebook.com/ThemeVan Twitter: http://twitter.com/ThemeVan

s43 commented 9 years ago

You didn't created it, but if you used the framework's theme boilerplate, it's already on it.

Just search for "vp_bind_bigcontinents" and you'll find that line somewhere, comment it ( or remove )

ghost commented 9 years ago

Yes, Fixed!

Thanks so much!

s43 notifications@github.com于2015年2月3日星期二写道:

You didn't created it, but if you used the framework's theme boilerplate, it's already on it.

Just search for "vp_bind_bigcontinents" and you'll find that line somewhere, comment it ( or remove )

— Reply to this email directly or view it on GitHub https://github.com/vafour/vafpress-framework/issues/105#issuecomment-72625020 .

Cheers!

BadJohnny

“The support service is not include customization or third-party plugin problem, it's only include trouble using and bug fixing. Thanks!.”

ThemeVan- Premium WordPress Themes Website: http://www.themevan.com Support Forum: http://www.themevan.com/support Google+ Group: https://plus.google.com/communities/114840886520756430726 Facebook Page: http://facebook.com/ThemeVan Twitter: http://twitter.com/ThemeVan

s43 commented 9 years ago

Glad to hear that ;)