Merge branch 'master' of github.com:Elgg/Elgg

documentation/examples/plugins/manifest.xml → documentation/examples/plugins/manifest_options/manifest.xml

@@ -16,7 +16,7 @@
 
 	<requires>
 		<type>elgg_release</type>
-		<version>1.8-svn</version>
+		<version>1.8</version>
 	</requires>
 
 	<screenshot>

documentation/examples/plugins/skeleton/manifest.xml

@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<plugin_manifest xmlns="http://www.elgg.org/plugin_manifest/1.8">
+	<name>My Plugin</name>
+	<author>My Name</author>
+	<version>1.0</version>
+	<description>This is a description of my plugin and its features.</description>
+	<website>http://www.elgg.org/</website>
+	<copyright>(C) My Name or Company 2012</copyright>
+	<license>GNU Public License version 2</license>
+
+	<requires>
+		<type>elgg_release</type>
+		<version>1.8</version>
+	</requires>
+
+	<category>communication</category>
+
+</plugin_manifest>

engine/classes/ElggEntity.php

@@ -1434,10 +1434,11 @@ public function getLocation() {
 	 *
 	 * @param string $location String representation of the location
 	 *
-	 * @return true
+	 * @return bool
 	 */
 	public function setLocation($location) {
-		return $this->location = $location;
+		$this->location = $location;
+		return true;
 	}
 
 	/**
@@ -1446,7 +1447,7 @@ public function setLocation($location) {
 	 * @param float $lat  Latitude
 	 * @param float $long Longitude
 	 *
-	 * @return true
+	 * @return bool
 	 * @todo Unimplemented
 	 */
 	public function setLatLong($lat, $long) {
@@ -1459,20 +1460,20 @@ public function setLatLong($lat, $long) {
 	/**
 	 * Return the entity's latitude.
 	 *
-	 * @return int
+	 * @return float
 	 * @todo Unimplemented
 	 */
 	public function getLatitude() {
-		return $this->get('geo:lat');
+		return (float)$this->get('geo:lat');
 	}
 
 	/**
 	 * Return the entity's longitude
 	 *
-	 * @return Int
+	 * @return float
 	 */
 	public function getLongitude() {
-		return $this->get('geo:long');
+		return (float)$this->get('geo:long');
 	}
 
 	/*

engine/classes/Locatable.php

@@ -13,7 +13,7 @@ interface Locatable {
 	 * @param string $location Textual representation of location
 	 *
 	 * @return bool
-	 **/
+	 */
 	public function setLocation($location);
 
 	/**

languages/en.php

@@ -425,6 +425,7 @@
 	'riveritem:plural:user' => 'some users',
 	'river:ingroup' => 'in the group %s',
 	'river:none' => 'No activity',
+	'river:update' => 'Update for %s',
 
 	'river:widget:title' => "Activity",
 	'river:widget:description' => "Display latest activity",
@@ -1098,6 +1099,7 @@
 	'generic_comment:notdeleted' => "Sorry, we could not delete this comment.",
 	'generic_comment:failure' => "An unexpected error occurred when adding your comment. Please try again.",
 	'generic_comment:none' => 'No comments',
+	'generic_comment:title' => 'Comment by %s',
 
 	'generic_comment:email:subject' => 'You have a new comment!',
 	'generic_comment:email:body' => "You have a new comment on your item \"%s\" from %s. It reads:

mod/oauth_api/vendors/oauth/library/OAuthRequestLogger.php

@@ -190,7 +190,7 @@ static function flush ()
 			// Log the request
 			if (OAuthRequestLogger::$store_log)
 			{
-				$store = elggconnect_get_oauth_store();//OAuthStore::instance();
+				$store = OAuthStore::instance();
 				$store->addLog($keys, $received, $sent, $base_string, OAuthRequestLogger::$note, OAuthRequestLogger::$user_id);
 			}
 

mod/oauth_api/vendors/oauth/library/OAuthRequestSigner.php

@@ -56,7 +56,7 @@ class OAuthRequestSigner extends OAuthRequest
 	 */
 	function __construct ( $request, $method = 'GET', $params = null, $body = null )
 	{
-		$this->store = elggconnect_get_oauth_store();//OAuthStore::instance();
+		$this->store = OAuthStore::instance();
 
 		if (is_string($params))
 		{

mod/oauth_api/vendors/oauth/library/OAuthRequestVerifier.php

@@ -49,7 +49,7 @@ class OAuthRequestVerifier extends OAuthRequest
 	 */
 	function __construct ( $uri = null, $method = 'GET' )
 	{
-		$this->store = elggconnect_get_oauth_store();//OAuthStore::instance();
+		$this->store = OAuthStore::instance();
 		parent::__construct($uri, $method);
 
 		OAuthRequestLogger::start($this);

mod/oauth_api/vendors/oauth/library/OAuthRequester.php

@@ -150,7 +150,7 @@ static function requestRequestToken ( $consumer_key, $usr_id, $params = null, $m
 			$params['xoauth_token_ttl'] = intval($options['token_ttl']);
 		}
 
-		$store	= elggconnect_get_oauth_store();//OAuthStore::instance();
+		$store	= OAuthStore::instance();
 		$r		= $store->getServer($consumer_key, $usr_id);
 		$uri 	= $r['request_token_uri'];
 
@@ -220,7 +220,7 @@ static function requestAccessToken ( $consumer_key, $token, $usr_id, $method = '
 	{
 		OAuthRequestLogger::start();
 
-		$store	    = elggconnect_get_oauth_store();//OAuthStore::instance();
+		$store	    = OAuthStore::instance();
 		$r		    = $store->getServerTokenSecrets($consumer_key, $token, 'request', $usr_id);
 		$uri 	    = $r['access_token_uri'];
 		$token_name	= $r['token_name'];

mod/oauth_api/vendors/oauth/library/OAuthServer.php

@@ -58,7 +58,7 @@ public function requestToken ()
 			}
 
 			// Create a request token
-			$store  = elggconnect_get_oauth_store();//OAuthStore::instance();
+			$store  = OAuthStore::instance();
 			$token  = $store->addConsumerRequestToken($this->getParam('oauth_consumer_key', true), $options);
 			$result = 'oauth_token='.$this->urlencode($token['token'])
 					.'&oauth_token_secret='.$this->urlencode($token['token_secret']);
@@ -104,7 +104,7 @@ public function authorizeVerify ( )
 	{
 		OAuthRequestLogger::start($this);
 
-		$store = elggconnect_get_oauth_store();//OAuthStore::instance();
+		$store = OAuthStore::instance();
 		$token = $this->getParam('oauth_token', true);
 		$rs    = $store->getConsumerRequestToken($token);
 		if (empty($rs))
@@ -142,7 +142,7 @@ public function authorizeFinish ( $authorized, $user_id )
 			&&	$_SESSION['verify_oauth_token'] == $token)
 		{
 			// Flag the token as authorized, or remove the token when not authorized
-			$store = elggconnect_get_oauth_store();//OAuthStore::instance();
+			$store = OAuthStore::instance();
 
 			// Fetch the referrer host from the oauth callback parameter
 			$referrer_host  = '';
@@ -198,7 +198,7 @@ public function accessToken ()
 				$options['token_ttl'] = $ttl;
 			}
 
-			$store  = elggconnect_get_oauth_store();//OAuthStore::instance();
+			$store  = OAuthStore::instance();
 			$token  = $store->exchangeConsumerRequestForAccessToken($this->getParam('oauth_token', true), $options);
 			$result = 'oauth_token='.$this->urlencode($token['token'])
 					.'&oauth_token_secret='.$this->urlencode($token['token_secret']);

views/default/annotation/generic_comment.php

@@ -2,8 +2,8 @@
 /**
  * Elgg generic comment view
  *
- * @uses $vars['annotation']    ElggAnnotation object
- * @uses $vars['full_view']          Display fill view or brief view
+ * @uses $vars['annotation']  ElggAnnotation object
+ * @uses $vars['full_view']   Display fill view or brief view
  */
 
 if (!isset($vars['annotation'])) {

views/rss/annotation/generic_comment.php

@@ -0,0 +1,33 @@
+<?php
+/**
+ * Elgg RSS view for a generic_comment annotation
+ *
+ * @package Elgg
+ * @subpackage Core
+ */
+
+$annotation = $vars['annotation'];
+
+$poster = $annotation->getOwnerEntity();
+$poster_name = htmlspecialchars($poster->name, ENT_NOQUOTES, 'UTF-8');
+$pubdate = date('r', $annotation->getTimeCreated());
+$permalink = $annotation->getURL();
+
+$title = elgg_echo('generic_comment:title', array($poster_name));
+
+$creator = elgg_view('page/components/creator', array('entity' => $annotation));
+$extensions = elgg_view('extensions/item');
+
+$item = <<<__HTML
+<item>
+	<guid isPermaLink='true'>$permalink</guid>
+	<pubDate>$pubdate</pubDate>
+	<link>$permalink</link>
+	<title><![CDATA[$title]]></title>
+	<description><![CDATA[{$vars['annotation']->value}]]></description>
+	$creator$extensions
+</item>
+
+__HTML;
+
+echo $item;

views/rss/group/default.php

@@ -1,37 +1,37 @@
 <?php
 /**
- * Elgg default group view
+ * RSS group view
  *
  * @package Elgg
  * @subpackage Core
  */
 
-?>
+$permalink = htmlspecialchars($vars['entity']->getURL(), ENT_NOQUOTES, 'UTF-8');
+$pubdate = date('r', $vars['entity']->getTimeCreated());
+$title = htmlspecialchars($vars['entity']->name, ENT_NOQUOTES, 'UTF-8');
 
+if ($vars['entity']->description) {
+	$description = autop($vars['entity']->description);
+} elseif ($vars['entity']->briefdescription) {
+	$description = autop($vars['entity']->briefdescription);
+} else {
+	$description = '';
+}
+
+$creator = elgg_view('page/components/creator', $vars);
+$georss = elgg_view('page/components/georss', $vars);
+$extension = elgg_view('extensions/item');
+
+$item = <<<__HTML
 <item>
-<guid isPermaLink='true'><?php echo htmlspecialchars($vars['entity']->getURL()); ?></guid>
-<pubDate><?php echo date("r",$vars['entity']->time_created) ?></pubDate>
-<link><?php echo htmlspecialchars($vars['entity']->getURL()); ?></link>
-<title><![CDATA[<?php echo (($vars['entity']->name)); ?>]]></title>
-<description><![CDATA[<?php echo (autop($vars['entity']->description)); ?>]]></description>
-<?php
-		$owner = $vars['entity']->getOwnerEntity();
-		if ($owner) {
-?>
-<dc:creator><?php echo $owner->name; ?></dc:creator>
-<?php
-		}
-?>
-<?php
-		if (
-			($vars['entity'] instanceof Locatable) &&
-			($vars['entity']->getLongitude()) &&
-			($vars['entity']->getLatitude())
-		) {
-			?>
-			<georss:point><?php echo $vars['entity']->getLatitude(); ?> <?php echo $vars['entity']->getLongitude(); ?></georss:point>
-			<?php
-		}
-?>
-<?php echo elgg_view('extensions/item'); ?>
+	<guid isPermaLink="true">$permalink</guid>
+	<pubDate>$pubdate</pubDate>
+	<link>$permalink</link>
+	<title><![CDATA[$title]]></title>
+	<description><![CDATA[$description]]></description>
+	$creator$georss$extension
 </item>
+
+__HTML;
+
+echo $item;

views/rss/object/default.php

@@ -1,25 +1,22 @@
 <?php
 /**
- * Elgg default object view
+ * RSS object view
  *
  * @package Elgg
  * @subpackage Core
  */
 
 $title = $vars['entity']->title;
 if (empty($title)) {
-	$subtitle = strip_tags($vars['entity']->description);
-	$title = substr($subtitle, 0, 32);
-	if (strlen($subtitle) > 32) {
-		$title .= ' ...';
-	}
+	$title = strip_tags($vars['entity']->description);
+	$title = elgg_get_excerpt($title, 32);
 }
 
-$permalink = htmlspecialchars($vars['entity']->getURL());
-$pubdate = date('r', $vars['entity']->time_created);
+$permalink = htmlspecialchars($vars['entity']->getURL(), ENT_NOQUOTES, 'UTF-8');
+$pubdate = date('r', $vars['entity']->getTimeCreated());
 
-$creator = elgg_view('object/creator', $vars);
-$georss = elgg_view('object/georss', $vars);
+$creator = elgg_view('page/components/creator', $vars);
+$georss = elgg_view('page/components/georss', $vars);
 $extension = elgg_view('extensions/item');
 
 $item = <<<__HTML

views/rss/output/url.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * RSS url view
+ * RSS url output view
  * 
  */
 echo elgg_view('output/url', $vars, false, false, 'default');