Silverpeas » Silverpeas Core

Nous avons effectivement essayé de lui faire renvoyer helper.getMainFrame() sans succès.

Nous avons constaté le même problème sur le lien en haut à gauche dans l'administration. En effet, lorsqu'on prend un look différent du look initial, le lien renvoie vers la page MainFrameSilverpeasV5.jsp, et comme indiqué la redirection de celle-ci n'est pas bonne (il manque le contexte).

Pour comparaison, l'archetype du look génère le code suivant :

    if (!"/${rootArtifactId}/jsp/MainFrame${className}.jsp".equalsIgnoreCase(helper.getMainFrame())) {
        session.setAttribute("RedirectToSpaceId", spaceIdFromRedirect);
        %>
            <script type="text/javascript">
              top.location="<c:url><%=helper.getMainFrame()%></c:url>";
            </script>
        <%
    }

Le look client n'a pas été testé dans le cadre d'un retour au look initial silvpereas sur un espace car habituellement le look client est utilisé pour l'ensemble de l'application.
En fait si on changeait la configuration du look SilverpeasLight pour qu'il renvoi /admin/jsp/MainFrameSilverpeasV5.jsp au lieu de MainFrameSilverpeasV5.jsp cela devrait avoir le 'bon' comportement non ?

Je n'en suis pas sûr puisque tant qu'il y a cette ligne dans MainFrameSilverpeasV5.jsp:
top.location="<%=helper.getMainFrame()%>"
ça risque de ne pas marcher.
Il faut la changer par top.location="<c:url><%=helper.getMainFrame()%></c:url>"; et ajouter l'import des taglibs correspondant.

Je reviens sur ce sujet car nous avons découvert que le fil d'ariane souffrait du même problème.
En effet, le lien parent qui ouvre l'espace redirige aussi vers http://localhost/<lookClient>/jsp/MainFrame<LookClient>.jsp

En fait dans MainFrameSilverpeasV5.jsp je pense qu'il faudrait plutôt

if (!"MainFrameSilverpeasV5.jsp".equalsIgnoreCase(helper.getMainFrame())  && ! "/admin/jsp/MainFrameSilverpeasV5.jsp".equalsIgnoreCase(helper.getMainFrame())) {
  session.setAttribute("RedirectToSpaceId", spaceIdFromRedirect);
  String topLocation = gef.getLookFrame();
  if(!topLocation.startsWith("/")) {
    topLocation = "/admin/jsp/" + topLocation;
   }
 %>
  <c:set var="topLocation"><%=topLocation%></c:set>
  <script type="text/javascript">
    top.location="<c:url value="${topLocation}" />";
  </script>
 <%
 }

Je suis tombé sur une boucle de redirection.

Nous avons mis en place cette correction :

    if (!"MainFrameSilverpeasV5.jsp".equalsIgnoreCase(helper.getMainFrame()))
    {
        session.setAttribute("RedirectToSpaceId", spaceIdFromRedirect);
        %>
            <script type="text/javascript">
                top.location.href="<%=m_sContext%><%=helper.getMainFrame()%>";
            </script>
        <%
    }

Après tests, les problèmes que nous rencontrons lorsque l'utilisateur choisi un look viennent du fait que la page MainFrameSilverpeasV5 essaie de récupérer le paramètre defaultHomePage du look.

Hors à la génération du projet depuis l'archétype, le fichier properties du look ne contient pas ce paramètre.