diff options
author | Mario <mario@mariovavti.com> | 2024-11-14 18:40:15 +0000 |
---|---|---|
committer | Mario <mario@mariovavti.com> | 2024-11-14 18:40:15 +0000 |
commit | 01da60b385edad423396abd0858ead5d65a41801 (patch) | |
tree | b29b7e6413d0e02e97962e7a99004503f75e97f3 /Zotlabs/Module/Oauth.php | |
parent | 72c47cd8db94032bf05a6956f3f15888772051aa (diff) | |
parent | 951800eca6045f6e0dc1e9a12c225c8e55b044e9 (diff) | |
download | volse-hubzilla-01da60b385edad423396abd0858ead5d65a41801.tar.gz volse-hubzilla-01da60b385edad423396abd0858ead5d65a41801.tar.bz2 volse-hubzilla-01da60b385edad423396abd0858ead5d65a41801.zip |
Merge branch 'various-phpstan-issues' into 'dev'
Several issues discovered by PHPStan
See merge request hubzilla/core!2168
Diffstat (limited to 'Zotlabs/Module/Oauth.php')
-rw-r--r-- | Zotlabs/Module/Oauth.php | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/Zotlabs/Module/Oauth.php b/Zotlabs/Module/Oauth.php index 061296257..6063c7738 100644 --- a/Zotlabs/Module/Oauth.php +++ b/Zotlabs/Module/Oauth.php @@ -103,7 +103,8 @@ class Oauth extends Controller { if((argc() > 1) && (argv(1) === 'add')) { $tpl = get_markup_template("oauth_edit.tpl"); - $o .= replace_macros($tpl, array( + + return replace_macros($tpl, array( '$form_security_token' => get_form_security_token("oauth"), '$title' => t('Add application'), '$submit' => t('Submit'), @@ -114,7 +115,6 @@ class Oauth extends Controller { '$redirect' => array('redirect', t('Redirect'), '', t('Redirect URI - leave blank unless your application specifically requires this')), '$icon' => array('icon', t('Icon url'), '', t('Optional')), )); - return $o; } if((argc() > 2) && (argv(1) === 'edit')) { @@ -129,7 +129,7 @@ class Oauth extends Controller { $app = $r[0]; $tpl = get_markup_template("oauth_edit.tpl"); - $o .= replace_macros($tpl, array( + return replace_macros($tpl, array( '$form_security_token' => get_form_security_token("oauth"), '$title' => t('Add application'), '$submit' => t('Update'), @@ -140,7 +140,6 @@ class Oauth extends Controller { '$redirect' => array('redirect', t('Redirect'), $app['redirect_uri'], ''), '$icon' => array('icon', t('Icon url'), $app['icon'], ''), )); - return $o; } if((argc() > 2) && (argv(1) === 'delete')) { @@ -163,7 +162,8 @@ class Oauth extends Controller { $tpl = get_markup_template("oauth.tpl"); - $o .= replace_macros($tpl, array( + + return replace_macros($tpl, array( '$form_security_token' => get_form_security_token("oauth"), '$baseurl' => z_root(), '$title' => t('Connected OAuth Apps'), @@ -175,8 +175,6 @@ class Oauth extends Controller { '$remove' => t('Remove authorization'), '$apps' => $r, )); - return $o; - } } |