Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strict Standard Errors with PHP 5.4 and WP 3.6 #101

Open
kristofferR opened this issue Aug 12, 2013 · 2 comments
Open

Strict Standard Errors with PHP 5.4 and WP 3.6 #101

kristofferR opened this issue Aug 12, 2013 · 2 comments

Comments

@kristofferR
Copy link

After updating to WP 3.6 I get a bunch of PHP Strict Standard errors due to Front-End Editor:

Strict Standards: Static function FEE_Field_Base::get_object_type() should not be abstract in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/base.php on line 36

Strict Standards: Declaration of FEE_Field_Terms::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 275

Strict Standards: Declaration of FEE_Field_Tags::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 284

Strict Standards: Declaration of FEE_Field_Category::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 293

Strict Standards: Declaration of FEE_Field_Post_Thumbnail::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 335

Strict Standards: Declaration of FEE_Field_Post_Meta::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 401

Strict Standards: Declaration of FEE_Field_Widget::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/widget.php on line 98

Strict Standards: Declaration of FEE_Field_Comment::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 54

Strict Standards: Declaration of FEE_Field_Term_Field::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 104

Strict Standards: Declaration of FEE_Field_Single_Title::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 120

Strict Standards: Declaration of FEE_Field_Option::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 267

I know I could ignore the errors by changing my PHP configuration, but the code should preferably be fixed anyway.

@scribu
Copy link
Owner

scribu commented Aug 12, 2013

Care to open a pull request?

On Monday, August 12, 2013, Kristoffer wrote:

After updating to WP 3.6 I get a bunch of PHP Strict Standard errors due
to Front-End Editor:

Strict Standards: Static function FEE_Field_Base::get_object_type() should not be abstract in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/base.php on line 36

Strict Standards: Declaration of FEE_Field_Terms::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 275

Strict Standards: Declaration of FEE_Field_Tags::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 284

Strict Standards: Declaration of FEE_Field_Category::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 293

Strict Standards: Declaration of FEE_Field_Post_Thumbnail::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 335

Strict Standards: Declaration of FEE_Field_Post_Meta::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/post.php on line 401

Strict Standards: Declaration of FEE_Field_Widget::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/widget.php on line 98

Strict Standards: Declaration of FEE_Field_Comment::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 54

Strict Standards: Declaration of FEE_Field_Term_Field::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 104

Strict Standards: Declaration of FEE_Field_Single_Title::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 120

Strict Standards: Declaration of FEE_Field_Option::wrap() should be compatible with FEE_Field_Base::wrap($content, $data) in /Users/kristoffer/Sites/wpsite/wp-content/plugins/front-end-editor/php/fields/other.php on line 267

I know I could ignore the errors by changing my PHP configuration, but the
code should preferably be fixed anyway.


Reply to this email directly or view it on GitHubhttps://github.com//issues/101
.

http://scribu.net

@scribu
Copy link
Owner

scribu commented Jan 7, 2014

See #102

scribu pushed a commit that referenced this issue Aug 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants