Jetspeed 2
  1. Jetspeed 2
  2. JS2-1249

portlet decorator actions displaying alt text from the wrong locale

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.0
    • Fix Version/s: 2.2.2
    • Component/s: Decorations/Themes
    • Labels:
      None
    • Environment:
      Windows XP SP3
      Sun Java 1.5
      Jetspeed 2.2.0

      Observed in Firefox 3.6.x, Firefox 4 and Chrome 10.x

      Description

      When switching the locale in the browser configuration between Japanese and English, the alt text for the portlet decorator actions are not consistently in the correct locale.

      While using the ja locale, the alt text will remain in English for 'Maximize', 'Minimize' etc, and vice versa, while using en locale, the Japanese characters may display for the alt text.

      While stepping through AbstractDecoratorActionsFactory.createAction, one line 169

      169 HashMap resourcesMap = (HashMap)actionResourcesMap.get();

      If this returns a reference, the values in the resourceMap are not cleared and initialized with the values from the localized bundle.
      The resourceMap contains the localized bundle, it seems that even if a valid reference is returned, the locale of the bundle in the resourceMap should be compared with locale in the jetspeedRequestContext to see if it needs to be updated.

      Because the resourceMap is not cleared when the locale changes, the localizedName string used to create the new DecoratorAction is in the wrong locale.

      I initially posted this on the jetspeed forum, and then opened this issue.
      http://mail-archives.apache.org/mod_mbox/portals-jetspeed-user/201104.mbox/%3CBANLkTim52-YDXnqJn84U0i0GWrgTVW=fbg@mail.gmail.com%3E

        Activity

        Daniel DeGroff created issue -
        Woonsan Ko made changes -
        Field Original Value New Value
        Assignee Woonsan Ko [ woon_san ]
        Fix Version/s 2.2.2 [ 12313846 ]
        Hide
        Woonsan Ko added a comment -

        Thank you so much, Daniel!
        Cheers,
        Woonsan

        Show
        Woonsan Ko added a comment - Thank you so much, Daniel! Cheers, Woonsan
        Woonsan Ko made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Woonsan Ko
            Reporter:
            Daniel DeGroff
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development