Remove the mandatory URL from announcements entries

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 6.0.2 RC
• Fix Version/s: --Sprint 12/11, 6.1.0 CE RC1
• Component/s: Collaboration > Announcements
• Labels: None
• Liferay Contributor's Agreement:
  Accept
• Similar Issues: Show 4 results ; ;

  LPS-1841	Announcements\Alert Portlet Improvements
  LPS-9083	Lack of URL check algorithm when adding an announcement entry
  LPS-17443	Announcement Entries do not show up in the main view of the Announcments Portlet.
  LPS-32596	URL changes from a user's customized user group page to the user's private page after interacting with the Announcement portlet on the page

Until RC1 URL wasn't mandatory and now you can't add an entry without an URL

Many announcements (if not all) are just announcements, meaning they don't need to point to another URL

[Shagul H Khajamohideen]

Hi,

Can someone confirm if URL has been a mandatory field? If not, the validate method in AnnouncementsLocalServiceImpl should check for not null and not url.

Currently the below check will always throw an exception for empty or invalid URL.

if (!Validator.isUrl(url))

{ throw new EntryURLException(); }

Verified in August 9, 2010 nightly build.

http://www.liferay.com/community/forums/-/message_boards/message/5533244

[Carlos Palol]

I can't see any reason for an announcement to require a URL. This must be a bug.

The URL validation should be: "EITHER empty OR valid URL".

[Tagnaouti Boubker]

This issue is a backlog candidate for the 100 PaperCuts program. Please consider participating! See http://liferay.com/community/100-papercuts

[Jaromir Hamala]

This issue is currently being addressed in Sprint 2 of the 100 PaperCuts program. Please see http://liferay.com/community/100-papercuts

[Jaromir Hamala]

Patch against 6.0.x branch

[Jaromir Hamala]

Hello,
this issue is duplicate to LPS-13091 and has been already resolved in trunk. Anyway I've attached the patch for 6.0.x branch just for case anybody would like to apply it by themself.

[Cynthia Wilburn]

Resolving as duplicate as per comments. QA will validate and reopen if necessary.