diff --git a/.woodpecker/.continuous-deployment.yml b/.woodpecker/.continuous-deployment.yml index 5a29fe612..a486228c7 100644 --- a/.woodpecker/.continuous-deployment.yml +++ b/.woodpecker/.continuous-deployment.yml @@ -1,8 +1,3 @@ -matrix: - include: - - PHP_MAJOR_VERSION: 7.4 - PHP_VERSION: 7.4.18 - depends_on: - phpunit - code_standards_check @@ -33,7 +28,7 @@ pipeline: settings: backend: "filesystem" restore: true - cache_key: "{{ .Repo.Name }}_php${PHP_MAJOR_VERSION}_{{ arch }}_{{ os }}" + cache_key: "{{ .Repo.Name }}_php7.4_{{ arch }}_{{ os }}" archive_format: "gzip" mount: - '.composer' @@ -44,7 +39,7 @@ pipeline: branch: [ develop, '*-rc' ] event: push composer_install: - image: friendicaci/php${PHP_MAJOR_VERSION}:php${PHP_VERSION} + image: friendicaci/php7.4:php7.4.18 commands: - export COMPOSER_HOME=.composer - composer validate diff --git a/.woodpecker/.releaser.yml b/.woodpecker/.releaser.yml index 5b56a6e1d..d254b8199 100644 --- a/.woodpecker/.releaser.yml +++ b/.woodpecker/.releaser.yml @@ -1,8 +1,3 @@ -matrix: - include: - - PHP_MAJOR_VERSION: 7.4 - PHP_VERSION: 7.4.18 - depends_on: - phpunit - code_standards_check @@ -31,7 +26,7 @@ pipeline: settings: backend: "filesystem" restore: true - cache_key: "{{ .Repo.Name }}_php${PHP_MAJOR_VERSION}_{{ arch }}_{{ os }}" + cache_key: "{{ .Repo.Name }}_php7.4_{{ arch }}_{{ os }}" archive_format: "gzip" mount: - '.composer' @@ -42,7 +37,7 @@ pipeline: branch: stable event: tag composer_install: - image: friendicaci/php${PHP_MAJOR_VERSION}:php${PHP_VERSION} + image: friendicaci/php7.4:php7.4.18 commands: - export COMPOSER_HOME=.composer - composer validate diff --git a/src/Console/ServerBlock.php b/src/Console/ServerBlock.php index 38ae35d10..c25b57ae7 100644 --- a/src/Console/ServerBlock.php +++ b/src/Console/ServerBlock.php @@ -118,6 +118,9 @@ HELP; foreach ($blocklist as $domain) { fputcsv($fp, $domain); } + + // Success + return 0; } /** * Imports a list of domains and a reason for the block from a CSV @@ -130,6 +133,7 @@ HELP; $filename = $this->getArgument(1); $currBlockList = $config->get('system', 'blocklist', []); $newBlockList = []; + if (($fp = fopen($filename, 'r')) !== false) { while (($data = fgetcsv($fp, 1000, ',')) !== false) { $domain = $data[0]; @@ -146,11 +150,13 @@ HELP; $newBlockList[] = $data; } } + foreach ($currBlockList as $blocked) { if (!in_array($blocked, $newBlockList)) { $newBlockList[] = $blocked; } } + if ($config->set('system', 'blocklist', $newBlockList)) { $this->out(sprintf("Entries from %s that were not blocked before are now blocked", $filename)); return 0; @@ -158,7 +164,6 @@ HELP; $this->out(sprintf("Couldn't save '%s' as blocked server", $domain)); return 1; } - } else { throw new Exception(sprintf('The file "%s" could not be opened for importing', $filename)); } @@ -178,6 +183,9 @@ HELP; $table->addRow($domain); } $this->out($table->getTable()); + + // Success + return 0; } /** diff --git a/src/Model/Contact.php b/src/Model/Contact.php index 5997b1569..a6bb882e8 100644 --- a/src/Model/Contact.php +++ b/src/Model/Contact.php @@ -1169,11 +1169,11 @@ class Contact * @throws HTTPException\InternalServerErrorException * @throws \ImagickException */ - public static function getIdForURL(string $url, int $uid = 0, $update = null, array $default = []): int + public static function getIdForURL(string $url = null, int $uid = 0, $update = null, array $default = []): int { $contact_id = 0; - if ($url == '') { + if (empty($url)) { Logger::notice('Empty url, quitting', ['url' => $url, 'user' => $uid, 'default' => $default]); return 0; } diff --git a/src/Protocol/ActivityPub/Processor.php b/src/Protocol/ActivityPub/Processor.php index cc869e00e..c6420436e 100644 --- a/src/Protocol/ActivityPub/Processor.php +++ b/src/Protocol/ActivityPub/Processor.php @@ -900,7 +900,7 @@ class Processor } // Store send a follow request for every reshare - but only when the item had been stored - if ($stored && ($item['private'] != Item::PRIVATE) && ($item['gravity'] == GRAVITY_PARENT) && ($item['author-link'] != $item['owner-link'])) { + if ($stored && ($item['private'] != Item::PRIVATE) && ($item['gravity'] == GRAVITY_PARENT) && !empty($item['author-link']) && ($item['author-link'] != $item['owner-link'])) { $author = APContact::getByURL($item['owner-link'], false); // We send automatic follow requests for reshared messages. (We don't need though for forum posts) if ($author['type'] != 'Group') { diff --git a/src/Protocol/DFRN.php b/src/Protocol/DFRN.php index edd6c1450..625afb215 100644 --- a/src/Protocol/DFRN.php +++ b/src/Protocol/DFRN.php @@ -48,10 +48,10 @@ use Friendica\Network\Probe; use Friendica\Util\Crypto; use Friendica\Util\DateTimeFormat; use Friendica\Util\Images; -use Friendica\Util\Network; use Friendica\Util\Proxy; use Friendica\Util\Strings; use Friendica\Util\XML; +use GuzzleHttp\Psr7\Uri; /** * This class contain functions to create and send DFRN XML files @@ -1013,7 +1013,7 @@ class DFRN $path_parts = explode('/', $parts['path']); array_pop($path_parts); $parts['path'] = implode('/', $path_parts); - $contact['batch'] = Network::unparseURL($parts); + $contact['batch'] = Uri::fromParts($parts); } $dest_url = ($public_batch ? $contact['batch'] : $contact['notify']); diff --git a/src/Protocol/Diaspora.php b/src/Protocol/Diaspora.php index 838d40550..6d3c23dad 100644 --- a/src/Protocol/Diaspora.php +++ b/src/Protocol/Diaspora.php @@ -50,6 +50,7 @@ use Friendica\Util\Network; use Friendica\Util\Strings; use Friendica\Util\XML; use Friendica\Worker\Delivery; +use GuzzleHttp\Psr7\Uri; use SimpleXMLElement; /** @@ -755,14 +756,14 @@ class Diaspora * @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \ImagickException */ - private static function key(string $handle): string + private static function key(string $handle = null): string { $handle = strval($handle); Logger::notice("Fetching diaspora key for: " . $handle); $fcontact = FContact::getByURL($handle); - if ($fcontact) { + if (!empty($fcontact['pubkey'])) { return $fcontact['pubkey']; } @@ -1417,7 +1418,7 @@ class Diaspora $parts = parse_url($person['url']); unset($parts['path']); - $host_url = Network::unparseURL($parts); + $host_url = Uri::fromParts($parts); return $host_url . '/objects/' . $guid; } @@ -4006,12 +4007,12 @@ class Diaspora /** * Sends profile data * - * @param int $uid The user id - * @param bool $recips optional, default false + * @param int $uid The user id + * @param array $recips optional, default empty array * @return void * @throws \Exception */ - public static function sendProfile(int $uid, bool $recips = false) + public static function sendProfile(int $uid, array $recips = []) { if (!$uid) { return; diff --git a/src/Protocol/Email.php b/src/Protocol/Email.php index 758451844..577760a25 100644 --- a/src/Protocol/Email.php +++ b/src/Protocol/Email.php @@ -115,13 +115,13 @@ class Email } /** - * @param Connection|resource $mbox mailbox - * @param integer $uid user id + * @param Connection|resource $mbox mailbox + * @param string $sequence * @return mixed */ - public static function messageMeta($mbox, int $uid) + public static function messageMeta($mbox, string $sequence) { - $ret = (($mbox && $uid) ? @imap_fetch_overview($mbox, $uid, FT_UID) : [[]]); // POSSIBLE CLEANUP --> array(array()) is probably redundant now + $ret = (($mbox && $sequence) ? @imap_fetch_overview($mbox, $sequence, FT_UID) : [[]]); // POSSIBLE CLEANUP --> array(array()) is probably redundant now return (count($ret)) ? $ret : []; } @@ -296,6 +296,7 @@ class Email } return $x; } + return ''; } /** @@ -569,9 +570,9 @@ class Email * Removes signature from message * * @param string $message Unfiltered message - * @return string Message with no signature + * @return array Message array with no signature (elements "body" and "sig") */ - private static function removeSig(string $message): string + private static function removeSig(string $message): array { $sigpos = strrpos($message, "\n-- \n"); $quotepos = strrpos($message, "[/quote]"); @@ -662,7 +663,7 @@ class Email return implode("\n", $lines); } - private static function convertQuote(strng $body, string $reply): string + private static function convertQuote(string $body, string $reply): string { // Convert Quotes $arrbody = explode("\n", trim($body)); diff --git a/src/Protocol/OStatus.php b/src/Protocol/OStatus.php index 36dbb06c8..1efbb0bf3 100644 --- a/src/Protocol/OStatus.php +++ b/src/Protocol/OStatus.php @@ -1168,7 +1168,7 @@ class OStatus break; default: - Logger::warning('Unsupported rel=' . $attribute['rel'] . ', href=' . $attribute['href'] . ', object-type=' . $attribute['object-type']); + Logger::warning('Unsupported rel=' . $attribute['rel'] . ', href=' . $attribute['href'] . ', object-type=' . $item['object-type']); } } } diff --git a/src/Util/XML.php b/src/Util/XML.php index 0424400a8..e371ea3cd 100644 --- a/src/Util/XML.php +++ b/src/Util/XML.php @@ -42,9 +42,9 @@ class XML * @param bool $remove_header Should the XML header be removed or not? * @param array $namespaces List of namespaces * @param bool $root interally used parameter. Mustn't be used from outside. - * @return void + * @return string */ - public static function fromArray(array $array, &$xml, bool $remove_header = false, array $namespaces = [], bool $root = true) + public static function fromArray(array $array, &$xml, bool $remove_header = false, array $namespaces = [], bool $root = true): string { if ($root) { foreach ($array as $key => $value) { @@ -130,19 +130,20 @@ class XML self::fromArray($value, $element, $remove_header, $namespaces, false); } } + return ''; } /** * Copies an XML object * - * @param object $source The XML source - * @param object $target The XML target - * @param string $elementname Name of the XML element of the target + * @param object|string $source The XML source + * @param object $target The XML target + * @param string $elementname Name of the XML element of the target * @return void */ public static function copy(&$source, &$target, string $elementname) { - if (count($source->children()) == 0) { + if (is_string($source)) { $target->addChild($elementname, self::escape($source)); } else { $child = $target->addChild($elementname); @@ -184,9 +185,9 @@ class XML * @param array $attributes Array containing the attributes * @return void */ - public static function addElement(DOMDocument $doc, DOMElement &$parent, string $element, string $value = '', array $attributes = []) + public static function addElement(DOMDocument $doc, DOMElement &$parent, string $element, string $value = null, array $attributes = []) { - $element = self::createElement($doc, $element, $value, $attributes); + $element = self::createElement($doc, $element, $value ?? '', $attributes); $parent->appendChild($element); } @@ -273,6 +274,7 @@ class XML return []; } + $parent = []; libxml_use_internal_errors(true); libxml_clear_errors(); diff --git a/tests/src/Core/Config/Cache/ConfigFileLoaderTest.php b/tests/src/Core/Config/Cache/ConfigFileLoaderTest.php index b956e558e..43decf105 100644 --- a/tests/src/Core/Config/Cache/ConfigFileLoaderTest.php +++ b/tests/src/Core/Config/Cache/ConfigFileLoaderTest.php @@ -174,7 +174,7 @@ class ConfigFileLoaderTest extends MockedTest ->at($this->root) ->setContent(file_get_contents($file)); - $configFileLoader = new \Friendica\Core\Config\Util\ConfigFileLoader( + $configFileLoader = new ConfigFileLoader( $this->root->url(), $this->root->url() . DIRECTORY_SEPARATOR . Config::CONFIG_DIR, $this->root->url() . DIRECTORY_SEPARATOR . Config::STATIC_DIR @@ -228,7 +228,7 @@ class ConfigFileLoaderTest extends MockedTest ->at($this->root->getChild('addon')->getChild('test')->getChild('config')) ->setContent(file_get_contents($file)); - $configFileLoader = new \Friendica\Core\Config\Util\ConfigFileLoader( + $configFileLoader = new ConfigFileLoader( $this->root->url(), $this->root->url() . DIRECTORY_SEPARATOR . Config::CONFIG_DIR, $this->root->url() . DIRECTORY_SEPARATOR . Config::STATIC_DIR @@ -265,7 +265,7 @@ class ConfigFileLoaderTest extends MockedTest ->at($this->root->getChild('config')) ->setContent(file_get_contents($fileDir . 'B.config.php')); - $configFileLoader = new \Friendica\Core\Config\Util\ConfigFileLoader( + $configFileLoader = new ConfigFileLoader( $this->root->url(), $this->root->url() . DIRECTORY_SEPARATOR . Config::CONFIG_DIR, $this->root->url() . DIRECTORY_SEPARATOR . Config::STATIC_DIR @@ -299,7 +299,7 @@ class ConfigFileLoaderTest extends MockedTest ->at($this->root->getChild('config')) ->setContent(file_get_contents($fileDir . 'B.ini.php')); - $configFileLoader = new \Friendica\Core\Config\Util\ConfigFileLoader( + $configFileLoader = new ConfigFileLoader( $this->root->url(), $this->root->url() . DIRECTORY_SEPARATOR . Config::CONFIG_DIR, $this->root->url() . DIRECTORY_SEPARATOR . Config::STATIC_DIR @@ -333,7 +333,7 @@ class ConfigFileLoaderTest extends MockedTest ->at($this->root->getChild('config')) ->setContent(file_get_contents($fileDir . 'B.ini.php')); - $configFileLoader = new \Friendica\Core\Config\Util\ConfigFileLoader( + $configFileLoader = new ConfigFileLoader( $this->root->url(), $this->root->url() . DIRECTORY_SEPARATOR . Config::CONFIG_DIR, $this->root->url() . DIRECTORY_SEPARATOR . Config::STATIC_DIR