From dc15e9a08037c648ff639051fb58c36db64def6b Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sat, 25 Nov 2017 21:30:56 -0500 Subject: [PATCH 1/2] Fix formatting for IFTTT and XMPP --- ifttt/ifttt.php | 232 ++++++++++++++++++++++++++---------------------- xmpp/xmpp.php | 106 ++++++++++++---------- 2 files changed, 188 insertions(+), 150 deletions(-) diff --git a/ifttt/ifttt.php b/ifttt/ifttt.php index 2828d502..800294ff 100644 --- a/ifttt/ifttt.php +++ b/ifttt/ifttt.php @@ -1,181 +1,203 @@ */ +require_once 'mod/item.php'; +require_once 'include/items.php'; +require_once 'include/text.php'; -require_once("mod/item.php"); -require_once("include/items.php"); - +use Friendica\App; use Friendica\Core\PConfig; +use Friendica\Database\DBM; -function ifttt_install() { - register_hook('connector_settings', 'addon/ifttt/ifttt.php', 'ifttt_settings'); - register_hook('connector_settings_post','addon/ifttt/ifttt.php', 'ifttt_settings_post'); +function ifttt_install() +{ + register_hook('connector_settings', 'addon/ifttt/ifttt.php', 'ifttt_settings'); + register_hook('connector_settings_post', 'addon/ifttt/ifttt.php', 'ifttt_settings_post'); } -function ifttt_uninstall() { - unregister_hook('connector_settings', 'addon/ifttt/ifttt.php', 'ifttt_settings'); +function ifttt_uninstall() +{ + unregister_hook('connector_settings', 'addon/ifttt/ifttt.php', 'ifttt_settings'); unregister_hook('connector_settings_post', 'addon/ifttt/ifttt.php', 'ifttt_settings_post'); } -function ifttt_module() { +function ifttt_module() +{ + } -function ifttt_content(&$a) { +function ifttt_content() +{ + } -function ifttt_settings(&$a,&$s) { +function ifttt_settings(App $a, &$s) +{ + if (!local_user()) { + return; + } - if(! local_user()) - return; - - $key = PConfig::get(local_user(),'ifttt','key'); + $key = PConfig::get(local_user(), 'ifttt', 'key'); if (!$key) { $key = substr(random_string(),0,20); - PConfig::set(local_user(),'ifttt','key', $key); + PConfig::set(local_user(), 'ifttt', 'key', $key); } $s .= ''; - $s .= '

'. t('IFTTT Mirror').'

'; - $s .= '
'; - $s .= ''; } -function ifttt_settings_post(&$a,&$b) { - - if(x($_POST,'ifttt-submit')) - if (isset($_POST['ifttt-rekey'])) - PConfig::delete(local_user(), 'ifttt', 'key'); +function ifttt_settings_post() +{ + if (x($_POST, 'ifttt-submit') && isset($_POST['ifttt-rekey'])) { + PConfig::delete(local_user(), 'ifttt', 'key'); + } } -function ifttt_post(&$a) { - if ($a->argc != 2) - return; - - $user = $a->argv[1]; - - $r = q("SELECT `uid` FROM `user` WHERE `nickname` = '%s' LIMIT 1", dbesc($user)); - if (!$r) { - logger("User ".$user." not found.", LOGGER_DEBUG); +function ifttt_post(App $a) +{ + if ($a->argc != 2) { return; } - $uid = $r[0]["uid"]; + $nickname = $a->argv[1]; - logger("Received a post for user ".$uid." from ifttt ".print_r($_REQUEST, true), LOGGER_DEBUG); - - if (!isset($_REQUEST["key"])) { - logger("No key found."); + $user = dba::select('user', ['uid'], ['nickname' => $nickname], ['limit' => 1]); + if (!DBM::is_result($user)) { + logger('User ' . $nickname . ' not found.', LOGGER_DEBUG); return; } - $key = $_REQUEST["key"]; + $uid = $user['uid']; + + logger('Received a post for user ' . $uid . ' from ifttt ' . print_r($_REQUEST, true), LOGGER_DEBUG); + + if (!isset($_REQUEST['key'])) { + logger('No key found.'); + return; + } + + $key = $_REQUEST['key']; // Check the key - if ($key != PConfig::get($uid,'ifttt','key')) { - logger("Invalid key for user ".$uid, LOGGER_DEBUG); + if ($key != PConfig::get($uid, 'ifttt', 'key')) { + logger('Invalid key for user ' . $uid, LOGGER_DEBUG); return; } $item = array(); - if (isset($_REQUEST["type"])) - $item["type"] = $_REQUEST["type"]; + if (isset($_REQUEST['type'])) { + $item['type'] = $_REQUEST['type']; + } - if (!in_array($item["type"], array("status", "link", "photo"))) { - logger("Unknown item type ".$item["type"], LOGGER_DEBUG); + if (!in_array($item['type'], array('status', 'link', 'photo'))) { + logger('Unknown item type ' . $item['type'], LOGGER_DEBUG); return; } - if (isset($_REQUEST["link"])) - $item["link"] = trim($_REQUEST["link"]); - if (isset($_REQUEST["image"])) - $item["image"] = trim($_REQUEST["image"]); - if (isset($_REQUEST["title"])) - $item["title"] = trim($_REQUEST["title"]); - if (isset($_REQUEST["msg"])) - $item["msg"] = trim($_REQUEST["msg"]); - if (isset($_REQUEST["description"])) - $item["description"] = trim($_REQUEST["description"]); - if (isset($_REQUEST["date"])) - $item["date"] = date("c", strtotime($date = str_replace(" at ", ", ", $_REQUEST["date"]))); - if (isset($_REQUEST["url"])) - $item["url"] = trim($_REQUEST["url"]); + if (isset($_REQUEST['link'])) { + $item['link'] = trim($_REQUEST['link']); + } + if (isset($_REQUEST['image'])) { + $item['image'] = trim($_REQUEST['image']); + } + if (isset($_REQUEST['title'])) { + $item['title'] = trim($_REQUEST['title']); + } + if (isset($_REQUEST['msg'])) { + $item['msg'] = trim($_REQUEST['msg']); + } + if (isset($_REQUEST['description'])) { + $item['description'] = trim($_REQUEST['description']); + } + if (isset($_REQUEST['date'])) { + $item['date'] = date('c', strtotime($date = str_replace(' at ', ', ', $_REQUEST['date']))); + } + if (isset($_REQUEST['url'])) { + $item['url'] = trim($_REQUEST['url']); + } - if ((substr($item["msg"], 0, 3) == "<<<") && (substr($item["msg"], -3, 3) == ">>>")) - $item["msg"] = substr($item["msg"], 3, -3); + if ((substr($item['msg'], 0, 3) == '<<<') && (substr($item['msg'], -3, 3) == '>>>')) { + $item['msg'] = substr($item['msg'], 3, -3); + } ifttt_message($uid, $item); } -function ifttt_message($uid, $item) { - +function ifttt_message($uid, $item) +{ $a = get_app(); - $_SESSION["authenticated"] = true; - $_SESSION["uid"] = $uid; + $_SESSION['authenticated'] = true; + $_SESSION['uid'] = $uid; unset($_REQUEST); - $_REQUEST["type"] = "wall"; - $_REQUEST["api_source"] = true; - $_REQUEST["profile_uid"] = $uid; - $_REQUEST["source"] = "IFTTT"; - $_REQUEST["title"] = ""; - $_REQUEST["body"] = $item["msg"]; - //$_REQUEST["date"] = $item["date"]; - //$_REQUEST["uri"] = $item["url"]; + $_REQUEST['type'] = 'wall'; + $_REQUEST['api_source'] = true; + $_REQUEST['profile_uid'] = $uid; + $_REQUEST['source'] = 'IFTTT'; + $_REQUEST['title'] = ''; + $_REQUEST['body'] = $item['msg']; + //$_REQUEST['date'] = $item['date']; + //$_REQUEST['uri'] = $item['url']; - if (strstr($item["url"], "facebook.com")) { - $hash = hash("ripemd128", item["url"]); - $_REQUEST["extid"] = NETWORK_FACEBOOK; - $_REQUEST['message_id'] = item_new_uri($a->get_hostname(), $uid, NETWORK_FACEBOOK.":".$hash); + if (strstr($item['url'], 'facebook.com')) { + $hash = hash('ripemd128', item['url']); + $_REQUEST['extid'] = NETWORK_FACEBOOK; + $_REQUEST['message_id'] = item_new_uri($a->get_hostname(), $uid, NETWORK_FACEBOOK . ':' . $hash); } - if ($item["type"] == "link") { - $data = query_page_info($item["link"]); + if ($item['type'] == 'link') { + $data = query_page_info($item['link']); - if (isset($item["title"]) && (trim($item["title"]) != "")) - $data["title"] = $item["title"]; + if (isset($item['title']) && (trim($item['title']) != '')) { + $data['title'] = $item['title']; + } - if (isset($item["description"]) && (trim($item["description"]) != "")) - $data["text"] = $item["description"]; + if (isset($item['description']) && (trim($item['description']) != '')) { + $data['text'] = $item['description']; + } - $_REQUEST["body"] .= add_page_info_data($data); - } elseif (($item["type"] == "photo") && ($item["image"] != "")) - $_REQUEST["body"] .= "\n\n[img]".$item["image"]."[/img]\n"; + $_REQUEST['body'] .= add_page_info_data($data); + } elseif (($item['type'] == 'photo') && ($item['image'] != '')) { + $_REQUEST['body'] .= "\n\n[img]" . $item['image'] . "[/img]\n"; + } - //print_r($_REQUEST); item_post($a); } diff --git a/xmpp/xmpp.php b/xmpp/xmpp.php index 473b9b8b..867a3c06 100644 --- a/xmpp/xmpp.php +++ b/xmpp/xmpp.php @@ -1,42 +1,48 @@ */ - +use Friendica\App; use Friendica\Core\Config; use Friendica\Core\PConfig; -function xmpp_install() { +function xmpp_install() +{ register_hook('plugin_settings', 'addon/xmpp/xmpp.php', 'xmpp_plugin_settings'); register_hook('plugin_settings_post', 'addon/xmpp/xmpp.php', 'xmpp_plugin_settings_post'); register_hook('page_end', 'addon/xmpp/xmpp.php', 'xmpp_script'); register_hook('logged_in', 'addon/xmpp/xmpp.php', 'xmpp_login'); } -function xmpp_uninstall() { +function xmpp_uninstall() +{ unregister_hook('plugin_settings', 'addon/xmpp/xmpp.php', 'xmpp_plugin_settings'); unregister_hook('plugin_settings_post', 'addon/xmpp/xmpp.php', 'xmpp_plugin_settings_post'); unregister_hook('page_end', 'addon/xmpp/xmpp.php', 'xmpp_script'); unregister_hook('logged_in', 'addon/xmpp/xmpp.php', 'xmpp_login'); } -function xmpp_plugin_settings_post($a,$post) { - if(! local_user() || (! x($_POST,'xmpp-settings-submit'))) +function xmpp_plugin_settings_post() +{ + if (!local_user() || (!x($_POST, 'xmpp-settings-submit'))) { return; - PConfig::set(local_user(),'xmpp','enabled',intval($_POST['xmpp_enabled'])); - PConfig::set(local_user(),'xmpp','individual',intval($_POST['xmpp_individual'])); - PConfig::set(local_user(),'xmpp','bosh_proxy',$_POST['xmpp_bosh_proxy']); + } + PConfig::set(local_user(), 'xmpp', 'enabled', intval($_POST['xmpp_enabled'])); + PConfig::set(local_user(), 'xmpp', 'individual', intval($_POST['xmpp_individual'])); + PConfig::set(local_user(), 'xmpp', 'bosh_proxy', $_POST['xmpp_bosh_proxy']); - info( t('XMPP settings updated.') . EOL); + info(t('XMPP settings updated.') . EOL); } -function xmpp_plugin_settings(&$a,&$s) { - - if(! local_user()) +function xmpp_plugin_settings(App $a, &$s) +{ + if (!local_user()) { return; + } /* Add our stylesheet to the xmpp so we can make our settings look nice */ @@ -44,13 +50,13 @@ function xmpp_plugin_settings(&$a,&$s) { /* Get the current state of our config variable */ - $enabled = intval(PConfig::get(local_user(),'xmpp','enabled')); + $enabled = intval(PConfig::get(local_user(), 'xmpp', 'enabled')); $enabled_checked = (($enabled) ? ' checked="checked" ' : ''); - $individual = intval(PConfig::get(local_user(),'xmpp','individual')); + $individual = intval(PConfig::get(local_user(), 'xmpp', 'individual')); $individual_checked = (($individual) ? ' checked="checked" ' : ''); - $bosh_proxy = PConfig::get(local_user(),"xmpp","bosh_proxy"); + $bosh_proxy = PConfig::get(local_user(), "xmpp", "bosh_proxy"); /* Add some HTML to the existing form */ $s .= ''; @@ -72,9 +78,9 @@ function xmpp_plugin_settings(&$a,&$s) { $s .= '
'; } - if (!Config::get("xmpp", "central_userbase") || PConfig::get(local_user(),"xmpp","individual")) { - $s .= ''; - $s .= ' '; + if (!Config::get("xmpp", "central_userbase") || PConfig::get(local_user(), "xmpp", "individual")) { + $s .= ''; + $s .= ' '; $s .= '
'; } @@ -83,68 +89,77 @@ function xmpp_plugin_settings(&$a,&$s) { /* provide a submit button */ $s .= '
'; - } -function xmpp_login($a,$b) { +function xmpp_login() +{ if (!$_SESSION["allow_api"]) { $password = substr(random_string(),0,16); PConfig::set(local_user(), "xmpp", "password", $password); } } -function xmpp_plugin_admin(&$a, &$o){ +function xmpp_plugin_admin(App $a, &$o) +{ $t = get_markup_template("admin.tpl", "addon/xmpp/"); $o = replace_macros($t, array( '$submit' => t('Save Settings'), - '$bosh_proxy' => array('bosh_proxy', t('Jabber BOSH host'), Config::get('xmpp', 'bosh_proxy'), ''), + '$bosh_proxy' => array('bosh_proxy', t('Jabber BOSH host'), Config::get('xmpp', 'bosh_proxy'), ''), '$central_userbase' => array('central_userbase', t('Use central userbase'), Config::get('xmpp', 'central_userbase'), t('If enabled, users will automatically login to an ejabberd server that has to be installed on this machine with synchronized credentials via the "auth_ejabberd.php" script.')), )); } -function xmpp_plugin_admin_post(&$a){ - $bosh_proxy = ((x($_POST,'bosh_proxy')) ? trim($_POST['bosh_proxy']) : ''); - $central_userbase = ((x($_POST,'central_userbase')) ? intval($_POST['central_userbase']) : false); - Config::set('xmpp','bosh_proxy',$bosh_proxy); - Config::set('xmpp','central_userbase',$central_userbase); - info( t('Settings updated.'). EOL ); +function xmpp_plugin_admin_post() +{ + $bosh_proxy = ((x($_POST, 'bosh_proxy')) ? trim($_POST['bosh_proxy']) : ''); + $central_userbase = ((x($_POST, 'central_userbase')) ? intval($_POST['central_userbase']) : false); + Config::set('xmpp', 'bosh_proxy', $bosh_proxy); + Config::set('xmpp', 'central_userbase', $central_userbase); + info(t('Settings updated.') . EOL); } -function xmpp_script(&$a,&$s) { - xmpp_converse($a,$s); +function xmpp_script(App $a) +{ + xmpp_converse($a); } -function xmpp_converse(&$a,&$s) { - if (!local_user()) +function xmpp_converse(App $a) +{ + if (!local_user()) { return; + } - if ($_GET["mode"] == "minimal") + if ($_GET["mode"] == "minimal") { return; + } - if ($a->is_mobile || $a->is_tablet) + if ($a->is_mobile || $a->is_tablet) { return; + } - if (!PConfig::get(local_user(),"xmpp","enabled")) + if (!PConfig::get(local_user(), "xmpp", "enabled")) { return; + } - if (in_array($a->query_string, array("admin/federation/"))) + if (in_array($a->query_string, array("admin/federation/"))) { return; + } - $a->page['htmlhead'] .= ''."\n"; - $a->page['htmlhead'] .= ''."\n"; + $a->page['htmlhead'] .= '' . "\n"; + $a->page['htmlhead'] .= '' . "\n"; - if (Config::get("xmpp", "central_userbase") && !PConfig::get(local_user(),"xmpp","individual")) { + if (Config::get("xmpp", "central_userbase") && !PConfig::get(local_user(), "xmpp", "individual")) { $bosh_proxy = Config::get("xmpp", "bosh_proxy"); $password = PConfig::get(local_user(), "xmpp", "password"); if ($password == "") { - $password = substr(random_string(),0,16); + $password = random_string(16); PConfig::set(local_user(), "xmpp", "password", $password); } - $jid = $a->user["nickname"]."@".$a->get_hostname()."/converse-".substr(random_string(),0,5);; + $jid = $a->user["nickname"] . "@" . $a->get_hostname() . "/converse-" . random_string(5); $auto_login = "auto_login: true, authentication: 'login', @@ -157,13 +172,15 @@ function xmpp_converse(&$a,&$s) { $auto_login = ""; } - if ($bosh_proxy == "") + if ($bosh_proxy == "") { return; + } - if (in_array($a->argv[0], array("manage", "logout"))) + if (in_array($a->argv[0], array("manage", "logout"))) { $additional_commands = "converse.user.logout();\n"; - else + } else { $additional_commands = ""; + } $on_ready = ""; @@ -200,4 +217,3 @@ function xmpp_converse(&$a,&$s) { }); "; } -?> From 7f900007c75a4c2da6dfd8e2aa4a8dc29a23e3a6 Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sat, 25 Nov 2017 21:31:10 -0500 Subject: [PATCH 2/2] Simplify random_string() calls --- ifttt/ifttt.php | 2 +- xmpp/xmpp.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ifttt/ifttt.php b/ifttt/ifttt.php index 800294ff..384b955c 100644 --- a/ifttt/ifttt.php +++ b/ifttt/ifttt.php @@ -45,7 +45,7 @@ function ifttt_settings(App $a, &$s) $key = PConfig::get(local_user(), 'ifttt', 'key'); if (!$key) { - $key = substr(random_string(),0,20); + $key = random_string(20); PConfig::set(local_user(), 'ifttt', 'key', $key); } diff --git a/xmpp/xmpp.php b/xmpp/xmpp.php index 867a3c06..e523c65c 100644 --- a/xmpp/xmpp.php +++ b/xmpp/xmpp.php @@ -94,7 +94,7 @@ function xmpp_plugin_settings(App $a, &$s) function xmpp_login() { if (!$_SESSION["allow_api"]) { - $password = substr(random_string(),0,16); + $password = random_string(16); PConfig::set(local_user(), "xmpp", "password", $password); } }