Details

    • Type: Bug Bug
    • Status: In Review
    • Resolution: Unresolved
    • Affects Version/s: 6.2.10 EE GA1, 6.2.X EE, 7.0.0 M3
    • Fix Version/s: None
    • Component/s: Calendar
    • Labels:
      None
    • Fix Priority:
      3
    • Similar Issues:
      Show 5 results 

      Description

      RSSAction needs a null check in getEntryURLAssetPublisher(), or RSS link will not open correctly.

      Steps to Reproduce

      1. Add Asset Publisher to a page
      2. Add Web Content Article
      3. Click the Gear icon in Asset Published and select Configuration
      4. Under Asset Publisher - Configuration, leave the default settings and click on the Subscriptions tab
      5. Enable RSS subscription
      6. Save changes
      7. Open up a browser capable of reading RSS feeds (in my test I used Firefox)
      8. Click on the RSS popout link

        Issue Links

          Activity

          Hide
          David Kubitza added a comment -

          The problem is not the missing check. In my case assetRendererFactory is null, because it tries to find the renderer for the asset class CalendarResource which does not exist. So you have to fix the calendar portlet.
          Also, the null pointer check is nice anyways, I think, it's the wrong place. At https://github.com/liferay/liferay-portal/blob/553adad92d68c0ebd8ee4da73fa3d97d54775dc5/portal-impl/src/com/liferay/portlet/assetpublisher/action/RSSAction.java#L107 it's getting the URL. If you return an empty string, then add a check for it here and skip the asset or check for classname at the beginning of the for-loop. That way, other portlets can't trigger the same error as the calendar does now.

          Show
          David Kubitza added a comment - The problem is not the missing check. In my case assetRendererFactory is null, because it tries to find the renderer for the asset class CalendarResource which does not exist. So you have to fix the calendar portlet. Also, the null pointer check is nice anyways, I think, it's the wrong place. At https://github.com/liferay/liferay-portal/blob/553adad92d68c0ebd8ee4da73fa3d97d54775dc5/portal-impl/src/com/liferay/portlet/assetpublisher/action/RSSAction.java#L107 it's getting the URL. If you return an empty string, then add a check for it here and skip the asset or check for classname at the beginning of the for-loop. That way, other portlets can't trigger the same error as the calendar does now.
          Hide
          Adam Brandizzi added a comment -

          Since there are no steps to reproduce, I could not see the problem happening, but it is vaguely similar to LPS-43713. Does this problem happens after updating a database from 6.1.x to 6.2.x?

          Show
          Adam Brandizzi added a comment - Since there are no steps to reproduce, I could not see the problem happening, but it is vaguely similar to LPS-43713 . Does this problem happens after updating a database from 6.1.x to 6.2.x?
          Hide
          David Kubitza added a comment -

          Hi Adam, as you can see George added steps. Nethertheless the problem was descripted already well enough, even a pull request exists and my comment how to solve this issue!
          No portal updates, just install a 6.2 portal. Please follow the activities of this ticket and check out the links.

          George: You can fix it temporary by using an EXT plugin.

          Show
          David Kubitza added a comment - Hi Adam, as you can see George added steps. Nethertheless the problem was descripted already well enough, even a pull request exists and my comment how to solve this issue! No portal updates, just install a 6.2 portal. Please follow the activities of this ticket and check out the links. George: You can fix it temporary by using an EXT plugin.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Days since last comment:
                47 weeks, 6 days ago

                Development

                  Structure Helper Panel