$data can turn into bool here

[friendica.git] / src / Util / ParseUrl.php

diff --git a/src/Util/ParseUrl.php b/src/Util/ParseUrl.php
index 1cc35fadcc2d07e7fd37e892c00dabfa818bab30..9f0fd8a9bca82be9681597314bdd52324a341e68 100644 (file)
--- a/src/Util/ParseUrl.php
+++ b/src/Util/ParseUrl.php
@@ -59,7 +59,7 @@ class ParseUrl
         * @param string $accept content-type to accept
         * @return array content type
         */
-       public static function getContentType(string $url, string $accept = HttpClientAccept::DEFAULT)
+       public static function getContentType(string $url, string $accept = HttpClientAccept::DEFAULT): array
        {
                $curlResult = DI::httpClient()->head($url, [HttpClientOptions::ACCEPT_CONTENT => $accept]);
 
@@ -69,6 +69,7 @@ class ParseUrl
                }
 
                if (!$curlResult->isSuccess()) {
+                       Logger::debug('Got HTTP Error', ['http error' => $curlResult->getReturnCode(), 'url' => $url]);
                        return [];
                }
 
@@ -100,7 +101,7 @@ class ParseUrl
         * @see   ParseUrl::getSiteinfo() for more information about scraping
         * embeddable content
         */
-       public static function getSiteinfoCached($url, $do_oembed = true): array
+       public static function getSiteinfoCached(string $url, bool $do_oembed = true): array
        {
                if (empty($url)) {
                        return [
@@ -179,7 +180,7 @@ class ParseUrl
         * </body>
         * @endverbatim
         */
-       public static function getSiteinfo($url, $do_oembed = true, $count = 1)
+       public static function getSiteinfo(string $url, bool $do_oembed = true, int $count = 1)
        {
                if (empty($url)) {
                        return [
@@ -632,15 +633,16 @@ class ParseUrl
         * @param string $string Tags
         * @return array with formatted Hashtags
         */
-       public static function convertTagsToArray($string)
+       public static function convertTagsToArray(string $string): array
        {
                $arr_tags = str_getcsv($string);
                if (count($arr_tags)) {
                        // add the # sign to every tag
-                       array_walk($arr_tags, ["self", "arrAddHashes"]);
+                       array_walk($arr_tags, ['self', 'arrAddHashes']);
 
                        return $arr_tags;
                }
+               return [];
        }
 
        /**
@@ -652,9 +654,9 @@ class ParseUrl
         * @param int    $k   Counter for internal use
         * @return void
         */
-       private static function arrAddHashes(&$tag, $k)
+       private static function arrAddHashes(string &$tag, int $k)
        {
-               $tag = "#" . $tag;
+               $tag = '#' . $tag;
        }
 
        /**
@@ -671,41 +673,41 @@ class ParseUrl
         *
         * @return string The url with a scheme
         */
-       private static function completeUrl($url, $scheme)
+       private static function completeUrl(string $url, string $scheme): string
        {
                $urlarr = parse_url($url);
 
                // If the url does allready have an scheme
                // we can stop the process here
-               if (isset($urlarr["scheme"])) {
-                       return($url);
+               if (isset($urlarr['scheme'])) {
+                       return $url;
                }
 
                $schemearr = parse_url($scheme);
 
-               $complete = $schemearr["scheme"]."://".$schemearr["host"];
+               $complete = $schemearr['scheme'] . '://' . $schemearr['host'];
 
-               if (!empty($schemearr["port"])) {
-                       $complete .= ":".$schemearr["port"];
+               if (!empty($schemearr['port'])) {
+                       $complete .= ':' . $schemearr['port'];
                }
 
-               if (!empty($urlarr["path"])) {
-                       if (strpos($urlarr["path"], "/") !== 0) {
-                               $complete .= "/";
+               if (!empty($urlarr['path'])) {
+                       if (strpos($urlarr['path'], '/') !== 0) {
+                               $complete .= '/';
                        }
 
-                       $complete .= $urlarr["path"];
+                       $complete .= $urlarr['path'];
                }
 
-               if (!empty($urlarr["query"])) {
-                       $complete .= "?".$urlarr["query"];
+               if (!empty($urlarr['query'])) {
+                       $complete .= '?' . $urlarr['query'];
                }
 
-               if (!empty($urlarr["fragment"])) {
-                       $complete .= "#".$urlarr["fragment"];
+               if (!empty($urlarr['fragment'])) {
+                       $complete .= '#' . $urlarr['fragment'];
                }
 
-               return($complete);
+               return $complete;
        }
 
        /**
@@ -715,7 +717,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseParts(array $siteinfo, array $jsonld)
+       private static function parseParts(array $siteinfo, array $jsonld): array
        {
                if (!empty($jsonld['@graph']) && is_array($jsonld['@graph'])) {
                        foreach ($jsonld['@graph'] as $part) {
@@ -760,7 +762,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLd(array $siteinfo, array $jsonld)
+       private static function parseJsonLd(array $siteinfo, array $jsonld): array
        {
                $type = JsonLD::fetchElement($jsonld, '@type');
                if (empty($type)) {
@@ -853,7 +855,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdAuthor(array $siteinfo, array $jsonld)
+       private static function parseJsonLdAuthor(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -937,7 +939,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdArticle(array $siteinfo, array $jsonld)
+       private static function parseJsonLdArticle(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -1007,7 +1009,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdWebPage(array $siteinfo, array $jsonld)
+       private static function parseJsonLdWebPage(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -1046,7 +1048,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdWebSite(array $siteinfo, array $jsonld)
+       private static function parseJsonLdWebSite(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -1084,7 +1086,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdWebOrganization(array $siteinfo, array $jsonld)
+       private static function parseJsonLdWebOrganization(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -1130,7 +1132,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdWebPerson(array $siteinfo, array $jsonld)
+       private static function parseJsonLdWebPerson(array $siteinfo, array $jsonld): array
        {
                $jsonldinfo = [];
 
@@ -1175,7 +1177,7 @@ class ParseUrl
         * @param array $jsonld
         * @return array siteinfo
         */
-       private static function parseJsonLdMediaObject(array $siteinfo, array $jsonld, string $name)
+       private static function parseJsonLdMediaObject(array $siteinfo, array $jsonld, string $name): array
        {
                $media = [];