review code from hellekin comments

diff --git a/actions/simplepie/group_module.php b/actions/simplepie/group_module.php
deleted file mode 100644 (file)
index 93fe353..0000000
--- a/actions/simplepie/group_module.php
+++ /dev/null
@@ -1,8 +0,0 @@
-<?php
-
-$group = get_entity(get_input('group_guid'));
-$feed_url = get_input('feed_url');
-
-if ($group && $group->canEdit()) {
-       $group->feed_url = $feed_url;
-}

diff --git a/languages/en.php b/languages/en.php
index 07ff029052f9c3c67f742f2e33d3846f6ba969c3..082b4c485a1fbf1b1a99b5ef6c16ec7759ac88e0 100644 (file)
--- a/languages/en.php
+++ b/languages/en.php
@@ -4,12 +4,14 @@ $english = array(
        'simplepie:widget' => 'RSS Feed',\r
        'simplepie:description' => 'Add an external blog to your profile',\r
        'simplepie:notset' => 'Feed url is not set',\r
-       'simplepie:notfind' => 'Cannot find feed. Check the feed url.',\r
+       'simplepie:notfound' => 'Cannot find feed. Check the feed url.',\r
        'simplepie:feed_url' => 'Feed URL',\r
        'simplepie:num_items' => 'Number of items',\r
        'simplepie:excerpt' => 'Include excerpt',\r
        'simplepie:post_date' => 'Include post date',\r
        'simplepie:postedon' => 'Posted on',\r
+       'simplepie:invalid_url' => 'Invalid url, copy it from the navigation bar please',\r
+\r
 );\r
 \r
 add_translation("en", $english);\r

diff --git a/languages/es.php b/languages/es.php
index 51b53e9f24a3262d7afc95a7d3ce68cb236807e0..85f24e4a50376c279e20bdd0108537a3913af91e 100644 (file)
--- a/languages/es.php
+++ b/languages/es.php
@@ -1,16 +1,22 @@
 <?php\r
 \r
 $spanish = array(\r
-       'simplepie:widget' => 'Enlace RSS',\r
-       'simplepie:description' => 'Agregar un blog externo',\r
-       'simplepie:notset' => 'Enlace RSS no configurado',\r
-       'simplepie:notfind' => 'no se encontro el feed. Revisa el feed url.',\r
-       'simplepie:feed_url' => 'Feed URL',\r
-       'simplepie:num_items' => 'Numero de items',\r
-       'simplepie:excerpt' => 'Incluir contenido',\r
-       'simplepie:post_date' => 'Incluir fecha del post',\r
-       'simplepie:postedon' => 'Posted on',\r
+        'simplepie:widget' => 'Enlace RSS',\r
+        'simplepie:description' => 'Agregar un blog externo',\r
+        'simplepie:notset' => 'Enlace RSS no configurado',\r
+        'simplepie:notfound' => 'no se encontro el feed. Revisa el feed url.',\r
+        'simplepie:feed_url' => 'Feed URL',\r
+        'simplepie:num_items' => 'Numero de items',\r
+        'simplepie:excerpt' => 'Incluir contenido',\r
+        'simplepie:post_date' => 'Incluir fecha del post',\r
+        'simplepie:postedon' => 'Posted on',\r
+        'simplepie:invalid_url' => 'Url invalida, copiela desde la barra del navegador por favor',\r
 );\r
 \r
 add_translation("en", $english);\r
 \r
+\r
+\r
+\r
+\r
+\r

diff --git a/start.php b/start.php
index 26c2e501704cf6d40fd167ea51eb9fa4c49707e2..34a7ced3f8076bd85d99b87a93ffd1c9803c25c4 100644 (file)
--- a/start.php
+++ b/start.php
@@ -25,6 +25,9 @@ function simplepie_init() {
         add_group_tool_option('rss', elgg_echo('simplepie:enablerss'), false);\r
         elgg_extend_view('groups/tool_latest', 'simplepie/group_module');\r
 \r
-       elgg_register_action('simplepie/group_module', elgg_get_plugins_path() . 'simplepie/actions/simplepie/group_module.php');\r
+       elgg_register_action('simplepie/save_group_feed', elgg_get_plugins_path() . 'simplepie/actions/simplepie/save_group_feed.php');\r
 }\r
 \r
+function simplepie_is_url($url) {\r
+       return preg_match("/\b(?:(?:https?|ftp):\/\/|www\.)[-a-z0-9+&@#\/%?=~_|!:,.;]*[-a-z0-9+&@#\/%=~_|]/i", $url);\r
+}\r

diff --git a/views/default/forms/simplepie/group_module.php b/views/default/forms/simplepie/group_module.php
deleted file mode 100644 (file)
index a7c1f68..0000000
--- a/views/default/forms/simplepie/group_module.php
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-/**
- * Simplepie feed reader widget settings
- */
-
-// set default value
-
-$url_label = elgg_echo("simplepie:feed_url");
-$url_textbox = elgg_view('input/text', array(
-       'name' => 'feed_url',
-       'value' => $vars['entity']->feed_url,
-));
-
-$group_field = elgg_view('input/hidden', array(
-       'name' => 'group_guid',
-       'value' => $vars['entity']->guid,
-));
-
-$save_button = elgg_view('input/submit', array(
-       'value' => elgg_echo('save'),
-));
-
-
-
-
-echo <<<HTML
-<div>
-       $url_label
-       $url_textbox
-       $group_field
-</div>
-<div>
-       $save_button
-</div>
-HTML;

