]> git.mxchange.org Git - friendica-addons.git/blobdiff - discourse/discourse.php
fix $_POST variable
[friendica-addons.git] / discourse / discourse.php
index 3b0f7cf407f17d05d99c8e20d683dfeb147d2ae5..f52edb9fb74597294cc1aa9127aead42e850e6ba 100644 (file)
@@ -7,59 +7,80 @@
  * Author: Michael Vogel <http://pirati.ca/profile/heluecht>
  *
  */
-//use DOMDocument;
-//use DOMXPath;
+
 use Friendica\App;
+use Friendica\Content\Text\Markdown;
 use Friendica\Core\Hook;
-use Friendica\Core\L10n;
 use Friendica\Core\Logger;
-use Friendica\Core\PConfig;
 use Friendica\Core\Protocol;
+use Friendica\Core\Renderer;
 use Friendica\Database\DBA;
+use Friendica\DI;
 use Friendica\Model\Contact;
-use Friendica\Util\XML;
-use Friendica\Content\Text\Markdown;
-use Friendica\Util\Network;
+use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Strings;
-Use Friendica\Util\DateTimeFormat;
+
+/* Todo:
+ * - Obtaining API tokens to be able to read non public posts as well
+ * - Handling duplicates (possibly using some non visible marker)
+ * - Fetching missing posts
+ * - Fetch topic information
+ * - Support mail free mode when write tokens are available
+ * - Fix incomplete (relative) links (hosts are missing)
+*/
 
 function discourse_install()
 {
-       Hook::register('email_getmessage',    __FILE__, 'discourse_email_getmessage');
-       Hook::register('addon_settings',      __FILE__, 'discourse_settings');
-       Hook::register('addon_settings_post', __FILE__, 'discourse_settings_post');
+       Hook::register('email_getmessage',        __FILE__, 'discourse_email_getmessage');
+       Hook::register('connector_settings',      __FILE__, 'discourse_settings');
+       Hook::register('connector_settings_post', __FILE__, 'discourse_settings_post');
 }
 
-function discourse_uninstall()
+function discourse_settings(App $a, &$s)
 {
-       Hook::unregister('email_getmessage',        __FILE__, 'discourse_email_getmessage');
-       Hook::unregister('connector_settings',      __FILE__, 'discourse_settings');
-       Hook::unregister('connector_settings_post', __FILE__, 'discourse_settings_post');
-}
+       if (!local_user()) {
+               return;
+       }
 
-function discourse_addon_settings(App $a, &$s)
-{
+       $enabled = intval(DI::pConfig()->get(local_user(), 'discourse', 'enabled'));
+
+       $t = Renderer::getMarkupTemplate('settings.tpl', 'addon/discourse/');
+       $s .= Renderer::replaceMacros($t, [
+               '$title'   => DI::l10n()->t('Discourse'),
+               '$enabled' => ['enabled', DI::l10n()->t('Enable processing of Discourse mailing list mails'), $enabled, DI::l10n()->t('If enabled, incoming mails from Discourse will be improved so they look much better. To make it work, you have to configure the e-mail settings in Friendica. You also have to enable the mailing list mode in Discourse. Then you have to add the Discourse mail account as contact.')],
+               '$submit'  => DI::l10n()->t('Save Settings'),
+       ]);
 }
 
