Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

NEWSREADER-1: Fixes for the single news feed instance. #2

Merged
merged 1 commit into from
Apr 4, 2012
Merged

NEWSREADER-1: Fixes for the single news feed instance. #2

merged 1 commit into from
Apr 4, 2012

Conversation

mmoayyed
Copy link

@mmoayyed mmoayyed commented Apr 4, 2012 

  1.   Updated XML schema declarations to Spring 3.0 (existing dependency) 

  2.   Fixed a crash in the single news feed instance, where an empty news feed resulted in a negative upper loop boundary, failing the JSP that was going to iterate over the news set. 

  3.   Fixed the HELP mode of the portlet for single news feed, presently crashing.

  4.   Allowed the single news feed preferences to be able to manage the feed name, url and rendering class on its own. (the UP fix was only recently checked into trunk) 

  5.   Fixed the javascript namespacing issue when editing single news feed urls.

@SavvasMisaghMoayyed
NEWSREADER-1: Fixed a bug with the single newsfeed renderer JSP where…
0a2e024 
… empty feeds caused an exception. Also allowed the portlet to administer its own preferences, allowing the user to enter the feed title, url and class name. Also updated the schema declarations to Spring 3.0 and updated the single news feed UI to display the message for misbehaving feeds that could be loaded/parsed.
drewwills added a commit that referenced this pull request Apr 4, 2012
@drewwills
Merge pull request #2 from mmoayyed/NEWSREADER-1
196c803 
NEWSREADER-1: Fixes for the single news feed instance.
@drewwills drewwills merged commit 196c803 into uPortal-Project:master Apr 4, 2012
@drewwills
Copy link
Contributor

thanks Misagh!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mmoayyed @drewwills @SavvasMisaghMoayyed