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
Fix wrong use of class constants in DFRN #5036
Conversation
I had a bunch of errors of the following type (PHP 7.2.3 / Ubuntu 18.04 LTS)
|
Could you change the target of the PR to the 2018.05-rc branch? this is clearly a bugfix and not a new feature. |
Here you are, @tobiasd |
src/Protocol/DFRN.php
Outdated
@@ -2141,7 +2141,7 @@ private static function updateContent($current, $item, $importer, $entrytype) | |||
|
|||
$changed = true; | |||
|
|||
if ($entrytype == DFRN_REPLY_RC) { | |||
if ($entrytype == DFRN::DFRN_REPLY_RC) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be DFRN::REPLY_RC
instead, can you please rename all DFRN_*
constants to just *
and modify their use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 736c76d
Pleasure doing business with you sir! |
http://php.net/manual/language.oop5.constants.php