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.

Sign up for GitHub

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

Jump to bottom

Change $this->WP_Widget to parent::__construct #219

Closed
herrvigg opened this issue Jun 22, 2018 · 2 comments
Closed

Change $this->WP_Widget to parent::__construct #219

herrvigg opened this issue Jun 22, 2018 · 2 comments
Labels
duplicate This issue or pull request already exists legacy PR Legacy PR imported from original repo

Comments

@herrvigg
Copy link
Collaborator

Issue by adnanoner
Tuesday Aug 18, 2015 at 20:43 GMT
Originally opened as qTranslate-Team/qtranslate-x#219

Ensures compatibility with WP 4.3 (and php 7)

adnanoner included the following code: https://github.com/qTranslate-Team/qtranslate-x/pull/219/commits
@herrvigg herrvigg added duplicate This issue or pull request already exists legacy PR Legacy PR imported from original repo labels Jun 22, 2018
@herrvigg
Copy link
Collaborator Author

Comment by ianchanning
Wednesday Aug 26, 2015 at 09:18 GMT

I hadn't spotted this fix. This fixes #226 (so that it shows up in that issue). I've just put in the same fix in a pull request that also fixes the reasons behind the withdrawl from the WordPress repository #222.

@herrvigg
Copy link
Collaborator Author

Comment by johnclause
Tuesday Sep 01, 2015 at 23:49 GMT

It is taken care in qTranslate-Team/qtranslate-x#230
Thanks a lot for pointing it out though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists legacy PR Legacy PR imported from original repo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@herrvigg