search

kraftbj / genesis-enews-extended

WordPress widget to provide additional functionality to the Genesis eNews widget.
https://kraft.blog/genesis-enews-extended/
Other
12 stars 15 forks source link

Various WP_Widget errors #77

Closed kraftbj closed 11 years ago

kraftbj commented 11 years ago

Strict Standards: Declaration of BJGK_Genesis_eNews_Extended::widget() should be compatible with WP_Widget::widget($args, $instance) in class-bjgk-genesis-enews-extended.php on line 0

Strict Standards: Declaration of BJGK_Genesis_eNews_Extended::update() should be compatible with WP_Widget::update($new_instance, $old_instance) in class-bjgk-genesis-enews-extended.php on line 0

Strict Standards: Declaration of BJGK_Genesis_eNews_Extended::form() should be compatible with WP_Widget::form($instance) in class-bjgk-genesis-enews-extended.php on line 0

hobbsy commented 11 years ago

feel free to contact me Brandon if you need any more information, or if you have difficulties reproducing this error message

kimparsell commented 11 years ago

I can confirm that the error still exists (Genesis 2.0.1, Genesis eNews Extended 1.3.1, WordPress 3.6).

kraftbj commented 11 years ago

I wasn't seeing this since I'm typically on PHP 5.3. Upgraded one server to 5.4 to duplicate error. It was being caused by my use of type hints in the functions, e.g. function update( array $new_instance, array $old_instance ) instead of function update( $new_instance, $old_instance ), which is why the errors were being thrown.