Issue Details (XML | Word | Printable)

Key: MDL-13932
Type: Task Task
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Dan Poltawski
Reporter: Nicolas Connault
Votes: 1
Watchers: 4
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Rewrite RSS block

Created: 14/Mar/08 07:57 PM   Updated: 30/Jul/09 10:14 PM
Return to search
Component/s: RSS
Affects Version/s: 2.0
Fix Version/s: 2.0

File Attachments: 1. Text File rssrework250609.patch (67 kB)
Issue Links:
Relates
 

Participants: Charles Fulton, Dan Poltawski, Nicolas Connault, Petr Skoda and Tim Hunt
Security Level: None
Resolved date: 30/Jul/09
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
Update simplepie lib to latest version and use it instead of magpie. Rewrite role-related code so that the block finally works well for student submissions and on the MyMoodle page.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Dan Poltawski committed 4 files to 'Moodle CVS' - 13/Apr/09 11:29 PM
Adding SimplePie feed parsing library MDL-7946, MDL-13932

To replace unmaintained magipie. Simplepie is actively maintained has some
nice features such as feed autodisovery. Implemented a simple extended class
which uses the Moodle core curl class from filelib.

Added unit test using the moodle.org sample files. Please note these tests will
fail if moodle can't make http connections out, so ensure proxy settings
etc are correct.
ADD lib/simplepie/simplepie.inc   Rev. 1.1    (+0 -0 lines)
ADD lib/simpletest/testrss.php   Rev. 1.1    (+0 -0 lines)
ADD lib/simplepie/readme_moodle.txt   Rev. 1.1    (+0 -0 lines)
ADD lib/simplepie/moodle_simplepie.php   Rev. 1.1    (+0 -0 lines)
Dan Poltawski committed 2 files to 'Moodle CVS' - 14/Apr/09 12:00 AM
simplepie: fix class name caseing: MDL-7946, MDL-13932
MODIFY lib/simpletest/testrss.php   Rev. 1.2    (+3 -3 lines)
MODIFY lib/simplepie/moodle_simplepie.php   Rev. 1.2    (+4 -4 lines)
Dan Poltawski committed 2 files to 'Moodle CVS' - 27/Jul/09 10:18 PM
lib/simplepie: Improve Integration MDL-7946, MDL-13932

Added more unit tests, improved style and fixed:
- Failed behaviour of redirects
- Incorrect behaviour on feed retrival failure
- Ensured rss cache clear for unit test setup
MODIFY lib/simpletest/testrss.php   Rev. 1.4    (+54 -9 lines)
MODIFY lib/simplepie/moodle_simplepie.php   Rev. 1.4    (+47 -19 lines)
tjhunt committed 14 files to 'Moodle CVS' - 29/Jul/09 07:18 PM
block_rss_client: MDL-13932 Clean up a lot of the crap.

This was all in order to do MDL-19889!

