Fix overly strict return value for terminateFriendship methods

[friendica.git] / mod / pubsub.php

diff --git a/mod/pubsub.php b/mod/pubsub.php
index fe5bb3c0b61e1b2575bec54a036722136a930069..1dfbd0b3dc9960f72848dc156400f6e2e489b5d9 100644 (file)
--- a/mod/pubsub.php
+++ b/mod/pubsub.php
@@ -50,8 +50,8 @@ function hub_post_return()
 
 function pubsub_init(App $a)
 {
-       $nick       = (($a->argc > 1) ? Strings::escapeTags(trim($a->argv[1])) : '');
-       $contact_id = (($a->argc > 2) ? intval($a->argv[2])       : 0 );
+       $nick       = ((DI::args()->getArgc() > 1) ? Strings::escapeTags(trim(DI::args()->getArgv()[1])) : '');
+       $contact_id = ((DI::args()->getArgc() > 2) ? intval(DI::args()->getArgv()[2])       : 0 );
 
        if ($_SERVER['REQUEST_METHOD'] === 'GET') {
                $hub_mode      = Strings::escapeTags(trim($_GET['hub_mode'] ?? ''));
@@ -96,7 +96,7 @@ function pubsub_init(App $a)
                }
 
                if (!empty($hub_mode)) {
-                       DBA::update('contact', ['subhub' => $subscribe], ['id' => $contact['id']]);
+                       Contact::update(['subhub' => $subscribe], ['id' => $contact['id']]);
                        Logger::log($hub_mode . ' success for contact ' . $contact_id . '.');
                }
                hub_return(true, $hub_challenge);
@@ -110,8 +110,8 @@ function pubsub_post(App $a)
        Logger::log('Feed arrived from ' . $_SERVER['REMOTE_ADDR'] . ' for ' .  DI::args()->getCommand() . ' with user-agent: ' . $_SERVER['HTTP_USER_AGENT']);
        Logger::log('Data: ' . $xml, Logger::DATA);
 
-       $nick       = (($a->argc > 1) ? Strings::escapeTags(trim($a->argv[1])) : '');
-       $contact_id = (($a->argc > 2) ? intval($a->argv[2])       : 0 );
+       $nick       = ((DI::args()->getArgc() > 1) ? Strings::escapeTags(trim(DI::args()->getArgv()[1])) : '');
+       $contact_id = ((DI::args()->getArgc() > 2) ? intval(DI::args()->getArgv()[2])       : 0 );
 
        $importer = DBA::selectFirst('user', [], ['nickname' => $nick, 'account_expired' => false, 'account_removed' => false]);
        if (!DBA::isResult($importer)) {