fix undeclared variables or wrong variable names (/mod)

diff --git a/mod/dfrn_request.php b/mod/dfrn_request.php
index 34cc9cf..e8bc509 100644 (file)
--- a/mod/dfrn_request.php
+++ b/mod/dfrn_request.php
@@ -168,7 +168,7 @@ function dfrn_request_post(App $a)
                                        $parms['key'] // this was already escaped
                                );
                                if (DBM::is_result($r)) {
-                                       Group::addMember(User::getDefaultGroup($uid, $r[0]["network"]), $r[0]['id']);
+                                       Group::addMember(User::getDefaultGroup(local_user(), $r[0]["network"]), $r[0]['id']);
 
                                        if (isset($photo)) {
                                                Contact::updateAvatar($photo, local_user(), $r[0]["id"], true);

diff --git a/mod/editpost.php b/mod/editpost.php
index af8f4e3..f28e7a5 100644 (file)
--- a/mod/editpost.php
+++ b/mod/editpost.php
@@ -38,6 +38,8 @@ function editpost_content(App $a) {
                return;
        }
 
+       $geotag = '';
+
        $o .= replace_macros(get_markup_template("section_title.tpl"),[
                '$title' => L10n::t('Edit post')
        ]);

diff --git a/mod/install.php b/mod/install.php
index f3969a6..3adcb75 100644 (file)
--- a/mod/install.php
+++ b/mod/install.php
@@ -222,6 +222,7 @@ function install_content(App $a) {
                        $dbdata = notags(trim($_POST['dbdata']));
                        $phpath = notags(trim($_POST['phpath']));
 
+                       $adminmail = notags(trim($_POST['adminmail']));
 
                        $tpl = get_markup_template('install_db.tpl');
                        $o .= replace_macros($tpl, [

diff --git a/mod/localtime.php b/mod/localtime.php
index b10f157..9a754ac 100644 (file)
--- a/mod/localtime.php
+++ b/mod/localtime.php
@@ -30,7 +30,7 @@ function localtime_content(App $a)
                $t = 'now';
        }
 
-       $o .= '<h3>' . L10n::t('Time Conversion') . '</h3>';
+       $o  = '<h3>' . L10n::t('Time Conversion') . '</h3>';
 
        $o .= '<p>' . L10n::t('Friendica provides this service for sharing events with other networks and friends in unknown timezones.') . '</p>';
 

diff --git a/mod/lostpass.php b/mod/lostpass.php
index 73167e6..56ad7e3 100644 (file)
--- a/mod/lostpass.php
+++ b/mod/lostpass.php
@@ -125,6 +125,7 @@ function lostpass_form()
 function lostpass_generate_password($user)
 {
        $o = '';
+       $a = get_app();
 
        $new_password = User::generateNewPassword();
        $result = User::updatePassword($user['uid'], $new_password);

diff --git a/mod/message.php b/mod/message.php
index f1e3b76..8940ee4 100644 (file)
--- a/mod/message.php
+++ b/mod/message.php
@@ -250,7 +250,6 @@ function message_content(App $a)
                        '$to' => L10n::t('To:'),
                        '$showinputs' => 'true',
                        '$prefill' => $prefill,
-                       '$autocomp' => $autocomp,
                        '$preid' => $preid,
                        '$subject' => L10n::t('Subject:'),
                        '$subjtxt' => x($_REQUEST, 'subject') ? strip_tags($_REQUEST['subject']) : '',

diff --git a/mod/network.php b/mod/network.php
index 834eedf..3013e94 100644 (file)
--- a/mod/network.php
+++ b/mod/network.php
@@ -444,7 +444,7 @@ function networkFlatView(App $a, $update = 0)
        }
 
        if (strlen($file)) {
-               $sql_post_table .= sprintf("INNER JOIN (SELECT `oid` FROM `term` WHERE `term` = '%s' AND `otype` = %d AND `type` = %d AND `uid` = %d ORDER BY `tid` DESC) AS `term` ON `item`.`id` = `term`.`oid` ",
+               $sql_post_table = sprintf("INNER JOIN (SELECT `oid` FROM `term` WHERE `term` = '%s' AND `otype` = %d AND `type` = %d AND `uid` = %d ORDER BY `tid` DESC) AS `term` ON `item`.`id` = `term`.`oid` ",
                        dbesc(protect_sprintf($file)), intval(TERM_OBJ_POST), intval(TERM_FILE), intval(local_user()));
        } else {
                $sql_post_table = " INNER JOIN `thread` ON `thread`.`iid` = `item`.`parent`";

diff --git a/mod/notify.php b/mod/notify.php
index 0ab2b0d..dff7d14 100644 (file)
--- a/mod/notify.php
+++ b/mod/notify.php
@@ -77,7 +77,7 @@ function notify_content(App $a)
                $notif_content .= L10n::t('No more system notifications.');
        }
 
-       $o .= replace_macros($notif_tpl, [
+       $o = replace_macros($notif_tpl, [
                '$notif_header' => L10n::t('System Notifications'),
                '$tabs' => false, // $tabs,
                '$notif_content' => $notif_content,

diff --git a/mod/photos.php b/mod/photos.php
index 9515d17..4b031e5 100644 (file)
--- a/mod/photos.php
+++ b/mod/photos.php
@@ -865,7 +865,7 @@ function photos_post(App $a)
 
        // Create item container
        $lat = $lon = null;
-       if ($exif && $exif['GPS'] && Feature::isEnabled($channel_id, 'photo_location')) {
+       if ($exif && $exif['GPS'] && Feature::isEnabled($page_owner_uid, 'photo_location')) {
                $lat = Photo::getGps($exif['GPS']['GPSLatitude'], $exif['GPS']['GPSLatitudeRef']);
                $lon = Photo::getGps($exif['GPS']['GPSLongitude'], $exif['GPS']['GPSLongitudeRef']);
        }

diff --git a/mod/ping.php b/mod/ping.php
index 1bb8f14..62ab264 100644 (file)
--- a/mod/ping.php
+++ b/mod/ping.php
@@ -571,8 +571,8 @@ function ping_format_xml_data($data, $sysnotify, $notifs, $sysmsgs, $sysmsgs_inf
        $forums = [];
        if (count($forums_unseen)) {
                foreach ($forums_unseen as $key => $item) {
-                       $forums[$count . ':forum'] = $item['count'];
-                       $forums[$count . ':@attributes'] = ['id' => $item['id']];
+                       $forums[$key . ':forum'] = $item['count'];
+                       $forums[$key . ':@attributes'] = ['id' => $item['id']];
                }
                $data['forums'] = $forums;
        }

diff --git a/mod/probe.php b/mod/probe.php
index b4f6ff3..5e7a525 100644 (file)
--- a/mod/probe.php
+++ b/mod/probe.php
@@ -15,7 +15,7 @@ function probe_content(App $a)
                killme();
        }
 
-       $o .= '<h3>Probe Diagnostic</h3>';
+       $o  = '<h3>Probe Diagnostic</h3>';
 
        $o .= '<form action="probe" method="get">';
        $o .= 'Lookup address: <input type="text" style="width: 250px;" name="addr" value="' . $_GET['addr'] . '" />';

diff --git a/mod/profile_photo.php b/mod/profile_photo.php
index 370e19c..5556553 100644 (file)
--- a/mod/profile_photo.php
+++ b/mod/profile_photo.php
@@ -257,7 +257,7 @@ function profile_photo_content(App $a) {
 
                $tpl = get_markup_template('profile_photo.tpl');
 
-               $o .= replace_macros($tpl,[
+               $o = replace_macros($tpl,[
                        '$user' => $a->user['nickname'],
                        '$lbl_upfile' => L10n::t('Upload File:'),
                        '$lbl_profiles' => L10n::t('Select a profile:'),
@@ -272,9 +272,8 @@ function profile_photo_content(App $a) {
        }
        else {
                $filename = $a->config['imagecrop'] . '-' . $a->config['imagecrop_resolution'] . '.'.$a->config['imagecrop_ext'];
-               $resolution = $a->config['imagecrop_resolution'];
                $tpl = get_markup_template("cropbody.tpl");
-               $o .= replace_macros($tpl,[
+               $o = replace_macros($tpl,[
                        '$filename' => $filename,
                        '$profile' => intval($_REQUEST['profile']),
                        '$resource' => $a->config['imagecrop'] . '-' . $a->config['imagecrop_resolution'],
@@ -313,6 +312,7 @@ function profile_photo_crop_ui_head(App $a, Image $Image) {
 
 
        $smallest = 0;
+       $filename = '';
 
        $r = Photo::store($Image, local_user(), 0, $hash, $filename, L10n::t('Profile Photos'), 0);
 

diff --git a/mod/profperm.php b/mod/profperm.php
index 3486a1d..0f864d4 100644 (file)
--- a/mod/profperm.php
+++ b/mod/profperm.php
@@ -35,6 +35,8 @@ function profperm_content(App $a) {
                return;
        }
 
+       $o = '';
+
        // Switch to text mod interface if we have more than 'n' contacts or group members
 
        $switchtotext = PConfig::get(local_user(),'system','groupedit_image_limit');
@@ -165,4 +167,3 @@ function profperm_content(App $a) {
        return $o;
 
 }
-

diff --git a/mod/register.php b/mod/register.php
index bf87c92..4a59277 100644 (file)
--- a/mod/register.php
+++ b/mod/register.php
@@ -58,6 +58,7 @@ function register_post(App $a)
                        break;
        }
 
+       $netpublish = (((x($_POST, 'profile_publish_reg')) &intval($_POST['profile_publish_reg']) = 1) ? 1 : 0);
 
        $arr = $_POST;
 

diff --git a/mod/regmod.php b/mod/regmod.php
index 89e348d..aa64986 100644 (file)
--- a/mod/regmod.php
+++ b/mod/regmod.php
@@ -55,7 +55,7 @@ function user_allow($hash)
 
        L10n::pushLang($register[0]['language']);
 
-       User::sendRegisterOpenEmail(
+       $res = User::sendRegisterOpenEmail(
                $user[0]['email'],
                $a->config['sitename'],
                System::baseUrl(),
@@ -100,7 +100,7 @@ function regmod_content(App $a)
 
        if (!local_user()) {
                info(L10n::t('Please login.') . EOL);
-               $o .= '<br /><br />' . Login::form($a->query_string, $a->config['register_policy'] == REGISTER_CLOSED ? 0 : 1);
+               $o = '<br /><br />' . Login::form($a->query_string, $a->config['register_policy'] == REGISTER_CLOSED ? 0 : 1);
                return $o;
        }
 

diff --git a/mod/removeme.php b/mod/removeme.php
index 27d501d..0363bf9 100644 (file)
--- a/mod/removeme.php
+++ b/mod/removeme.php
@@ -49,7 +49,7 @@ function removeme_content(App $a)
        $_SESSION['remove_account_verify'] = $hash;
 
        $tpl = get_markup_template('removeme.tpl');
-       $o .= replace_macros($tpl, [
+       $o = replace_macros($tpl, [
                '$basedir' => System::baseUrl(),
                '$hash' => $hash,
                '$title' => L10n::t('Remove My Account'),

diff --git a/mod/search.php b/mod/search.php
index f1f10ad..afe9412 100644 (file)
--- a/mod/search.php
+++ b/mod/search.php
@@ -19,6 +19,7 @@ require_once 'mod/dirfind.php';
 function search_saved_searches() {
 
        $o = '';
+       $search = ((x($_GET,'search')) ? notags(trim(rawurldecode($_GET['search']))) : '');
 
        if (! Feature::isEnabled(local_user(),'savedsearch'))
                return $o;
@@ -150,7 +151,7 @@ function search_content(App $a) {
        }
 
        // contruct a wrapper for the search header
-       $o .= replace_macros(get_markup_template("content_wrapper.tpl"),[
+       $o = replace_macros(get_markup_template("content_wrapper.tpl"),[
                'name' => "search-header",
                '$title' => L10n::t("Search"),
                '$title_size' => 3,

diff --git a/mod/tagger.php b/mod/tagger.php
index 6016d44..0ee8d88 100644 (file)
--- a/mod/tagger.php
+++ b/mod/tagger.php
@@ -72,7 +72,7 @@ function tagger_content(App $a) {
        $targettype = (($item['resource-id']) ? ACTIVITY_OBJ_IMAGE : ACTIVITY_OBJ_NOTE );
 
        $link = xmlify('<link rel="alternate" type="text/html" href="'
-               . System::baseUrl() . '/display/' . $owner['nickname'] . '/' . $item['id'] . '" />' . "\n") ;
+               . System::baseUrl() . '/display/' . $r['nickname'] . '/' . $item['id'] . '" />' . "\n") ;
 
        $body = xmlify($item['body']);
 

diff --git a/mod/viewcontacts.php b/mod/viewcontacts.php
index b45980f..2058567 100644 (file)
--- a/mod/viewcontacts.php
+++ b/mod/viewcontacts.php
@@ -46,6 +46,8 @@ function viewcontacts_content(App $a)
                return;
        }
 
+       $is_owner = ((local_user() && ($a->profile['profile_uid'] == local_user())) ? true : false);
+
        $o = "";
 
        // tabs
@@ -97,9 +99,6 @@ function viewcontacts_content(App $a)
                $url = $rr['url'];
 
                // route DFRN profiles through the redirect
-
-               $is_owner = ((local_user() && ($a->profile['profile_uid'] == local_user())) ? true : false);
-
                if ($is_owner && ($rr['network'] === NETWORK_DFRN) && ($rr['rel'])) {
                        $url = 'redir/' . $rr['id'];
                } else {

diff --git a/mod/viewsrc.php b/mod/viewsrc.php
index 376e2ef..d2f0d01 100644 (file)
--- a/mod/viewsrc.php
+++ b/mod/viewsrc.php
@@ -13,6 +13,7 @@ function viewsrc_content(App $a) {
                return;
        }
 
+       $o = ''; 
        $item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0);
 
        if(! $item_id) {

diff --git a/mod/wallmessage.php b/mod/wallmessage.php
index 250b7d7..75aca1f 100644 (file)
--- a/mod/wallmessage.php
+++ b/mod/wallmessage.php
@@ -128,7 +128,7 @@ function wallmessage_content(App $a) {
        ]);
 
        $tpl = get_markup_template('wallmessage.tpl');
-       $o .= replace_macros($tpl, [
+       $o = replace_macros($tpl, [
                '$header' => L10n::t('Send Private Message'),
                '$subheader' => L10n::t('If you wish for %s to respond, please check that the privacy settings on your site allow private mail from unknown senders.', $user['username']),
                '$to' => L10n::t('To:'),
@@ -139,7 +139,6 @@ function wallmessage_content(App $a) {
                '$text' => ((x($_REQUEST, 'body')) ? escape_tags(htmlspecialchars($_REQUEST['body'])) : ''),
                '$readonly' => '',
                '$yourmessage' => L10n::t('Your message:'),
-               '$select' => $select,
                '$parent' => '',
                '$upload' => L10n::t('Upload photo'),
                '$insert' => L10n::t('Insert web link'),

diff --git a/mod/webfinger.php b/mod/webfinger.php
index c886793..6f49a8f 100644 (file)
--- a/mod/webfinger.php
+++ b/mod/webfinger.php
@@ -20,7 +20,7 @@ function webfinger_content(App $a)
                killme();
        }
 
-       $o .= '<h3>Webfinger Diagnostic</h3>';
+       $o  = '<h3>Webfinger Diagnostic</h3>';
 
        $o .= '<form action="webfinger" method="get">';
        $o .= 'Lookup address: <input type="text" style="width: 250px;" name="addr" value="' . $_GET['addr'] .'" />';