From: Michael Vogel <icarus@dabo.de>
Date: Tue, 14 Aug 2018 19:37:44 +0000 (+0200)
Subject: Removed notices in the frio theme (#5610)
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=1283b99c6f018cf0ec52fc482fd70e7b6d80602c;p=friendica.git

Removed notices in the frio theme (#5610)

* Removed notices in the frio theme

* And fixed notices in the notifications

* And some more in frio

* Null is the default value

* And some small notice in the probing removed

* Added note

* Clarified note

* And a removed notice in OStatus
---

diff --git a/src/Core/NotificationsManager.php b/src/Core/NotificationsManager.php
index 734e4eb030..c994755e7a 100644
--- a/src/Core/NotificationsManager.php
+++ b/src/Core/NotificationsManager.php
@@ -165,39 +165,41 @@ class NotificationsManager extends BaseObject
 	 */
 	public function getTabs()
 	{
+		$selected = defaults(self::getApp()->argv, 1, '');
+
 		$tabs = [
 			[
 				'label' => L10n::t('System'),
 				'url'   => 'notifications/system',
-				'sel'   => ((self::getApp()->argv[1] == 'system') ? 'active' : ''),
+				'sel'   => (($selected == 'system') ? 'active' : ''),
 				'id'    => 'system-tab',
 				'accesskey' => 'y',
 			],
 			[
 				'label' => L10n::t('Network'),
 				'url'   => 'notifications/network',
-				'sel'   => ((self::getApp()->argv[1] == 'network') ? 'active' : ''),
+				'sel'   => (($selected == 'network') ? 'active' : ''),
 				'id'    => 'network-tab',
 				'accesskey' => 'w',
 			],
 			[
 				'label' => L10n::t('Personal'),
 				'url'   => 'notifications/personal',
-				'sel'   => ((self::getApp()->argv[1] == 'personal') ? 'active' : ''),
+				'sel'   => (($selected == 'personal') ? 'active' : ''),
 				'id'    => 'personal-tab',
 				'accesskey' => 'r',
 			],
 			[
 				'label' => L10n::t('Home'),
 				'url'   => 'notifications/home',
-				'sel'   => ((self::getApp()->argv[1] == 'home') ? 'active' : ''),
+				'sel'   => (($selected == 'home') ? 'active' : ''),
 				'id'    => 'home-tab',
 				'accesskey' => 'h',
 			],
 			[
 				'label' => L10n::t('Introductions'),
 				'url'   => 'notifications/intros',
-				'sel'   => ((self::getApp()->argv[1] == 'intros') ? 'active' : ''),
+				'sel'   => (($selected == 'intros') ? 'active' : ''),
 				'id'    => 'intro-tab',
 				'accesskey' => 'i',
 			],
diff --git a/src/Network/Probe.php b/src/Network/Probe.php
index 49c27fb4f2..366853c999 100644
--- a/src/Network/Probe.php
+++ b/src/Network/Probe.php
@@ -232,7 +232,7 @@ class Probe
 
 		if (!$lrdd) {
 			$parts = @parse_url($uri);
-			if (!$parts) {
+			if (!$parts || empty($parts["host"]) || empty($parts["path"])) {
 				return [];
 			}
 
diff --git a/src/Protocol/OStatus.php b/src/Protocol/OStatus.php
index 80c6edbe9f..971a91fa12 100644
--- a/src/Protocol/OStatus.php
+++ b/src/Protocol/OStatus.php
@@ -1351,13 +1351,15 @@ class OStatus
 
 		switch ($siteinfo["type"]) {
 			case 'photo':
-				$imgdata = Image::getInfoFromURL($siteinfo["image"]);
-				if ($imgdata) {
-					$attributes = ["rel" => "enclosure",
-							"href" => $siteinfo["image"],
-							"type" => $imgdata["mime"],
-							"length" => intval($imgdata["size"])];
-					XML::addElement($doc, $root, "link", "", $attributes);
+				if (!empty($siteinfo["image"])) {
+					$imgdata = Image::getInfoFromURL($siteinfo["image"]);
+					if ($imgdata) {
+						$attributes = ["rel" => "enclosure",
+								"href" => $siteinfo["image"],
+								"type" => $imgdata["mime"],
+								"length" => intval($imgdata["size"])];
+						XML::addElement($doc, $root, "link", "", $attributes);
+					}
 				}
 				break;
 			case 'video':
diff --git a/view/theme/frio/config.php b/view/theme/frio/config.php
index 55a653696a..13ab78477c 100644
--- a/view/theme/frio/config.php
+++ b/view/theme/frio/config.php
@@ -15,14 +15,14 @@ function theme_post(App $a)
 	}
 
 	if (isset($_POST['frio-settings-submit'])) {
-		PConfig::set(local_user(), 'frio', 'scheme',           $_POST['frio_scheme']);
-		PConfig::set(local_user(), 'frio', 'nav_bg',           $_POST['frio_nav_bg']);
-		PConfig::set(local_user(), 'frio', 'nav_icon_color',   $_POST['frio_nav_icon_color']);
-		PConfig::set(local_user(), 'frio', 'link_color',       $_POST['frio_link_color']);
-		PConfig::set(local_user(), 'frio', 'background_color', $_POST['frio_background_color']);
-		PConfig::set(local_user(), 'frio', 'contentbg_transp', $_POST['frio_contentbg_transp']);
-		PConfig::set(local_user(), 'frio', 'background_image', $_POST['frio_background_image']);
-		PConfig::set(local_user(), 'frio', 'bg_image_option',  $_POST['frio_bg_image_option']);
+		PConfig::set(local_user(), 'frio', 'scheme',           defaults($_POST, 'frio_scheme'));
+		PConfig::set(local_user(), 'frio', 'nav_bg',           defaults($_POST, 'frio_nav_bg'));
+		PConfig::set(local_user(), 'frio', 'nav_icon_color',   defaults($_POST, 'frio_nav_icon_color'));
+		PConfig::set(local_user(), 'frio', 'link_color',       defaults($_POST, 'frio_link_color'));
+		PConfig::set(local_user(), 'frio', 'background_color', defaults($_POST, 'frio_background_color'));
+		PConfig::set(local_user(), 'frio', 'contentbg_transp', defaults($_POST, 'frio_contentbg_transp'));
+		PConfig::set(local_user(), 'frio', 'background_image', defaults($_POST, 'frio_background_image'));
+		PConfig::set(local_user(), 'frio', 'bg_image_option',  defaults($_POST, 'frio_bg_image_option'));
 		PConfig::set(local_user(), 'frio', 'css_modified',     time());
 	}
 }
diff --git a/view/theme/frio/php/scheme.php b/view/theme/frio/php/scheme.php
index 538839b2c2..05829bb21f 100644
--- a/view/theme/frio/php/scheme.php
+++ b/view/theme/frio/php/scheme.php
@@ -46,7 +46,11 @@ function get_scheme_info($scheme)
 		foreach ($ll as $l) {
 			$l = trim($l, "\t\n\r */");
 			if ($l != '') {
-				list($k, $v) = array_map('trim', explode(':', $l, 2));
+				$values = array_map('trim', explode(':', $l, 2));
+				if (count($values) < 2) {
+					continue;
+				}
+				list($k, $v) = $values;
 				$k = strtolower($k);
 				if ($k == 'author') {
 					$r = preg_match('|([^<]+)<([^>]+)>|', $v, $m);
diff --git a/view/theme/frio/style.php b/view/theme/frio/style.php
index 30f8a0d771..e0f65960f6 100644
--- a/view/theme/frio/style.php
+++ b/view/theme/frio/style.php
@@ -225,7 +225,8 @@ header('ETag: "' . $etag . '"');
 header('Last-Modified: ' . $modified);
 
 // Only send the CSS file if it was changed.
-if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE']) || isset($_SERVER['HTTP_IF_NONE_MATCH'])) {
+/// @todo Check if this works at all (possibly clients are sending only the one or the other header) - compare with mod/photo.php
+if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE']) && isset($_SERVER['HTTP_IF_NONE_MATCH'])) {
 	$cached_modified = gmdate('r', strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']));
 	$cached_etag = str_replace(['"', '-gzip'], ['', ''],
 				stripslashes($_SERVER['HTTP_IF_NONE_MATCH']));