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

Add getName() method to Role to support localized role label

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.9.x
    • Fix Version/s: None
    • Labels:
    • Previous Issue Keys:
      KNL-1205

      Description

      This issue is related to SAK-25889.
      Since it seems no way to make KNL issue as a sub-task of SAK issue, I have to create this issue directly.

      The detail can be found in SAK-25889.

      Refer to org.sakaiproject.tool.impl.ToolImpl.getDescription(), I will use the same way to support localized role label.
      So a getName() function is added to Role to get the localized label.
      Besides, a getLocalizedDescription() function is added.
      The existed getDescription() function remains.

      Some properties file should be added first. These changes can be found in SAK-25890.

      IMPORTANT:
      When a site is created, the role description will be automatically set in DB table sakai_realm_role_desc.
      And the getLocalizedDescription first get the getDescription value to handle customized value.
      So, to get the localized description, update.sql should be executed.
      To prevent new created site to use this feature, the patch sql.diff should be applied.

      kernel.diff contains the patch to add the 2 new methods.
      sql.diff contains the patch to prevent default description be set when a new site is created.
      update.sql contains SQL to update the sakai_realm_role_desc table.

      The patch is based on 1.3.x branch

        Gliffy Diagrams

          Zeplin

            Attachments

            1. kernel.diff
              8 kB
            2. sakai-mock.diff
              0.5 kB
            3. sql.diff
              13 kB
            4. update.sql
              0.3 kB

              Issue Links

                Activity

                  People

                  Assignee:
                  Unassigned Unassigned
                  Reporter:
                  gaojun Gao Jun
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration