Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.2
    • Fix Version/s: Backlog
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • Similar issues:
      KULRICE-12007Cache Manager tree layout confusing
      KULRICE-13047IdentityCurrentAndArchiveService not properly caching values
      KULRICE-8298Add ehCache caching setup to KRAD
      KULRICE-13425No Cache Filter
      KULRICE-6482Document cache configuration and overriding
      KULRICE-8327Implement caching for Message Service
      KULRICE-10907Caching for key value finders
      KULRICE-6682add caching to internal DocumentTypeService
      KULRICE-5783Enable caching for ParameterRepositoryService
      KULRICE-12896Dataset Cleanup: Wipe out KRIM_ENTITY_CACHE_T entries
    • Epic Link:
    • Rice Module:
      KRAD
    • KRAD Feature Area:
      UIF MVC
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      MessageServiceImpl's getDefaultLocaleCode method is highly used, and I don't think the value changes often. We've cached the output of this method in a local variable, but that's probably not a preferred solution.

      It seems like if some form of caching was implemented, that could save a bunch of work!
      I didn't see that spring + ehcache was enabled for krad, so that's why we didn't use the cacheable annotation instead.

      private String defaultLocaleCode;
      protected String getDefaultLocaleCode() {
      if(defaultLocaleCode != null)
      return defaultLocaleCode;
      String localeCode = CoreFrameworkServiceLocator.getParameterService().getParameterValueAsString(
      KRADConstants.KNS_NAMESPACE, KRADConstants.DetailTypes.ALL_DETAIL_TYPE,
      KRADConstants.ParameterNames.DEFAULT_LOCALE_CODE);

      // if not configured fall back to english US
      if (StringUtils.isBlank(localeCode))

      { localeCode = "en-US"; }

      defaultLocaleCode = localeCode;
      return localeCode;
      }

        Issue Links

          Activity

          Matt Sargent made changes -
          Field Original Value New Value
          Documentation Review Status Pending Review [ 14643 ] Not Required [ 14642 ]
          Jerry Neal (Inactive) made changes -
          Fix Version/s 2.3 [ 16595 ]
          Jerry Neal (Inactive) made changes -
          Link This issue relates to KULRICE-6557 [ KULRICE-6557 ]
          Jessica Coltrin (Inactive) made changes -
          Fix Version/s 2.4 [ 16913 ]
          Fix Version/s 2.3 [ 16595 ]
          Jessica Coltrin (Inactive) made changes -
          Fix Version/s 2.4-backlog [ 16596 ]
          Fix Version/s 2.4 [ 16913 ]
          Jerry Neal (Inactive) made changes -
          Epic Link KULRICE-10089 [ 120211 ]
          Shem Patterson (Inactive) made changes -
          Workflow custom [ 163865 ] Copy of custom for rice [ 207799 ]
          Shem Patterson (Inactive) made changes -
          Workflow Copy of custom for rice [ 207799 ] custom [ 217547 ]
          Shem Patterson (Inactive) made changes -
          Workflow custom [ 217547 ] Rice Workflow [ 227295 ]
          Jessica Coltrin (Inactive) made changes -
          Fix Version/s 2.4 [ 16913 ]
          Fix Version/s 2.4-backlog [ 16596 ]
          Jerry Neal (Inactive) made changes -
          Component/s Development [ 11244 ]
          Component/s User Experience (UX) [ 13465 ]
          Component/s Performance [ 13588 ]
          Jerry Neal (Inactive) made changes -
          Fix Version/s Backlog [ 15811 ]
          Fix Version/s 2.4 [ 16913 ]
          Eric Westfall made changes -
          Labels Old

            People

            • Assignee:
              Unassigned
              Reporter:
              Jeff Domeyer (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:

                Structure Helper Panel