GTNPORTAL-3519 BZ1117813 Reverted change of category's name from 854461f

I remember that there was issues with the automatic name created for
portlets in the import-all action. That was aim for the change. One
suggestion can be to apply logic for remote portles inside
sanitizePortletName aux method. IMO.
El 11/07/2014 16:04, "Juraci Paixão Kröhling" <notifications@github.com>
escribió:

> In
> component/application-registry/src/main/java/org/exoplatform/application/registry/impl/ApplicationRegistryServiceImpl.java:
>
> > @@ -488,7 +488,7 @@ public void importAllPortlets() throws Exception {
> >                      }
> >
> >                      if (!isExist) {
> > -                        app = category.createContent(portletName, contentType, contentId);
> > +                        app = category.createContent(displayName, contentType, contentId);
>
> You are not supposed to look at this now. I took the opportunity that you
> are in PTO to make this change :D
>
> Basically, I reverted this part because it caused issues with WSRP: when
> there's a local portlet named "my-little-portlet" and a remote portlet with
> the same name, an exception occurs when doing the "Import Applications"
> from the application registry, and it's caused by 854461f
> <https://github.com/gatein/gatein-portal/commit/854461f> . The remaining
> of the commit is still there, but this particular line, responsible for the
> categories, had to be reverted.
>
> @vramik <https://github.com/vramik> tested the WSRP bug, while @pmensik
> <https://github.com/pmensik> tested the ones that resulted in 854461f
> <https://github.com/gatein/gatein-portal/commit/854461f> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gatein/gatein-portal/pull/887/files#r14821934>.
>