* The old tabbed interface is gone.
* The manage feeds UI has been completely rewritten to be a few
separate sane scripts like managefeeds.php, editfeed.php and viewfeed.php
* This is used from both Admin -> plugins -> blocks ... and as a link from the block edit_form.php
* new edit_form.php for the block replacing config_istance.php.
* A lot of stuff to do with UI specific to this block has been removed from rsslib.php
ADD blocks/rss_client/managefeeds.php   Rev. 1.1    (+0 -0 lines)
ADD blocks/rss_client/editfeed.php   Rev. 1.1    (+0 -0 lines)
MODIFY lang/en_utf8/error.php   Rev. 1.198    (+2 -1 lines)
MODIFY lib/rsslib.php   Rev. 1.62    (+4 -182 lines)
DEL blocks/rss_client/Attic/block_rss_client_action.php   Rev. 1.69    (+0 -0 lines)
DEL blocks/rss_client/Attic/block_rss_client_error.php   Rev. 1.8    (+0 -0 lines)
DEL blocks/rss_client/Attic/config_instance_tabs.php   Rev. 1.17    (+0 -0 lines)
MODIFY blocks/rss_client/block_rss_client.php   Rev. 1.94    (+18 -57 lines)
MODIFY lang/en_utf8/block_rss_client.php   Rev. 1.8    (+7 -1 lines)
ADD blocks/rss_client/viewfeed.php   Rev. 1.1    (+0 -0 lines)
ADD blocks/rss_client/edit_form.php   Rev. 1.1    (+0 -0 lines)
MODIFY blocks/rss_client/db/access.php   Rev. 1.9    (+0 -23 lines)
DEL blocks/rss_client/Attic/config_instance.html   Rev. 1.41    (+0 -0 lines)
MODIFY blocks/rss_client/settings.php   Rev. 1.2    (+2 -5 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 29/Jul/09 09:13 PM
block_rss_client: MDL-13932 Fix equality typo
MODIFY blocks/rss_client/editfeed.php   Rev. 1.2    (+1 -1 lines)
Dan Poltawski committed 3 files to 'Moodle CVS' - 29/Jul/09 09:44 PM
block_rss_client: MDL-13932/MDL-7946 - Switch to use Simplepie

Simplepie is actively maintained and supports a broader range of feeds
MODIFY blocks/rss_client/editfeed.php   Rev. 1.3    (+6 -9 lines)
MODIFY blocks/rss_client/viewfeed.php   Rev. 1.2    (+8 -19 lines)
MODIFY blocks/rss_client/block_rss_client.php   Rev. 1.95    (+195 -172 lines)
Mitsuhiro Yoshida committed 3 files to 'Lang CVS' - 30/Jul/09 12:13 AM
MDL-13932 Translated new strings for RSS block.
MODIFY ja_utf8/block_rss_client.php   Rev. 1.11    (+7 -1 lines)
MODIFY ja_utf8/error.php   Rev. 1.144    (+2 -1 lines)
MODIFY ja_utf8/README   Rev. 1.1038    (+1 -1 lines)
martignoni committed 2 files to 'Lang CVS' - 30/Jul/09 01:24 AM
MDL-13932, some strings added
MODIFY fr_utf8/block_rss_client.php   Rev. 1.12    (+7 -1 lines)
MODIFY fr_utf8/error.php   Rev. 1.201    (+1 -0 lines)
Dan Poltawski committed 13 files to 'Moodle CVS' - 30/Jul/09 10:49 PM
Remove lib/magpie: MDL-7946/MDL-13932

It is no longer maintained and simplepie has
replaced it as a maintained and robust alternative
DEL lib/magpie/Attic/cookbook   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/rss_fetch.inc   Rev. 1.10    (+0 -0 lines)
DEL lib/magpie/Attic/CHANGES   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/AUTHORS   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/TROUBLESHOOTING   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/README   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/htdocs/Attic/index.html   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/rss_cache.inc   Rev. 1.3    (+0 -0 lines)
MODIFY lib/thirdpartylibs.xml   Rev. 1.13    (+0 -7 lines)
DEL lib/magpie/htdocs/Attic/cookbook.html   Rev. 1.2    (+0 -0 lines)
DEL lib/magpie/Attic/README_MOODLE.txt   Rev. 1.5    (+0 -0 lines)
DEL lib/magpie/Attic/rss_utils.inc   Rev. 1.4    (+0 -0 lines)
DEL lib/magpie/Attic/rss_parse.inc   Rev. 1.4    (+0 -0 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 01/Aug/09 05:19 AM
blocks/rss_client: MDL-13932 fix config_shownumentries typo

This caused incorrect rules/defaults to be added

Also add client side validation of the number
MODIFY blocks/rss_client/edit_form.php   Rev. 1.3    (+4 -3 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 01/Aug/09 05:19 AM
blocks/rss_client: MDL-13932 Incease timeout when adding feeds

In order to stop 'slow loading feeds' causing performance problems
with standard moodle pages, i've set the timeout for simplepie quite
low. When cron runs i've set this timeout high and cache duration low.

The intention is that cron does all the hard work of connecting to remote
sites and users get this fast fast fast from cache.

However, in the case of adding a feed its probably better to try and get the
feed more agressively..
MODIFY blocks/rss_client/editfeed.php   Rev. 1.4    (+5 -1 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 01/Aug/09 05:20 AM
blocks/rss_client: MDL-13932: viewfeed.php typo

link -> description
MODIFY blocks/rss_client/viewfeed.php   Rev. 1.3    (+1 -1 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 01/Aug/09 05:20 AM
blocks/rss_client: MDL-13932 Fix typo when editting feed

This caused loss of which course we were adding/edditing a feed to.
MODIFY blocks/rss_client/editfeed.php   Rev. 1.5    (+1 -1 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 06/Aug/09 12:38 AM
blocks/rss_client: MDL-13932 - alter output formating

Doesn't make much sense to have FORMAT_MOODLE for external
rss content.
MODIFY blocks/rss_client/block_rss_client.php   Rev. 1.98    (+1 -1 lines)
Dan Poltawski committed 1 file to 'Moodle CVS' - 24/Aug/09 01:56 AM
blocks/rss_client MDL-13932 Fix wrong param order in $OUTPUT->link()
MODIFY blocks/rss_client/managefeeds.php   Rev. 1.6    (+1 -1 lines)

Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.
Atlassian JIRA the Professional Issue Tracker. (Enterprise Edition, Version: 3.13.5-#360) - Bug/feature request - Atlassian news - Contact Administrators