search

dmolsen / MIT-Mobile-Web

The Mobile Web Project was initially developed during the summer of 2009 and is a fork of the original v0.9 release of the MIT Mobile Web project that can still be found on SourceForge. The project was updated in May 2010. It is a product designed to make it easier for higher education institutions to deliver mobile-optimized information and services. It helps deliver task-based content like maps, events, and directory information optimized for device "families." Full documentation can be found at http://mobilewebosp.pbworks.com/ Updates to the project can be found on Twitter at http://twitter.com/mobilewebosp/
http://mobilewebosp.pbworks.com
MIT License
57 stars 19 forks source link

rss_services.php (bad assumption on the uniqueness of title field) #18

Closed RobinNil closed 14 years ago

RobinNil commented 14 years ago

The existing RSS library assumes that the title of each post is not the same.

This actually happened to me, if you have 2 postings in the feed with the same title, only the later one will get picked up. The library now will only pick up the posting (latest in the for-loop). The patch below will ensure that all postings be picked up.

It's a quick patch below.

$ diff ./rss_services.php ./rss_services.php.dist 
1c1
<         $items[$title] = $headline;
dmolsen commented 14 years ago

bug should be fixed in the master branch. i'm still testing a few things there but i'll push it into the 2.0 branch soon. thanks much for finding that!