Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-42222

Overview page should fallback to short_desc then title

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: Awaiting Review
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Web Components
    • Labels:
      None
    • Test Plan:
      Hide

      Please add a Test Plan here.

      Show
      Please add a Test Plan here.

      Description

      Currently, the overview page tries to display the site description, but if the description is null, it will fallback to the title. But It might be nice to fallback to the short_desc, then the title

      diff --git a/web/web-portlet/src/java/org/sakaiproject/portlets/PortletIFrame.java b/web/web-portlet/src/java/org/sakaiproject/portlets/PortletIFrame.java
      index 803b671ab8..03ce9c1e8d 100644
      --- a/web/web-portlet/src/java/org/sakaiproject/portlets/PortletIFrame.java
      +++ b/web/web-portlet/src/java/org/sakaiproject/portlets/PortletIFrame.java
      @@ -336,6 +336,9 @@ public class PortletIFrame extends GenericPortlet {
                                              if (rv == null)
                                              {
                                                      String siteInfo = StringUtils.trimToNull(s.getDescription());
      +                                               if ( siteInfo == null ) {
      +                                                       siteInfo = StringUtils.trimToNull(s.getShortDescription());
      +                                               }
                                                      if ( siteInfo == null ) {
                                                              siteInfo = StringUtils.trimToNull(s.getTitle());
                                                      }
      

        Gliffy Diagrams

          Zeplin

            Attachments

              Activity

                People

                Assignee:
                Unassigned Unassigned
                Reporter:
                austinUH Austin
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                  Dates

                  Created:
                  Updated:

                    Git Integration