Details

    • Liferay Contributor's Agreement:
      Accept
    • Similar Issues:
      Show 4 results 

      Description

      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

        Activity

        Hide
        Shagul H Khajamohideen added a comment -

        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

        Show
        Shagul H Khajamohideen added a comment - 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
        Hide
        Carlos Palol added a comment -

        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".

        Show
        Carlos Palol added a comment - 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".
        Hide
        Tagnaouti Boubker added a comment - - edited

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

        Show
        Tagnaouti Boubker added a comment - - edited This issue is a backlog candidate for the 100 PaperCuts program. Please consider participating! See http://liferay.com/community/100-papercuts
        Hide
        Jaromir Hamala added a comment -

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

        Show
        Jaromir Hamala added a comment - This issue is currently being addressed in Sprint 2 of the 100 PaperCuts program. Please see http://liferay.com/community/100-papercuts
        Hide
        Jaromir Hamala added a comment -

        Patch against 6.0.x branch

        Show
        Jaromir Hamala added a comment - Patch against 6.0.x branch
        Hide
        Jaromir Hamala added a comment -

        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.

        Show
        Jaromir Hamala added a comment - 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.
        Hide
        Cynthia Wilburn (Inactive) added a comment -

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

        Show
        Cynthia Wilburn (Inactive) added a comment - Resolving as duplicate as per comments. QA will validate and reopen if necessary.

          People

          • Votes:
            10 Vote for this issue
            Watchers:
            10 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Days since last comment:
              3 years, 43 weeks, 3 days ago

              Development

                Structure Helper Panel