From a9c731798e2f82c4961240627b41b61e29e3aedd Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Wed, 17 Mar 2010 14:16:43 -0700 Subject: [PATCH] Tweak to OStatus long message cropping: use original source notice URL for the link in the text version, don't shorten the link for the HTML so we can append params to it in JS. --- plugins/OStatus/classes/Ostatus_profile.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/plugins/OStatus/classes/Ostatus_profile.php b/plugins/OStatus/classes/Ostatus_profile.php index 7b18fed9c8..9d2be8abc4 100644 --- a/plugins/OStatus/classes/Ostatus_profile.php +++ b/plugins/OStatus/classes/Ostatus_profile.php @@ -542,8 +542,7 @@ class Ostatus_profile extends Memcached_DataObject } $shortSummary = common_shorten_links($summary); if (Notice::contentTooLong($shortSummary)) { - $url = common_shorten_url(common_local_url('attachment', - array('attachment' => $attachment->id))); + $url = common_shorten_url($sourceUrl); $shortSummary = substr($shortSummary, 0, Notice::maxContent() - (mb_strlen($url) + 2)); @@ -552,10 +551,12 @@ class Ostatus_profile extends Memcached_DataObject // We mark up the attachment link specially for the HTML output // so we can fold-out the full version inline. + $attachUrl = common_local_url('attachment', + array('attachment' => $attachment->id)); $rendered = common_render_text($shortSummary) . ' ' . '' . // TRANS: expansion link for too-long remote messages htmlspecialchars(_m('(more)')) .