-function discourse_addon_settings_post(App $a)
+function discourse_settings_post(App $a)
 {
+       if (!local_user() || empty($_POST['discourse-submit'])) {
+                return;
+        }
+
+       DI::pConfig()->set(local_user(), 'discourse', 'enabled', intval($_POST['enabled']));
 }
 
 function discourse_email_getmessage(App $a, &$message)
 {
-//     Logger::info('Got raw message', $message);
+       if (empty($message['item']['uid'])) {
+               return;
+       }
+
+       if (!DI::pConfig()->get($message['item']['uid'], 'discourse', 'enabled')) {
+               return;
+       }
 
        // We do assume that all Discourse servers are running with SSL
        if (preg_match('=topic/(.*\d)/(.*\d)@(.*)=', $message['item']['uri'], $matches) &&
                discourse_fetch_post_from_api($message, $matches[2], $matches[3])) {
-               Logger::info('Fetched comment via API', ['host' => $matches[3], 'topic' => $matches[1], 'post' => $matches[2]]);
+               Logger::info('Fetched comment via API (message-id mode)', ['host' => $matches[3], 'topic' => $matches[1], 'post' => $matches[2]]);
                return;
        }
 
        if (preg_match('=topic/(.*\d)@(.*)=', $message['item']['uri'], $matches) &&
                discourse_fetch_topic_from_api($message, 'https://' . $matches[2], $matches[1], 1)) {
-               discourse_fetch_post_from_api($message, $matches[2], $matches[3]);
-               Logger::info('Fetched starting post via API', ['host' => $matches[2], 'topic' => $matches[1]]);
+               Logger::info('Fetched starting post via API (message-id mode)', ['host' => $matches[2], 'topic' => $matches[1]]);
                return;
        }
 
@@ -70,21 +91,22 @@ function discourse_email_getmessage(App $a, &$message)
 
        if (empty($message['item']['plink']) || !preg_match('=(http.*)/t/.*/(.*\d)/(.*\d)=', $message['item']['plink'], $matches)) {
                Logger::info('This is no Discourse post');
+               return;
        }
 
        if (discourse_fetch_topic_from_api($message, $matches[1], $matches[2], $matches[3])) {
-               Logger::info('Fetched post from via API', ['host' => $matches[1], 'topic' => $matches[2], 'id' => $matches[3]]);
+               Logger::info('Fetched post via API (plink mode)', ['host' => $matches[1], 'topic' => $matches[2], 'id' => $matches[3]]);
                return;
        }
 
-       Logger::info('Fallback mode');
+       Logger::info('Fallback mode', ['plink' => $message['item']['plink']]);
        // Search in the HTML part for the discourse entry and the author profile
        if (!empty($message['html'])) {
                $message = discourse_get_html($message);
        }
 
        // Remove the title on comments, they don't serve any purpose there
-       if ($message['item']['parent-uri'] != $message['item']['uri']) {
+       if ($message['item']['thr-parent'] != $message['item']['uri']) {
                unset($message['item']['title']);
        }
 }
@@ -92,7 +114,7 @@ function discourse_email_getmessage(App $a, &$message)
 function discourse_fetch_post($host, $topic, $pid)
 {
        $url = $host . '/t/' . $topic . '/' . $pid . '.json';
-       $curlResult = Network::curl($url);
+       $curlResult = DI::httpRequest()->get($url);
        if (!$curlResult->isSuccess()) {
                Logger::info('No success', ['url' => $url]);
                return false;
@@ -129,7 +151,7 @@ function discourse_fetch_post_from_api(&$message, $post, $host)
 {
        $hostaddr = 'https://' . $host;
        $url = $hostaddr . '/posts/' . $post . '.json';
-       $curlResult = Network::curl($url);
+       $curlResult = DI::httpRequest()->get($url);
        if (!$curlResult->isSuccess()) {
                return false;
        }
@@ -176,12 +198,12 @@ function discourse_get_user($post, $hostaddr)
        $contact['nurl'] = Strings::normaliseLink($contact['url']);
        $contact['baseurl'] = $hostaddr;
        Logger::info('Contact', $contact);
-       $contact['id'] = Contact::getIdForURL($contact['url'], 0, true, $contact);
+       $contact['id'] = Contact::getIdForURL($contact['url'], 0, false, $contact);
         if (!empty($contact['id'])) {
                $avatar = $contact['photo'];
                unset($contact['photo']);
                DBA::update('contact', $contact, ['id' => $contact['id']]);
-               Contact::updateAvatar($avatar, 0, $contact['id']);
+               Contact::updateAvatar($contact['id'], $avatar);
                $contact['photo'] = $avatar;
        }
 
@@ -204,6 +226,12 @@ function discourse_process_post($message, $post, $hostaddr)
 
        if ($post['post_number'] == 1) {
                $message['item']['parent-uri'] = $message['item']['uri'] = 'topic/' . $post['topic_id'] . '@' . $host;
+
+               // Remove the Discourse forum name from the subject
+               $pattern = '=\[.*\].*\s(\[.*\].*)=';
+               if (preg_match($pattern, $message['item']['title'])) {
+                       $message['item']['title'] = preg_replace($pattern, '$1', $message['item']['title']);
+               }
                /// @ToDo Fetch thread information
        } else {
                $message['item']['uri'] = 'topic/' . $post['topic_id'] . '/' . $post['id'] . '@' . $host;
@@ -240,7 +268,7 @@ function discourse_get_html($message)
        $profile = discourse_get_profile($xpath);
        if (!empty($profile['url'])) {
                Logger::info('Found profile', $profile);
-               $message['item']['author-id'] = Contact::getIdForURL($profile['url'], 0, true, $profile);
+               $message['item']['author-id'] = Contact::getIdForURL($profile['url'], 0, false, $profile);
                $message['item']['author-link'] = $profile['url'];
                $message['item']['author-name'] = $profile['name'];
                $message['item']['author-avatar'] = $profile['photo'];