diff --git a/views/default/simplepie/group_module.php b/views/default/simplepie/group_module.php
index 7d0a7bc2ff814a8a45a62137f57e006498fe02b4..17b2dba0cbb9550ec83ace2f199283e35d6dc952 100644 (file)
--- a/views/default/simplepie/group_module.php
+++ b/views/default/simplepie/group_module.php
@@ -17,7 +17,7 @@ $feed_url = $group->feed_url;
 $content = '';
 
 if ($group->canEdit()) {
-       $content .= elgg_view_form("simplepie/group_module", array(
+       $content .= elgg_view_form("simplepie/save_group_feed", array(
                'id' => 'simplepie-form',
                'class' => $feed_url ? 'hidden' : '',
        ), $vars);
@@ -25,64 +25,66 @@ if ($group->canEdit()) {
 
 if ($feed_url) {
 
-        // get widget settings
-        $excerpt   = true;
-        $post_date = true;
+// get widget settings
+       $excerpt   = true;
+       $post_date = true;
        $num_items = 7;
 
-        $cache_location = elgg_get_data_path() . '/simplepie_cache/';
-        if (!file_exists($cache_location)) {
-                mkdir($cache_location, 0777);
-        }
+       $cache_location = elgg_get_data_path() . '/simplepie_cache/';
+       if (!file_exists($cache_location)) {
+               mkdir($cache_location, 0777);
+       }
 
-        $feed = new SimplePie($feed_url, $cache_location);
+       $feed = new SimplePie($feed_url, $cache_location);
 
-        // doubles timeout if going through a proxy
-        //$feed->set_timeout(20);
+// doubles timeout if going through a proxy
+//$feed->set_timeout(20);
 
-        // only display errors to profile owner
-        $num_posts_in_feed = $feed->get_item_quantity();
-        if (!$num_posts_in_feed) {
-                if (elgg_get_logged_in_user_guid() == elgg_get_page_owner_guid()) {
-                        $content .= '<p>' . elgg_echo('simplepie:notfind') . '</p>';
-                }
-        }
 
-        // don't display more feed items than user requested
-        if ($num_items > $num_posts_in_feed) {
-                $num_items = $num_posts_in_feed;
-        }
+// only display errors to profile owner
+       $num_posts_in_feed = $feed->get_item_quantity();
+if (!$num_posts_in_feed) {
+       if (elgg_get_logged_in_user_guid() == elgg_get_page_owner_guid()) {
+               $content .= '<p>' . elgg_echo('simplepie:notfound') . '</p>';
+       }
+}
+
+// don't display more feed items than user requested
+       if ($num_items > $num_posts_in_feed) {
+       $num_items = $num_posts_in_feed;
+}
+
+       $feed_link = elgg_view('output/url', array(
+       'href' => $feed->get_permalink(),
+       'text' => $feed->get_title(),
+));
+
+// need to center
+       $content .= "<h2 class=\"simplepie-heading\">$feed_link</h2>";
+       $content .= '<ul class="simplepie-list">';
+       foreach ($feed->get_items(0, $num_items) as $item) {
+       $item_link = elgg_view('output/url', array(
+       'href' => $item->get_permalink(),
+       'text' => $item->get_title(),
+));
+
+if ($excerpt) {
+       $text = strip_tags($item->get_description(true), $allowed_tags);
+       $excerpt = elgg_get_excerpt($text);
+}
+
+if ($post_date) {
+       $item_date_label = elgg_echo('simplepie:postedon');
+       $item_date = $item->get_date('j F Y | g:i a');
+       $post_date = "$item_date_label $item_date";
+}
+
+       $content .= <<<HTML
 
-        $feed_link = elgg_view('output/url', array(
-                'href' => $feed->get_permalink(),
-                'text' => $feed->get_title(),
-        ));
-
-        // need to center
-        $content .= "<h2 class=\"simplepie-heading\">$feed_link</h2>";
-        $content .= '<ul class="simplepie-list">';
-        foreach ($feed->get_items(0, $num_items) as $item) {
-                $item_link = elgg_view('output/url', array(
-                        'href' => $item->get_permalink(),
-                        'text' => $item->get_title(),
-                ));
-
-                if ($excerpt) {
-                        $text = strip_tags($item->get_description(true), $allowed_tags);
-                        $excerpt = elgg_get_excerpt($text);
-                }
-
-                if ($post_date) {
-                        $item_date_label = elgg_echo('simplepie:postedon');
-                        $item_date = $item->get_date('j F Y | g:i a');
-                        $post_date = "$item_date_label $item_date";
-                }
-
-                $content .= <<<HTML
 <li class="mbm elgg-item">
-        <h4 class="mbs">$item_link</h4>
-        <p class="elgg-subtext">$post_date</p>
-        <div class="elgg-content">$excerpt</div>
+       <h4 class="mbs">$item_link</h4>
+       <p class="elgg-subtext">$post_date</p>
+       <div class="elgg-content">$excerpt</div>
 </li>
 HTML;
 
@@ -91,8 +93,6 @@ HTML;
 
 }
 
-
-
 elgg_pop_context();
 
 if (!$content) {