-
Notifications
You must be signed in to change notification settings - Fork 109
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
[blogger] Cleanups: #656
[blogger] Cleanups: #656
Conversation
- used `use Friendica\App;` - added type-hints for `App` and `array` where applyable - removed extra space - added other spaces
blogger/blogger.php
Outdated
} | ||
|
||
|
||
function blogger_jot_nets(&$a, &$b) | ||
{ | ||
function blogger_jot_nets(App $a, &$b) { |
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.
New line please.
blogger/blogger.php
Outdated
{ | ||
if (x($_POST, 'blogger-submit')) { | ||
PConfig::set(local_user(), 'blogger', 'post', intval($_POST['blogger'])); | ||
if (isset($_POST['blogger-submit'])) { |
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.
The exact equivalent to x($_POST, 'blogger-submit')
is !empty($_POST['blogger-submit'])
.
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.
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.
Aren't you supposed to sleep? 😅
- ops, curly brace in new line in function/method declaration - proper way for x() replacement
use Friendica\App;
App
andarray
where applyable